Enable previously disabled tests

This enables a number of previously disabled tests which actually pass. This also tidies up the naming of some of the regular tests.
Consensys · Nov 25, 2024 · 986e514 · 986e514
1 parent 82341f1
commit 986e514
Show file tree

Hide file tree

Showing 11 changed files with 12 additions and 45 deletions.
diff --git a/.github/workflows/gradle-ethereum-tests.yml b/.github/workflows/gradle-ethereum-tests.yml
@@ -1,4 +1,4 @@
-name: reference tests
+name: Daily State Tests
 
 on:
   schedule:
@@ -91,7 +91,7 @@ jobs:
         uses: actions/upload-artifact@v4
         with:
           name: ethereum-tests-go-corset-report
-          path: | 
+          path: |
             reference-tests/build/reports/tests/**/*
             tmp/local/*
 
@@ -102,5 +102,5 @@ jobs:
           webhook: ${{ secrets.SLACK_WEBHOOK_URL }}
           webhook-type: webhook-trigger
           payload: |
-             name: "Daily Reference Tests"
+             name: "Daily State"
              status: "${{ job.status }}"
diff --git a/.github/workflows/gradle-nightly-tests.yml b/.github/workflows/gradle-nightly-tests.yml
@@ -39,7 +39,7 @@ jobs:
         with:
           name: nightly-tests-report
           path: build/reports/tests/**/*
-          
+
       - name: Failure Notification
         if: ${{ failure() || cancelled() }}
         uses: slackapi/[email protected]

diff --git a/.github/workflows/gradle-weekly-tests.yml b/.github/workflows/gradle-weekly-tests.yml
@@ -65,7 +65,7 @@ jobs:
           CORSET_FLAGS: disable
           GOCORSET_FLAGS: -wd --ansi-escapes=false --report --air
           WEEKLY_TESTS_PARALLELISM: 4
-          
+
       - name: Failure Notification
         if: ${{ failure() || cancelled() }}
         uses: slackapi/[email protected]

diff --git a/.github/workflows/reference-blockchain-tests.yml b/.github/workflows/reference-blockchain-tests.yml
@@ -1,4 +1,4 @@
-name: Daily Reference Blockchain Tests
+name: Daily Blockchain Tests
 
 on:
   schedule:
@@ -88,13 +88,13 @@ jobs:
         with:
           name: blockchain-refrence-tests-report
           path: reference-tests/build/reports/tests/**/*
-          
+
       - name: Failure Notification
         if: ${{ failure() || cancelled() }}
         uses: slackapi/[email protected]
         with:
           webhook: ${{ secrets.SLACK_WEBHOOK_URL }}
           webhook-type: webhook-trigger
           payload: |
-             name: "Daily Reference Tests"
+             name: "Daily Blockchain"
              status: "${{ job.status }}"
