Commit 20c0e146 authored by Inphi's avatar Inphi Committed by GitHub

ci: Skip expensive fp e2e tests in main workflow (#12780)

Co-authored-by: default avatarMatthew Slipper <me@matthewslipper.com>
parent 1f8044e5
...@@ -897,6 +897,10 @@ jobs: ...@@ -897,6 +897,10 @@ jobs:
description: Machine resource class description: Machine resource class
type: string type: string
default: ethereum-optimism/latitude-1 default: ethereum-optimism/latitude-1
skip_slow_tests:
description: Indicates that slow tests should be skipped
type: boolean
default: false
machine: true machine: true
resource_class: <<parameters.resource_class>> resource_class: <<parameters.resource_class>>
steps: steps:
...@@ -925,6 +929,7 @@ jobs: ...@@ -925,6 +929,7 @@ jobs:
# need to explicitly set it here to prevent Cannon from running when we don't # need to explicitly set it here to prevent Cannon from running when we don't
# want it to. # want it to.
export OP_E2E_CANNON_ENABLED="false" export OP_E2E_CANNON_ENABLED="false"
export OP_E2E_SKIP_SLOW_TEST=<<parameters.skip_slow_tests>>
# Note: We don't use circle CI test splits because we need to split by test name, not by package. There is an additional # Note: We don't use circle CI test splits because we need to split by test name, not by package. There is an additional
# constraint that gotestsum does not currently (nor likely will) accept files from different packages when building. # constraint that gotestsum does not currently (nor likely will) accept files from different packages when building.
JUNIT_FILE=../tmp/test-results/<<parameters.module>>_<<parameters.target>>.xml JSON_LOG_FILE=../tmp/testlogs/test.log make <<parameters.target>> JUNIT_FILE=../tmp/test-results/<<parameters.module>>_<<parameters.target>>.xml JSON_LOG_FILE=../tmp/testlogs/test.log make <<parameters.target>>
...@@ -1362,6 +1367,7 @@ workflows: ...@@ -1362,6 +1367,7 @@ workflows:
name: op-e2e-fault-proof-tests name: op-e2e-fault-proof-tests
module: op-e2e module: op-e2e
target: test-fault-proofs target: test-fault-proofs
skip_slow_tests: true
requires: requires:
- contracts-bedrock-build - contracts-bedrock-build
- cannon-prestate - cannon-prestate
......
...@@ -52,6 +52,13 @@ func UsesCannon(t e2eutils.TestingBase) { ...@@ -52,6 +52,13 @@ func UsesCannon(t e2eutils.TestingBase) {
} }
} }
// IsSlow indicates that the test is too expensive to run on the main CI workflow
func IsSlow(t e2eutils.TestingBase) {
if os.Getenv("OP_E2E_SKIP_SLOW_TEST") == "true" {
t.Skip("Skipping slow test")
}
}
type executorInfo struct { type executorInfo struct {
total uint64 total uint64
idx uint64 idx uint64
......
...@@ -167,7 +167,7 @@ func TestOutputAlphabetGame_ValidOutputRoot(t *testing.T) { ...@@ -167,7 +167,7 @@ func TestOutputAlphabetGame_ValidOutputRoot(t *testing.T) {
} }
func TestChallengerCompleteExhaustiveDisputeGame(t *testing.T) { func TestChallengerCompleteExhaustiveDisputeGame(t *testing.T) {
op_e2e.InitParallel(t) op_e2e.InitParallel(t, op_e2e.IsSlow)
testCase := func(t *testing.T, isRootCorrect bool) { testCase := func(t *testing.T, isRootCorrect bool) {
ctx := context.Background() ctx := context.Background()
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment