tweaks suggested by reviewer

Consensys · Jan 17, 2025 · 5ae61e9 · 5ae61e9
1 parent 998091e
commit 5ae61e9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/.github/workflows/reference-blockchain-tests.yml b/.github/workflows/reference-blockchain-tests.yml
@@ -44,12 +44,11 @@ jobs:
         uses: ./.github/actions/setup-environment
         with:
           go-corset: true
-        
+
       - name: Generate block chain reference tests
         run: ./gradlew :reference-tests:generateBlockchainReferenceTests -Dorg.gradle.caching=true
         env:
           JAVA_OPTS: -Dorg.gradle.daemon=false
-          ZKEVM_BIN: "zkevm_for_reference_tests.go.bin"
 
       - name: Download artifact
         uses: dawidd6/action-download-artifact@v6

diff --git a/reference-tests/build.gradle b/reference-tests/build.gradle
@@ -89,9 +89,8 @@ tasks.register('referenceGeneralStateTests', Test) {
   tasks.named("referenceBlockchainTests"),
 ].forEach {
   it.configure {
-    def ZKEVM_REFTESTS_BIN = System.getenv("ZKEVM_BIN") ?: "zkevm_for_reference_tests.bin"
     // Record zkevm.bin for CorsetValidator
-    environment.put("ZKEVM_BIN", ZKEVM_REFTESTS_BIN)
+    environment.put("ZKEVM_BIN", System.getenv("ZKEVM_BIN") ?: "zkevm_for_reference_tests.bin")
     // Configure parallelism
     systemProperty("junit.jupiter.execution.timeout.default", "15 m")  // 5 minutes
     systemProperty("junit.jupiter.execution.parallel.enabled", "true")