Deflake sequential proxy swap GIGA test#3485
Conversation
The Autobahn EVM GIGA integration job intermittently failed with "incorrect account sequence" on tokenA.approve inside a loop that issued approve+swap pairs back-to-back on the shared owner signer. Approve once up front (matching the other multi-swap test in this file) and wait between swaps so the live docker cluster's account sequence stays aligned with ethers' nonce tracking. This removes the extra per-iteration approve that widened the CheckTx race window without changing what the test exercises (two sequential multi-hop proxy swaps).
PR SummaryLow Risk Overview This reduces intermittent "incorrect account sequence" failures in environments where nonce/sequence updates can lag when submitting consecutive transactions from the same signer. Reviewed by Cursor Bugbot for commit e8cb3e9. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 395de04. Configure here.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3485 +/- ##
==========================================
+ Coverage 59.11% 59.13% +0.01%
==========================================
Files 2187 2187
Lines 182237 181998 -239
==========================================
- Hits 107723 107618 -105
+ Misses 64852 64719 -133
+ Partials 9662 9661 -1
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
| const receipt = await tx.wait(); | ||
| expect(receipt.status).to.equal(1); | ||
| if (i < 1) { | ||
| await delay(); |
There was a problem hiding this comment.
nit: I think delay() is fine, we could also check the sequence number like in #3486
But we can see whether delay just fixes the problem

The Autobahn EVM GIGA integration job intermittently failed with "incorrect account sequence" on tokenA.approve inside a loop that issued approve+swap pairs back-to-back on the shared owner signer.
Approve once up front (matching the other multi-swap test in this file) and wait between swaps so the live docker cluster's account sequence stays aligned with ethers' nonce tracking. This removes the extra per-iteration approve that widened the CheckTx race window without changing what the test exercises (two sequential multi-hop
proxy swaps).
Flaked in unrelated changes.