-
Notifications
You must be signed in to change notification settings - Fork 285
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: New State transition tests #806
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #806 +/- ##
==========================================
+ Coverage 98.05% 98.11% +0.06%
==========================================
Files 117 119 +2
Lines 11487 11752 +265
==========================================
+ Hits 11264 11531 +267
+ Misses 223 221 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
|
e45666b
to
fd1e894
Compare
45a6b4a
to
bb0984a
Compare
bb0984a
to
0742bd2
Compare
0742bd2
to
31f2a01
Compare
31f2a01
to
f3f09d9
Compare
2680d79
to
bae296a
Compare
c19a78e
to
11cfcf6
Compare
Needs rebase. |
217402b
to
7583240
Compare
Done. Also added some tests for extcode. |
New tests for: - contract creation, - account touches, - empty coinbase, - selfdestruct, - extcode.
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.
LGTM. The conflict on the extcode test file should be resolvable by just combining the 2 files, the tests in master
are brand new tests for EOF-extcode behavior
7583240
to
b951b55
Compare
New tests for:
These tests come from the work on #802 so may not be very coherent.