-
Notifications
You must be signed in to change notification settings - Fork 245
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
tests: Coverage improvements #474
Conversation
@@ -326,37 +336,23 @@ contract('EVM Script', accounts => { | |||
}) | |||
}) | |||
|
|||
context('registry', () => { | |||
it('can be disabled', async () => { | |||
context('> Registry actions', () => { |
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.
Note that these tests have been simplified as most of the functionality was already tested earlier.
@@ -1,31 +1,49 @@ | |||
pragma solidity 0.4.24; |
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.
This mock has been refactored to be simpler, and more targeted about what's missing.
2042452
to
76bac53
Compare
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
* APMRegistry: fix tests checking APMRegistry doesn't init without required permissions * EVMScriptRegistry: add more tests for coverage * ENSSubdomainRegistrary: fix initialization test
Improves the coverage of some missing branches.