Merge pull request #4303 from ethereum-optimism/jm/ci-verify-test-names

Verify test names in CI

packages/contracts-bedrock/package.json

@@ -31,7 +31,8 @@
     "slither": "./scripts/slither.sh",
     "clean": "rm -rf ./dist ./artifacts ./forge-artifacts ./cache ./tsconfig.tsbuildinfo ./src/contract-artifacts.ts",
     "lint:ts:check": "eslint . --max-warnings=0",
-    "lint:contracts:check": "yarn solhint -f table 'contracts/**/*.sol'",
+    "lint:forge-tests:check": "ts-node scripts/forge-test-names.ts",
+    "lint:contracts:check": "yarn solhint -f table 'contracts/**/*.sol' && yarn lint:forge-tests:check",
     "lint:check": "yarn lint:contracts:check && yarn lint:ts:check",
     "lint:ts:fix": "eslint --fix .",
     "lint:contracts:fix": "yarn prettier --write 'contracts/**/*.sol'",

packages/contracts-bedrock/scripts/forge-test-names.ts

@@ -78,8 +78,8 @@ const main = async () => {
     }
   }
 
-  if (errors) {
-    console.error(errors.join('\n'))
+  if (errors.length > 0) {
+    console.error(...errors)
     process.exit(1)
   }
 }