From 6d808560c92f7f5b5c2128e2413727092463d6e0 Mon Sep 17 00:00:00 2001 From: inphi Date: Wed, 4 Sep 2024 14:39:42 -0400 Subject: [PATCH] cannon: Run expensive tests in parallel --- cannon/mipsevm/tests/evm_common_test.go | 9 +++++++++ cannon/mipsevm/tests/evm_multithreaded_test.go | 4 ++++ 2 files changed, 13 insertions(+) diff --git a/cannon/mipsevm/tests/evm_common_test.go b/cannon/mipsevm/tests/evm_common_test.go index 063c63c83c6c..437501140f25 100644 --- a/cannon/mipsevm/tests/evm_common_test.go +++ b/cannon/mipsevm/tests/evm_common_test.go @@ -454,11 +454,14 @@ func TestEVMFault(t *testing.T) { } func TestHelloEVM(t *testing.T) { + t.Parallel() var tracer *tracing.Hooks // no-tracer by default, but see test_util.MarkdownTracer versions := GetMipsVersionTestCases(t) for _, v := range versions { + v := v t.Run(v.Name, func(t *testing.T) { + t.Parallel() evm := testutil.NewMIPSEVM(v.Contracts) evm.SetTracer(tracer) testutil.LogStepFailureAtCleanup(t, evm) @@ -502,11 +505,14 @@ func TestHelloEVM(t *testing.T) { } func TestClaimEVM(t *testing.T) { + t.Parallel() var tracer *tracing.Hooks // no-tracer by default, but see test_util.MarkdownTracer versions := GetMipsVersionTestCases(t) for _, v := range versions { + v := v t.Run(v.Name, func(t *testing.T) { + t.Parallel() evm := testutil.NewMIPSEVM(v.Contracts) evm.SetTracer(tracer) testutil.LogStepFailureAtCleanup(t, evm) @@ -549,11 +555,14 @@ func TestClaimEVM(t *testing.T) { } func TestEntryEVM(t *testing.T) { + t.Parallel() var tracer *tracing.Hooks // no-tracer by default, but see test_util.MarkdownTracer versions := GetMipsVersionTestCases(t) for _, v := range versions { + v := v t.Run(v.Name, func(t *testing.T) { + t.Parallel() evm := testutil.NewMIPSEVM(v.Contracts) evm.SetTracer(tracer) testutil.LogStepFailureAtCleanup(t, evm) diff --git a/cannon/mipsevm/tests/evm_multithreaded_test.go b/cannon/mipsevm/tests/evm_multithreaded_test.go index ebd5dc904e28..76f9b9462dc2 100644 --- a/cannon/mipsevm/tests/evm_multithreaded_test.go +++ b/cannon/mipsevm/tests/evm_multithreaded_test.go @@ -623,6 +623,7 @@ func TestEVM_NoopSyscall(t *testing.T) { } func TestEVM_UnsupportedSyscall(t *testing.T) { + t.Parallel() var tracer *tracing.Hooks var NoopSyscallNums = maps.Values(NoopSyscalls) @@ -638,7 +639,10 @@ func TestEVM_UnsupportedSyscall(t *testing.T) { for i, syscallNum := range unsupportedSyscalls { testName := fmt.Sprintf("Unsupported syscallNum %v", syscallNum) + i := i + syscallNum := syscallNum t.Run(testName, func(t *testing.T) { + t.Parallel() goVm, state, contracts := setup(t, i*3434) // Setup basic getThreadId syscall instruction state.Memory.SetMemory(state.GetPC(), syscallInsn)