fix(ci): lint test names after running tests (#13037)

Moves the task for linting test names to after the tests job so
that the contract artifacts actually exist. We don't compile
everything ahead of time and rely on forge test to do the
compilation in an effort to cut down total compile time.

.circleci/config.yml

@@ -684,10 +684,6 @@ jobs:
           name: Build go-ffi
           command: just build-go-ffi
           working_directory: packages/contracts-bedrock
-      - run:
-          name: Lint forge test names
-          command: just lint-forge-tests-check-no-build
-          working_directory: packages/contracts-bedrock
       - run:
           name: Run tests
           command: |
@@ -707,6 +703,10 @@ jobs:
             FOUNDRY_PROFILE: ci
           working_directory: packages/contracts-bedrock
           when: on_fail
+      - run:
+          name: Lint forge test names
+          command: just lint-forge-tests-check-no-build
+          working_directory: packages/contracts-bedrock
       - save_cache:
           name: Save Go build cache
           key: golang-build-cache-contracts-bedrock-tests-{{ checksum "go.sum" }}