Skip to content

Commit b82f5cc

Browse files
committed
chore(fast-inbox): fix l1 publisher revert assertion after inHash removal (A-1387)
The propose-failure test asserted the deleted Rollup__InvalidInHash selector (0xcd6f4233). With the legacy inHash L1 check gone, a checkpoint carrying messages that do not match the Inbox now reverts at validateInboxConsumption with Rollup__InvalidInboxRollingHash (0xed1f7bb5).
1 parent f3596c0 commit b82f5cc

1 file changed

Lines changed: 4 additions & 5 deletions

File tree

yarn-project/sequencer-client/src/publisher/l1_publisher.integration.test.ts

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -969,9 +969,8 @@ describe('L1Publisher integration', () => {
969969
});
970970

971971
it(`shows propose custom errors if tx simulation fails`, async () => {
972-
// Set up different l1-to-l2 messages than the ones on the inbox, so this submission reverts because the
973-
// INBOX.consume does not match the header.inHash and we get a Rollup__BlobHash that is not caught by
974-
// validateHeader before.
972+
// Set up different l1-to-l2 messages than the ones on the inbox, so the checkpoint's inboxRollingHash does not
973+
// match the referenced Inbox bucket and the submission reverts at the streaming-consumption check.
975974
const l1ToL2Messages = new Array(MAX_L1_TO_L2_MSGS_PER_CHECKPOINT).fill(new Fr(1n));
976975
const { checkpoint } = await buildSingleCheckpoint({ l1ToL2Messages });
977976

@@ -987,12 +986,12 @@ describe('L1Publisher integration', () => {
987986
await progressToSlot(BigInt(checkpoint.header.slotNumber));
988987
const result = await publisher.sendRequests();
989988
expect(result).toBeUndefined();
990-
// 0xcd6f4233 == Rollup__InvalidInHash selector
989+
// 0xed1f7bb5 == Rollup__InvalidInboxRollingHash selector
991990
expect(loggerWarnSpy).toHaveBeenCalledWith(
992991
'Bundle entry dropped: action reverted in sim',
993992
expect.objectContaining({
994993
action: 'propose',
995-
returnData: expect.stringMatching(/^0xcd6f4233/),
994+
returnData: expect.stringMatching(/^0xed1f7bb5/),
996995
}),
997996
);
998997
});

0 commit comments

Comments
 (0)