Skip to content

test: add get_stakers exit intent flow#27

Merged
arad-starkware merged 1 commit intomainfrom
arad/test_add_get_stakers_exit_intent_flow
Nov 19, 2025
Merged

test: add get_stakers exit intent flow#27
arad-starkware merged 1 commit intomainfrom
arad/test_add_get_stakers_exit_intent_flow

Conversation

@arad-starkware
Copy link
Collaborator

@arad-starkware arad-starkware commented Nov 17, 2025

This change is Reviewable


Note

Adds a test verifying get_stakers across epochs after a staker submits exit intent.

  • Tests: Add get_stakers_staker_exit_intent_flow_test in workspace/apps/staking/contracts/src/flow_test/test.cairo to validate get_stakers returns the staker in the current/next epoch after exit intent and excludes them from the following epoch.

Written by Cursor Bugbot for commit 1ff8fc2. This will update automatically on new commits. Configure here.

@arad-starkware arad-starkware self-assigned this Nov 17, 2025
@arad-starkware arad-starkware marked this pull request as ready for review November 17, 2025 12:51
Copy link
Collaborator

@noa-starkware noa-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewable status: 0 of 2 files reviewed, all discussions resolved (waiting on @arad-starkware)

@arad-starkware arad-starkware force-pushed the arad/test_add_get_stakers_exit_intent_flow branch from 2e69d7b to 2be6944 Compare November 17, 2025 13:10
@arad-starkware arad-starkware force-pushed the arad/test_add_get_stakers_with_zero_balance_flow branch from 28e5e23 to 8622540 Compare November 17, 2025 13:10
@arad-starkware arad-starkware force-pushed the arad/test_add_get_stakers_exit_intent_flow branch from 2be6944 to cbfd344 Compare November 17, 2025 13:16
@arad-starkware arad-starkware force-pushed the arad/test_add_get_stakers_with_zero_balance_flow branch from 8622540 to bf7e058 Compare November 17, 2025 13:16
@arad-starkware arad-starkware force-pushed the arad/test_add_get_stakers_with_zero_balance_flow branch from bf7e058 to 82c2b26 Compare November 19, 2025 11:48
@arad-starkware arad-starkware force-pushed the arad/test_add_get_stakers_exit_intent_flow branch from cbfd344 to 5ab5855 Compare November 19, 2025 11:48
@arad-starkware arad-starkware changed the base branch from arad/test_add_get_stakers_with_zero_balance_flow to graphite-base/27 November 19, 2025 12:34
@arad-starkware arad-starkware force-pushed the arad/test_add_get_stakers_exit_intent_flow branch from 5ab5855 to 1bbfecb Compare November 19, 2025 12:34
@arad-starkware arad-starkware changed the base branch from graphite-base/27 to main November 19, 2025 12:34
@arad-starkware arad-starkware force-pushed the arad/test_add_get_stakers_exit_intent_flow branch from 1bbfecb to 1ff8fc2 Compare November 19, 2025 12:35
@codecov
Copy link

codecov bot commented Nov 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.40%. Comparing base (0d4c1ff) to head (1ff8fc2).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #27      +/-   ##
==========================================
- Coverage   95.74%   95.40%   -0.34%     
==========================================
  Files          47       45       -2     
  Lines       11159     9389    -1770     
==========================================
- Hits        10684     8958    -1726     
+ Misses        475      431      -44     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@arad-starkware arad-starkware merged commit 6f0fd6e into main Nov 19, 2025
5 of 7 checks passed
Copy link
Collaborator Author

Merge activity

@arad-starkware arad-starkware deleted the arad/test_add_get_stakers_exit_intent_flow branch November 19, 2025 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants