Skip to content

test: deflake TestContractInvocationMultiple - #13630

Open
rvagg wants to merge 1 commit into
masterfrom
rvagg/deflake-TestContractInvocationMultiple
Open

test: deflake TestContractInvocationMultiple#13630
rvagg wants to merge 1 commit into
masterfrom
rvagg/deflake-TestContractInvocationMultiple

Conversation

@rvagg

@rvagg rvagg commented May 20, 2026

Copy link
Copy Markdown
Member

Closes: #12487

Drive-by fixup when I noticed this failing in my last PR, this should fix the flake.

@rvagg
rvagg requested review from Copilot and wjmelements May 20, 2026 12:15
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FilOz May 20, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Deflakes TestContractInvocationMultiple by adjusting the mining schedule and explicitly pausing/restarting mining around the “pending messages” assertion window to reduce timing races in the itest harness.

Changes:

  • Increased blockTime used by TestContractInvocationMultiple.
  • Captured the miners returned by BeginMining, paused them before submitting transactions, and restarted them afterward.
  • Added a “chain head stabilization” wait after pausing mining.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread itests/eth_transactions_test.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 📌 Triage

Development

Successfully merging this pull request may close these issues.

TestContractInvocationMultiple is flaky

2 participants