-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Add test checklist for eip7732 #4759
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
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please run make lint to format the file.
|
|
||
| #### Expectation | ||
|
|
||
| Bid is accepted, latest_execution_payload_bid is updated in state, and no pending payment is created since value is zero. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are many instances of inline code which are not shown as code. Eg latest_execution_payload_bid.
|
|
||
| #### Expectation | ||
|
|
||
| Bid is accepted, latest_execution_payload_bid is updated, builder balance remains unchanged, and a pending payment is created in the builder_pending_payments queue with weight = 0 and withdrawable_epoch = FAR_FUTURE_EPOCH. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would prefer we don't do the "variable = value" thing & instead write it as a proper sentence. Mostly for consistency; see the scenario section immediately above which does this.
|
This PR is still in draft |
No description provided.