-
Notifications
You must be signed in to change notification settings - Fork 89
chore(JSON-RPC,execution): consolidate repeating test code #1102
Conversation
6980f38
to
3ca5a58
Compare
Codecov Report
@@ Coverage Diff @@
## main #1102 +/- ##
=======================================
Coverage 74.19% 74.19%
=======================================
Files 73 73
Lines 6642 6642
Branches 6642 6642
=======================================
Hits 4928 4928
Misses 873 873
Partials 841 841
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @yair-starkware)
3ca5a58
to
f26af4b
Compare
I'm not sure why reviewable is showing this like a new code - I just added Eq + PartialEq to the derive |
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.
Reviewed 3 of 3 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @yair-starkware)
Pull Request type
Please check the type of change your PR introduces:
What is the current behavior?
Issue Number: N/A
What is the new behavior?
Does this introduce a breaking change?
Other information
This change is