From 1e0db860895908ea8aaab2c8c1d7973b290ffe04 Mon Sep 17 00:00:00 2001 From: DavePearce Date: Mon, 25 Nov 2024 09:50:50 +1300 Subject: [PATCH] Enable previously disabled tests This enables a number of previously disabled tests which actually pass. --- .../linea/replaytests/Issue1031Tests.java | 1 - .../linea/replaytests/Issue1116Tests.java | 1 - .../linea/replaytests/Issue1126Tests.java | 17 ----------------- .../linea/replaytests/Issue1216Tests.java | 2 -- .../linea/replaytests/LeosNewRanges.java | 4 ++-- .../replaytests/LeosNewRangesSplitTests.java | 4 ++-- .../linea/replaytests/ReplayTests.java | 12 ------------ 7 files changed, 4 insertions(+), 37 deletions(-) diff --git a/arithmetization/src/test/java/net/consensys/linea/replaytests/Issue1031Tests.java b/arithmetization/src/test/java/net/consensys/linea/replaytests/Issue1031Tests.java index 397247e380..b2f146011e 100644 --- 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 index 468dc64a31..64801859ea 100644 --- 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 index 9de1f197f4..d4b5060ea0 100644 --- 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 index 38514d60ac..e022b51172 100644 --- 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 index d4c9dfeabe..14e7b17d29 100644 --- 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 index feea117eef..f0d9202972 100644 --- 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 index 861fa82efc..e051625432 100644 --- 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");