Upgrade to execution-spec-tests version 4.0.0 (hyperledger#8325)

Use fixtures_develop which contains Pectra test fixtures. Unignore the BLS execution-spec-tests, but keep ignoring the older reference tests version Signed-off-by: Simon Dudley <[email protected]>
fab-10 · Feb 19, 2025 · c1c3715 · c1c3715
1 parent 984173e
commit c1c3715
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 8 deletions.
diff --git a/ethereum/referencetests/build.gradle b/ethereum/referencetests/build.gradle
@@ -198,7 +198,7 @@ dependencies {
   referenceTestImplementation project(path: ':testutil')
   referenceTestImplementation project(path: ':util')
   // the following will be resolved via custom ivy repository declared in root build.gradle
-  referenceTestImplementation 'ethereum:execution-spec-tests:3.0.0:fixtures_stable@tar.gz'
+  referenceTestImplementation 'ethereum:execution-spec-tests:4.0.0:fixtures_develop@tar.gz'
   referenceTestImplementation 'com.fasterxml.jackson.core:jackson-databind'
   referenceTestImplementation 'com.google.guava:guava'
   referenceTestImplementation 'io.tmio:tuweni-bytes'

diff --git a/...rc/reference-test/java/org/hyperledger/besu/ethereum/vm/BlockchainReferenceTestTools.java b/...rc/reference-test/java/org/hyperledger/besu/ethereum/vm/BlockchainReferenceTestTools.java
@@ -92,8 +92,7 @@ public class BlockchainReferenceTestTools {
     // EOF tests don't have Prague stuff like deposits right now
     params.ignore("/stEOF/");
 
-    // TODO: remove once updated EIP-2537 gas cost artifacts exist
-    params.ignore("/eip2537_bls_12_381_precompiles/");
+    // These are for the older reference tests but EIP-2537 is covered by eip2537_bls_12_381_precompiles in the execution-spec-tests
     params.ignore("/stEIP2537/");
   }
 

diff --git a/.../reference-test/java/org/hyperledger/besu/ethereum/vm/GeneralStateReferenceTestTools.java b/.../reference-test/java/org/hyperledger/besu/ethereum/vm/GeneralStateReferenceTestTools.java
@@ -117,8 +117,7 @@ private static ProtocolSpec protocolSpec(final String name) {
     // EOF tests are written against an older version of the spec
     params.ignore("/stEOF/");
 
-    // TODO: remove once updated EIP-2537 gas cost artifacts exist
-    params.ignore("/eip2537_bls_12_381_precompiles/");
+    // These are for the older reference tests but EIP-2537 is covered by eip2537_bls_12_381_precompiles in the execution-spec-tests
     params.ignore("/stEIP2537/");
 
   }

diff --git a/gradle/verification-metadata.xml b/gradle/verification-metadata.xml
@@ -1834,9 +1834,12 @@
             <sha256 value="75ce469935cafad584bbe0905843b0a812dfd1e77824280e4dd0cb18d2acd729" origin="Generated by Gradle"/>
          </artifact>
       </component>
-      <component group="ethereum" name="execution-spec-tests" version="3.0.0">
-         <artifact name="execution-spec-tests-3.0.0-fixtures_stable.tar.gz">
-            <sha256 value="110c3f7808bebf33541efedb5e57fc22dd1f33f83220343b1d720032739e785b" origin="Generated by Gradle"/>
+      <component group="ethereum" name="execution-spec-tests" version="4.0.0">
+         <artifact name="execution-spec-tests-4.0.0-fixtures_stable.tar.gz">
+            <sha256 value="fe1979d35ada8fe183175b6daa47bf2d726d3d06ea8016426327df764739d9a1" origin="Generated by Gradle"/>
+         </artifact>
+         <artifact name="execution-spec-tests-4.0.0-fixtures_develop.tar.gz">
+            <sha256 value="cbc98a7d7ee63f367b4fe032fbd18101c8f809464b2ebbd7e116b02a198f785f" origin="Generated by Gradle"/>
          </artifact>
       </component>
       <component group="info.picocli" name="picocli" version="3.0.0">