Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: wait for tx to be re-included #9964

Merged
merged 4 commits into from
Nov 14, 2024
Merged

fix: wait for tx to be re-included #9964

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
4068b47
fix: wait for tx to be re-included
alexghr Nov 14, 2024
83e77e4
Update yarn-project/end-to-end/src/prover-coordination/e2e_prover_coo…
alexghr Nov 14, 2024
b641d1e
fix: import sleep
alexghr Nov 14, 2024
f6a8511
Merge branch 'master' into ag/fix-prover-coordination
PhilWindle Nov 14, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions yarn-project/end-to-end/src/prover-coordination/e2e_prover_coordination.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ import {
EpochProofQuotePayload,
TxStatus,
createDebugLogger,
retryUntil,
sleep,
} from '@aztec/aztec.js';
import { type AztecAddress, EthAddress } from '@aztec/circuits.js';
Expand Down Expand Up @@ -402,6 +403,12 @@ describe('e2e_prover_coordination', () => {

// Wait a bit for the sequencer / node to notice a re-org
await sleep(2000);
await retryUntil(
async () => (await ctx.aztecNode.getTxReceipt(tx2BeforeReorg.txHash)).status === TxStatus.SUCCESS,
'wait for re-inclusion',
60,
1,
);
alexghr marked this conversation as resolved.
Show resolved Hide resolved

// the sequencer will add valid txs again but in a new block
const tx2AfterReorg = await ctx.aztecNode.getTxReceipt(tx2BeforeReorg.txHash);
Expand Down
Loading