-
-
Notifications
You must be signed in to change notification settings - Fork 132
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
test: auto mine functionality to eth-tester #1972
Conversation
assert not eth_tester_provider.auto_mine | ||
|
||
# Ensure can still manually mine. | ||
block = eth_tester_provider.get_block("latest").number |
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.
attempted to repro #1969 but mining still seems to work, at least at the provider level.
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.
Is this a blocker?
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.
not for this feature particularly. if we can get them to upgrade we can see if that helps them else will have to look for more info.
e5c3452
to
e3161de
Compare
assert not eth_tester_provider.auto_mine | ||
|
||
# Ensure can still manually mine. | ||
block = eth_tester_provider.get_block("latest").number |
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.
Is this a blocker?
What I did
fixes: #1968
How I did it
does the same as https://github.com/ApeWorX/ape-foundry/pull/59/files but for ape-test
thought: maybe need to squeeze this into the api class to keep things uniform, but i implemented the same as ape-foundry's just in case.
How to verify it
connect to ape-test provider which uses eth-tester, and then you can do this
by default, it is on.
Checklist