Skip to content

Commit 8026d04

Browse files
committed
chore: deflake multiple invalidations e2e test
This test was failing since **more** than the expected checkpoints were getting invalidated. This was caused by a slow setup, where more invalid checkpoints than we expected were created. Since we only care that more than one checkpoint is invalidated for this test, this fix just removes the assertion in that at most one extra checkpoint was being invalidated.
1 parent 71ffa49 commit 8026d04

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

yarn-project/end-to-end/src/e2e_epochs/epochs_invalidate_block.parallel.test.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -381,10 +381,9 @@ describe('e2e_epochs/epochs_invalidate_block', () => {
381381
timeoutPromise(test.L2_SLOT_DURATION_IN_S * 4 * 1000).then(() => CheckpointNumber(0)),
382382
]);
383383

384-
// The invalidated checkpoint should be the first one
385-
// Note that it may also be a checkpoint *before* the first one that gets mined in `initialSlot + 1n`
384+
// The invalidated checkpoint should be the first one,
385+
// but it may also be a checkpoint *before* the first one that gets mined in an early slot
386386
expect(invalidatedCheckpoint).toBeLessThanOrEqual(firstCheckpoint);
387-
expect(invalidatedCheckpoint).toBeGreaterThanOrEqual(CheckpointNumber(firstCheckpoint - 1));
388387

389388
// Restore bad nodes back to normal. They should eventually detect that their archive root does not
390389
// match the value on chain and roll back their invalid nodes.

0 commit comments

Comments
 (0)