Separate zkSync Tests to Separate CI Step

This PR moves the zkSync tests outside of the normal `test` NPM
script. Since this would make it no longer run by default in CI, we
added a separate `tests-zk` step to run the zkSync tests only.

This split allows us to potentially merge PRs when zkSync tests fail
as long as the normal test suite does not. It is not the first time
that CI starts failing without cause because of zkSync tests. As far
as I can tell, the zkSync tooling downloads an in-memory node binary
that it starts for actually executing the tests. In the past, tests
have randomly started to fail because of:

* Changes to the behaviour of the in-memory node ([example](https://github.com/safe-global/safe-smart-account/actions/runs/14359540825/job/40257156305?pr=938))
* Binary no longer hosted on the same URL ([example](https://github.com/safe-global/safe-smart-account/actions/runs/14359540825/job/40257156305?pr=938))

When this happens, we do not want to block changes that are unrelated
to contract logic (as they should not, in theory, have any affect on
the zkSync tests).

Furthermore, running the zkSync tests takes quite a bit longer than the
regular tests, so this also makes local `npm test` a bit faster.

Author: nlordell

.github/workflows/ci.yml

@@ -41,6 +41,18 @@ jobs:
             - run: npm run build
             - run: npm run test
 
+    tests-zk:
+        runs-on: ubuntu-latest
+        steps:
+            - uses: actions/checkout@v4
+            - uses: actions/setup-node@v4
+              with:
+                  node-version: ${{ env.NODE_VERSION }}
+                  cache: "npm"
+            - run: npm ci
+            - run: npm run build
+            - run: npm run test:zk
+
     coverage:
         runs-on: ubuntu-latest
         strategy:

package.json

@@ -18,11 +18,12 @@
         "build:zk": "HARDHAT_ENABLE_ZKSYNC=1 hardhat compile",
         "build:ts": "rimraf dist && tsc -p tsconfig.prod.json",
         "build:ts:dev": "rimraf dist && tsc -p tsconfig.json",
-        "test:zk": "HARDHAT_ENABLE_ZKSYNC=1 hardhat test",
-        "test": "hardhat test --network hardhat && npm run test:L1 && npm run test:L2 && npm run test:zk",
-        "test:parallel": "hardhat test --parallel",
+        "test": "npm run test:hardhat && npm run test:L1 && npm run test:L2",
+        "test:all": "npm run test && npm run test:zk",
+        "test:hardhat": "hardhat test --network hardhat",
         "test:L1": "HARDHAT_CHAIN_ID=1 hardhat test --grep '@L1'",
         "test:L2": "SAFE_CONTRACT_UNDER_TEST=SafeL2 hardhat test --network hardhat",
+        "test:zk": "HARDHAT_ENABLE_ZKSYNC=1 hardhat test",
         "coverage": "hardhat coverage",
         "codesize": "hardhat codesize",
         "benchmark": "hardhat test benchmark/*.ts",