skip tests with empty blocks.

Consensys · Oct 10, 2024 · 6d0fb0e · 6d0fb0e
1 parent aabc693
commit 6d0fb0e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/reference-tests/src/main/java/net/consensys/linea/ReferenceTestOutcomeWriter.java b/reference-tests/src/main/java/net/consensys/linea/ReferenceTestOutcomeWriter.java
@@ -37,7 +37,7 @@ public void launcherSessionClosed(LauncherSession session) {
     try {
       String directory =  setFileDirectory();
       log.info("Reference test will be written to file {}\\{}", directory, JSON_OUTPUT_FILENAME);
-      Void result = writeToJsonFile().get();
+      writeToJsonFile().get();
       log.info("Reference test results written to file {}", JSON_OUTPUT_FILENAME);
       log.info("Path exists: {}, file exist: {}", Paths.get(directory).toFile().exists(), Paths.get(directory).resolve(JSON_OUTPUT_FILENAME).toFile().exists());
     } catch (InterruptedException | ExecutionException e) {

diff --git a/reference-tests/src/test/java/net/consensys/linea/BlockchainReferenceTestTools.java b/reference-tests/src/test/java/net/consensys/linea/BlockchainReferenceTestTools.java
@@ -53,6 +53,7 @@
 import org.hyperledger.besu.ethereum.referencetests.ReferenceTestProtocolSchedules;
 import org.hyperledger.besu.ethereum.rlp.RLPException;
 import org.hyperledger.besu.testutil.JsonTestParameters;
+import org.junit.jupiter.api.Assumptions;
 
 @Slf4j
 public class BlockchainReferenceTestTools {
@@ -197,6 +198,8 @@ public static void executeTest(final BlockchainReferenceTestCaseSpec spec) {
       try {
         final Block block = candidateBlock.getBlock();
 
+        Assumptions.assumeTrue(block.getBody().getTransactions().size() > 0, "Skipping the test because the block has no transaction");
+
         zkTracer.traceStartBlock(block.getHeader());
 
         final ProtocolSpec protocolSpec = schedule.getByBlockHeader(block.getHeader());