Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #80 +/- ##
=======================================
Coverage 95.71% 95.71%
=======================================
Files 42 42
Lines 9959 9959
=======================================
Hits 9532 9532
Misses 427 427 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
NirLevi-starkware
left a comment
There was a problem hiding this comment.
@NirLevi-starkware reviewed 2 of 3 files at r1, all commit messages.
Reviewable status: 2 of 3 files reviewed, 3 unresolved discussions (waiting on @noa-starkware)
src/flow_test/flow_ideas.md line 48 at r1 (raw file):
- staker enter in V2, advance epoch, upgrade to V3, advance epoch, attest, - staker enter in V0, upgrade to V3, attest - staker enter in V0, advance epoch, update balance, upgrade to V3, attest, update balance, advance epoch, attest
i dont see this flow
src/flow_test/flow_ideas.md line 50 at r1 (raw file):
- staker enter in V0, advance epoch, update balance, upgrade to V3, attest, update balance, advance epoch, attest - staker enter in V1, advance epoch, update balance, upgrade to V3, attest, update balance, advance epoch, attest - staker enter in V0, advance epoch, update balance, advance epoch, update balance, upgrade to V3, attest
i dont see this flow
noa-starkware
left a comment
There was a problem hiding this comment.
Reviewable status: 2 of 3 files reviewed, 3 unresolved discussions (waiting on @NirLevi-starkware)
src/flow_test/flow_ideas.md line 48 at r1 (raw file):
Previously, NirLevi-starkware (Nir Levi) wrote…
i dont see this flow
No epochs in V0
src/flow_test/flow_ideas.md line 50 at r1 (raw file):
Previously, NirLevi-starkware (Nir Levi) wrote…
i dont see this flow
same
NirLevi-starkware
left a comment
There was a problem hiding this comment.
@NirLevi-starkware reviewed 1 of 3 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @noa-starkware)
7462d37 to
2ac7a0e
Compare
Note
Adds three staker attestation migration flows (V0/V1 paths) with reward assertions and wires them into MAINNET_LATEST fork tests; updates flow ideas doc.
StakerV0AttestFlow: stake inV0, upgrade, multi-epoch attest/increase sequences; assert staker rewards viacalculate_staker_strk_rewards_with_balances_v2.StakerV1AttestFlow: stake/increase inV1, upgrade, attest/increase across epochs; assert rewards consistency before/after increases.StakerV1ChangeBalanceAttestFlow: multiple stake increases inV1, upgrade, attest; verify expected rewards.src/flow_test/fork_test.cairo.src/flow_test/flow_ideas.mdwith additional staker migration scenarios.Written by Cursor Bugbot for commit 2ac7a0e. This will update automatically on new commits. Configure here.
This change is