From f76eaf6150981bc94d0e51f19fcae6856efa81ca Mon Sep 17 00:00:00 2001 From: MirandaWood Date: Mon, 25 Nov 2024 13:36:38 +0000 Subject: [PATCH] chore: disable e2e_epoch due to teardown timeout --- scripts/ci/get_e2e_jobs.sh | 3 ++- yarn-project/end-to-end/scripts/e2e_test_config.yml | 3 ++- yarn-project/end-to-end/src/e2e_epochs.test.ts | 9 ++++++--- 3 files changed, 10 insertions(+), 5 deletions(-) diff --git a/scripts/ci/get_e2e_jobs.sh b/scripts/ci/get_e2e_jobs.sh index 2dbdb42ac40..26d3853b320 100755 --- a/scripts/ci/get_e2e_jobs.sh +++ b/scripts/ci/get_e2e_jobs.sh @@ -25,7 +25,8 @@ allow_list=( "e2e_cross_chain_messaging" "e2e_crowdfunding_and_claim" "e2e_deploy_contract" - "e2e_epochs" + # TODO(#10177): reenable + # "e2e_epochs" "e2e_fees" "e2e_fees_failures" "e2e_fees_gas_estimation" diff --git a/yarn-project/end-to-end/scripts/e2e_test_config.yml b/yarn-project/end-to-end/scripts/e2e_test_config.yml index ffee94e2933..d126d8e259e 100644 --- a/yarn-project/end-to-end/scripts/e2e_test_config.yml +++ b/yarn-project/end-to-end/scripts/e2e_test_config.yml @@ -35,7 +35,8 @@ tests: e2e_devnet_smoke: {} docs_examples: use_compose: true - e2e_epochs: {} + # TODO(#10177): reenable + # e2e_epochs: {} e2e_escrow_contract: {} e2e_fees_account_init: test_path: 'e2e_fees/account_init.test.ts' diff --git a/yarn-project/end-to-end/src/e2e_epochs.test.ts b/yarn-project/end-to-end/src/e2e_epochs.test.ts index 3ac4e07afd8..ea5a38bbbcb 100644 --- a/yarn-project/end-to-end/src/e2e_epochs.test.ts +++ b/yarn-project/end-to-end/src/e2e_epochs.test.ts @@ -25,7 +25,7 @@ describe('e2e_epochs', () => { let handle: NodeJS.Timeout; const EPOCH_DURATION = 4; - const L1_BLOCK_TIME = 5; + const L1_BLOCK_TIME = 25; const L2_SLOT_DURATION_IN_L1_BLOCKS = 2; beforeAll(async () => { @@ -98,6 +98,7 @@ describe('e2e_epochs', () => { afterAll(async () => { clearInterval(handle); + // TODO(#10177): The below times out await context.teardown(); }); @@ -111,7 +112,7 @@ describe('e2e_epochs', () => { /** Waits until the given L2 block number is mined. */ const waitUntilL2BlockNumber = async (target: number) => { - await retryUntil(() => Promise.resolve(target === l2BlockNumber), `Wait until L2 block ${l2BlockNumber}`, 60, 0.1); + await retryUntil(() => Promise.resolve(target === l2BlockNumber), `Wait until L2 block ${target}`, 200, 0.1); }; it('does not allow submitting proof after epoch end', async () => { @@ -129,6 +130,8 @@ describe('e2e_epochs', () => { sequencerDelayer.pauseNextTxUntilTimestamp(epoch2Start + BigInt(L1_BLOCK_TIME)); // Next sequencer to publish a block should trigger a rollback to block 1 + // The below is a bit of a hack - to avoid the waitUntilL1Timestamp timing out, I wait until the reorg back to block 1 is complete + await waitUntilL2BlockNumber(1); await waitUntilL1Timestamp(l1Client, epoch2Start + BigInt(L1_BLOCK_TIME)); expect(await rollup.getBlockNumber()).toEqual(1n); expect(await rollup.getSlotNumber()).toEqual(8n); @@ -142,5 +145,5 @@ describe('e2e_epochs', () => { const lastL2BlockTxReceipt = await l1Client.getTransactionReceipt({ hash: lastL2BlockTxHash! }); expect(lastL2BlockTxReceipt.status).toEqual('success'); expect(lastL2BlockTxReceipt.blockNumber).toBeGreaterThan(lastProverTxReceipt!.blockNumber); - }); + }, 400_000); });