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

Add delegate-stack-extend command #4610

Conversation

BowTiedRadone
Copy link
Contributor

This PR adds the delegate-stack-extend command to the stateful property testing environment. It is part of #4548 and targets feat/pox-4-stateful-property-testing (#4550).

Copy link

codecov bot commented Apr 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.19%. Comparing base (b33cdfd) to head (73b23b1).

Additional details and impacted files
@@                           Coverage Diff                            @@
##           feat/pox-4-stateful-property-testing    #4610      +/-   ##
========================================================================
- Coverage                                 83.31%   83.19%   -0.13%     
========================================================================
  Files                                       470      470              
  Lines                                    332151   332151              
  Branches                                    317      317              
========================================================================
- Hits                                     276743   276326     -417     
- Misses                                    55400    55817     +417     
  Partials                                      8        8              

see 38 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b33cdfd...73b23b1. Read the comment docs.

Copy link
Member

@moodmosaic moodmosaic left a comment

Choose a reason for hiding this comment

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

LGTM; it follows precisely the pattern we use for all the existing commands. I'd like @friedger to look into the check method, and then we can merge.

@BowTiedRadone BowTiedRadone requested a review from setzeus April 2, 2024 12:08
@moodmosaic
Copy link
Member

Merging into feat/pox-4-stateful-property-testing.

@moodmosaic moodmosaic merged commit b26ad6a into feat/pox-4-stateful-property-testing Apr 2, 2024
2 checks passed
@moodmosaic moodmosaic deleted the feat/pox-4-delegate-stack-extend branch April 2, 2024 15:32
Copy link
Collaborator

@friedger friedger left a comment

Choose a reason for hiding this comment

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

check function looks good

(this.stacker.unlockHeight - FIRST_BURNCHAIN_BLOCK_HEIGHT) /
REWARD_CYCLE_LENGTH,
);
const lastExtendCycle = firstExtendCycle + this.extendCount - 1;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why do you need this value? You are substracting one and adding one afterwards.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@friedger I am doing this to simulate how the lock period is calculated inside PoX-4 contract

@blockstack-devops
Copy link
Contributor

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@stacks-network stacks-network locked as resolved and limited conversation to collaborators Nov 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants