Skip to content

Commit 3b85441

Browse files
fix: Consolidate some EVM E2E tests into one (#2423)
<!-- Please read and fill out this form before submitting your PR. Please make sure you have reviewed our contributors guide before submitting your first PR. NOTE: PR titles should follow semantic commits: https://www.conventionalcommits.org/en/v1.0.0/ --> ## Overview Reduces some container restart overhead by consolidating EVM tests Before: TestEvmSequencerE2E: 14.63s TestEvmMultipleTransactionInclusionE2E: 19.64s TestEvmInvalidTransactionRejectionE2E: 15.59s Now: TestEvmSequencerComprehensiveE2E ~21 seconds <!-- Please provide an explanation of the PR, including the appropriate context, background, goal, and rationale. If there is an issue with this information, please provide a tl;dr and link the issue. Ex: Closes #<issue number> --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Tests** * Combined multiple end-to-end tests for the EVM sequencer into a single comprehensive test, improving efficiency and providing clearer progress updates and summaries. The new test covers basic transactions, high-throughput scenarios, and invalid transaction rejection within one streamlined flow. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent dcf6c3c commit 3b85441

1 file changed

Lines changed: 82 additions & 181 deletions

File tree

0 commit comments

Comments
 (0)