diff --git a/arithmetization/src/test/java/net/consensys/linea/replaytests/Issue1031Tests.java b/arithmetization/src/test/java/net/consensys/linea/replaytests/Issue1031Tests.java
@@ -23,7 +23,6 @@
 @Tag("replay")
 public class Issue1031Tests {
 
-  // @Disabled
   @Test
   void issue_1090_range_10_20() {
     replay(LINEA_MAINNET, "10-20.json.gz");

diff --git a/arithmetization/src/test/java/net/consensys/linea/replaytests/Issue1116Tests.java b/arithmetization/src/test/java/net/consensys/linea/replaytests/Issue1116Tests.java
@@ -24,7 +24,6 @@
 @Tag("replay")
 public class Issue1116Tests {
 
-  // @Disabled
   @Test
   void issue_1116_block_8019521() {
     replay(LINEA_MAINNET, "8019521-8019521.json.gz");

diff --git a/arithmetization/src/test/java/net/consensys/linea/replaytests/Issue1126Tests.java b/arithmetization/src/test/java/net/consensys/linea/replaytests/Issue1126Tests.java
@@ -17,7 +17,6 @@
 import static net.consensys.linea.replaytests.ReplayTestTools.replay;
 import static net.consensys.linea.testing.ReplayExecutionEnvironment.LINEA_MAINNET;
 
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Tag;
 import org.junit.jupiter.api.Test;
 
@@ -63,97 +62,81 @@ void test_4477086_4477226() {
     replay(LINEA_MAINNET, "4477086-4477226.json.gz");
   }
 
-  @Disabled
   @Test
   void test_3290673_3290679() {
     replay(LINEA_MAINNET, "3290673-3290679.json.gz");
   }
 
-  @Disabled
   @Test
   void test_3290746_3290752() {
     replay(LINEA_MAINNET, "3290746-3290752.json.gz");
   }
 
-  @Disabled
   @Test
   void test_3315684_3315690() {
     replay(LINEA_MAINNET, "3315684-3315690.json.gz");
   }
 
-  @Disabled
   @Test
   void test_3374278_3374284() {
     replay(LINEA_MAINNET, "3374278-3374284.json.gz");
   }
 
-  @Disabled
   @Test
   void test_3385404_3385411() {
     replay(LINEA_MAINNET, "3385404-3385411.json.gz");
   }
 
-  @Disabled
   @Test
   void test_3410170_3410240() {
     replay(LINEA_MAINNET, "3410170-3410240.json.gz");
   }
 
-  @Disabled
   @Test
   void test_3423488_3423521() {
     replay(LINEA_MAINNET, "3423488-3423521.json.gz");
   }
 
-  @Disabled
   @Test
   void test_3424829_3424864() {
     replay(LINEA_MAINNET, "3424829-3424864.json.gz");
   }
 
-  @Disabled
   @Test
   void test_3429701_3429735() {
     replay(LINEA_MAINNET, "3429701-3429735.json.gz");
   }
 
-  @Disabled
   @Test
   void test_3431193_3431232() {
     replay(LINEA_MAINNET, "3431193-3431232.json.gz");
   }
 
-  @Disabled
   @Test
   void test_3431567_3431601() {
     replay(LINEA_MAINNET, "3431567-3431601.json.gz");
   }
 
-  @Disabled
   @Test
   void test_4323962_4324012() {
     replay(LINEA_MAINNET, "4323962-4324012.json.gz");
   }
 
-  @Disabled
   @Test
   void test_4343434_4343473() {
     replay(LINEA_MAINNET, "4343434-4343473.json.gz");
   }
 
-  @Disabled
   @Test
   void test_4519246_4519309() {
     replay(LINEA_MAINNET, "4519246-4519309.json.gz");
   }
 
-  @Disabled
   @Test
   void test_4556007_4556115() {
     replay(LINEA_MAINNET, "4556007-4556115.json.gz");
   }
 
-  @Disabled
   @Test
   void test_4583379_4583463() {
     replay(LINEA_MAINNET, "4583379-4583463.json.gz");

diff --git a/arithmetization/src/test/java/net/consensys/linea/replaytests/Issue1216Tests.java b/arithmetization/src/test/java/net/consensys/linea/replaytests/Issue1216Tests.java
@@ -20,7 +20,6 @@
 import net.consensys.linea.zktracer.module.hub.AccountSnapshot;
 import net.consensys.linea.zktracer.module.hub.Hub;
 import org.hyperledger.besu.datatypes.Address;
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Tag;
 import org.junit.jupiter.api.Test;
 
@@ -42,7 +41,6 @@ public class Issue1216Tests {
    * blowing up with a NPE since the "after" version of the self destructor didn't exist yet in the
    * code. This was solved in issue #1216.
    */
-  @Disabled
   @Tag("nightly")
   @Tag("replay")
   @Test

diff --git a/arithmetization/src/test/java/net/consensys/linea/replaytests/LeosNewRanges.java b/arithmetization/src/test/java/net/consensys/linea/replaytests/LeosNewRanges.java
@@ -17,10 +17,10 @@
 import static net.consensys.linea.replaytests.ReplayTestTools.replay;
 import static net.consensys.linea.testing.ReplayExecutionEnvironment.LINEA_MAINNET;
 
-import org.junit.jupiter.api.Disabled;
+import org.junit.jupiter.api.Tag;
 import org.junit.jupiter.api.Test;
 
-@Disabled
+@Tag("nightly")
 public class LeosNewRanges {
   @Test
   void leos_new_ranges_2258472_2258607() {

diff --git a/arithmetization/src/test/java/net/consensys/linea/replaytests/LeosNewRangesSplitTests.java b/arithmetization/src/test/java/net/consensys/linea/replaytests/LeosNewRangesSplitTests.java
@@ -17,10 +17,10 @@
 import static net.consensys.linea.replaytests.ReplayTestTools.replay;
 import static net.consensys.linea.testing.ReplayExecutionEnvironment.LINEA_MAINNET;
 
-import org.junit.jupiter.api.Disabled;
+import org.junit.jupiter.api.Tag;
 import org.junit.jupiter.api.Test;
 
-@Disabled
+@Tag("nightly")
 public class LeosNewRangesSplitTests {
   @Test
   void split_range_2258472_2258479() {

diff --git a/arithmetization/src/test/java/net/consensys/linea/replaytests/ReplayTests.java b/arithmetization/src/test/java/net/consensys/linea/replaytests/ReplayTests.java
@@ -51,50 +51,42 @@ void bulkReplay() {
     // bulkReplay(LINEA_MAINNET, "");
   }
 
-  // @Disabled
   @Test
   void failingMmuModexp() {
     replay(LINEA_MAINNET, "5995162.json.gz");
   }
 
-  // @Disabled
   @Test
   void failRlpAddress() {
     replay(LINEA_MAINNET, "5995097.json.gz");
   }
 
-  // @Disabled
   @Test
   void rlprcptManyTopicsWoLogData() {
     replay(LINEA_MAINNET, "6569423.json.gz");
   }
 
-  // @Disabled
   @Test
   void multipleFailingCallToEcrecover() {
     replay(LINEA_MAINNET, "5000544.json.gz");
   }
 
-  // @Disabled
   @Test
   @Tag("nightly")
   void incident777zkGethMainnet() {
     replay(LINEA_MAINNET, "7461019-7461030.json.gz");
   }
 
-  // @Disabled
   @Test
   void issue1006() {
     replay(LINEA_MAINNET, "6032696-6032699.json.gz");
   }
 
-  // @Disabled
   @Test
   void issue1004() {
     replay(LINEA_MAINNET, "6020023-6020029.json.gz");
   }
 
-  // @Disabled
   @Test
   void block_6110045() {
     // The purpose of this test is to check the mechanism for spotting divergence between the replay
@@ -103,19 +95,16 @@ void block_6110045() {
     replay(LINEA_MAINNET, "6110045.json.gz");
   }
 
-  // @Disabled
   @Test
   void failingCreate2() {
     replay(LINEA_MAINNET, "2250197-2250197.json.gz");
   }
 
-  // @Disabled
   @Test
   void blockHash1() {
     replay(LINEA_MAINNET, "8718090.json.gz");
   }
 
-  // @Disabled
   @Test
   void blockHash2() {
     replay(LINEA_MAINNET, "8718330.json.gz");
@@ -139,7 +128,6 @@ void failedCreate2() {
     replay(LINEA_MAINNET, "41640-41649.json.gz");
   }
 
-  @Disabled() // Unknown Problem
   @Test
   void largeInitCode() {
     replay(LINEA_SEPOLIA, "3318494.sepolia.json.gz");