Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] validator and delegator stress unit tests #7574

Closed
wants to merge 1 commit into from

Conversation

emmazzz
Copy link
Contributor

@emmazzz emmazzz commented Jan 23, 2023

No description provided.

@vercel
Copy link

vercel bot commented Jan 23, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

4 Ignored Deployments
Name Status Preview Comments Updated
explorer ⬜️ Ignored (Inspect) Jan 25, 2023 at 7:10PM (UTC)
explorer-storybook ⬜️ Ignored (Inspect) Jan 25, 2023 at 7:10PM (UTC)
frenemies ⬜️ Ignored (Inspect) Jan 25, 2023 at 7:10PM (UTC)
wallet-adapter ⬜️ Ignored (Inspect) Jan 25, 2023 at 7:10PM (UTC)

@@ -352,6 +352,9 @@ module sui::staking_pool {
if (total_sui_withdraw_amount >= principal_withdraw_amount)
total_sui_withdraw_amount - principal_withdraw_amount
else 0;
// if (balance::value(&pool.rewards_pool) < reward_withdraw_amount) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Uncomment these lines and the delegation stress test will pass

@emmazzz emmazzz force-pushed the delegation_test branch 4 times, most recently from 2efa85e to 842b600 Compare January 23, 2023 22:02
@emmazzz emmazzz changed the title [WIP] delegation stress unit tests [WIP] validator and delegator stress unit tests Jan 23, 2023
add delegation switch to one of the operations

add validator operations
@emmazzz emmazzz requested a review from tzakian March 3, 2023 23:38
tzakian added a commit that referenced this pull request Mar 7, 2023
This is the initial port of the delegation stress test that Emma wrote
in #7574 to the new delegation flow. We'll need to (and want to) add
additional actions (e.g., candidacy, removal etc).
tzakian added a commit that referenced this pull request Mar 8, 2023
This is the initial port of the delegation stress test that Emma wrote
in #7574 to the new delegation flow. We'll need to (and want to) add
additional actions (e.g., candidacy, removal etc).
tzakian added a commit that referenced this pull request Mar 8, 2023
This is the initial port of the delegation stress test that Emma wrote
in #7574 to the new delegation flow. We'll need to (and want to) add
additional actions (e.g., candidacy, removal etc).
@stefan-mysten
Copy link
Contributor

This will not be merged so closing.

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