Skip to content

Commit

Permalink
chore: disable e2e_epoch due to teardown timeout
Browse files Browse the repository at this point in the history
  • Loading branch information
MirandaWood committed Nov 25, 2024
1 parent b787de6 commit f76eaf6
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 5 deletions.
3 changes: 2 additions & 1 deletion scripts/ci/get_e2e_jobs.sh
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Expand Down
3 changes: 2 additions & 1 deletion yarn-project/end-to-end/scripts/e2e_test_config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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'
Expand Down
9 changes: 6 additions & 3 deletions yarn-project/end-to-end/src/e2e_epochs.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -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 () => {
Expand Down Expand Up @@ -98,6 +98,7 @@ describe('e2e_epochs', () => {

afterAll(async () => {
clearInterval(handle);
// TODO(#10177): The below times out
await context.teardown();
});

Expand All @@ -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 () => {
Expand All @@ -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);
Expand All @@ -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);
});

0 comments on commit f76eaf6

Please sign in to comment.