Separate zkSync Tests to Separate CI Step (#939)

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",