Skip to content

fix: use correct EthCheatCodes method name in epochs_missed_l1_slot test#21848

Merged
spalladino merged 1 commit intomerge-train/spartanfrom
claudebox/fix-merge-train-spartan-21769
Mar 20, 2026
Merged

fix: use correct EthCheatCodes method name in epochs_missed_l1_slot test#21848
spalladino merged 1 commit intomerge-train/spartanfrom
claudebox/fix-merge-train-spartan-21769

Conversation

@AztecBot
Copy link
Collaborator

Summary

Fix TypeScript compilation error in the new epochs_missed_l1_slot.test.ts introduced by #21769.

The test calls eth.timestamp() but the method on EthCheatCodes is actually lastBlockTimestamp(). This caused yarn tsgo -b --emitDeclarationOnly to fail with:

error TS2339: Property 'timestamp' does not exist on type 'EthCheatCodes'.

Fix

Changed eth.timestamp()eth.lastBlockTimestamp() on line 104 of the test file.

ClaudeBox log: https://claudebox.work/s/e62ab0794047aa8c?run=1

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Mar 20, 2026
@spalladino spalladino marked this pull request as ready for review March 20, 2026 15:11
@spalladino spalladino enabled auto-merge (squash) March 20, 2026 15:11
@spalladino spalladino merged commit af4228d into merge-train/spartan Mar 20, 2026
48 of 52 checks passed
@spalladino spalladino deleted the claudebox/fix-merge-train-spartan-21769 branch March 20, 2026 15:11
github-merge-queue bot pushed a commit that referenced this pull request Mar 20, 2026
BEGIN_COMMIT_OVERRIDE
feat(p2p): add tx validator for contract instance deployment addresses
(#21771)
fix: always deploy IRM for testnet (#21755)
fix: avoid mutating caller's array via splice in snapshot sync (A-718)
(#21759)
chore: update network logs skill (#21785)
feat(archiver): validate contract instance addresses before storing
(#21787)
fix: ensure no division by 0 (#21786)
feat: support private fork releases via ci-release (#21778)
fix: restrict scenario deployments to only nightly (#21798)
fix(stdlib): zero-pad bufferFromFields when declared length exceeds
payload (#21802)
test(protocol-contracts): verify max-size bytecode fits in contract
class log (#21818)
fix: wire BOT_DA_GAS_LIMIT through helm/terraform for staging-public
(#21809)
fix: remove jest-mock-extended from worker processes + fix
parallelize_strict silent failures (#21821)
fix(archiver): throw on duplicate contract class or instance additions
(#21799)
chore: remove broadcasted function events (#21805)
fix: sync dateProvider from anvil stdout on every mined block (#21829)
fix(sequencer): use wall-clock time instead of L1 block timestamp for
slot estimation (#21769)
fix: use correct EthCheatCodes method name in epochs_missed_l1_slot test
(#21848)
feat(p2p): add tx validator for contract class id verification (#21788)
feat: publisher funding (#21631)
feat: batch chonk verifier TS integration (#21823)
fix(sequencer): remove l1 block timestamp check (#21853)
fix: use local IVC inputs for batch_verifier bench test (#21857)
fix(p2p): centralize gossipsub penalization and fix inconsistencies
(#21863)
chore: publish GitHub releases to AztecProtocol/barretenberg (#21775)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants