-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add test for find and clear #229
Conversation
} | ||
|
||
@Test | ||
public void fnd_empty_mcGymmyHasCorrectContent() { |
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 think there's a typo here and in the test method names below; should be find_empty....
instead of fnd_...
. Just a minor nit.
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.
ok!
changed typo
Codecov Report
@@ Coverage Diff @@
## master #229 +/- ##
=========================================
Coverage 71.07% 71.07%
Complexity 552 552
=========================================
Files 87 87
Lines 1884 1884
Branches 151 151
=========================================
Hits 1339 1339
Misses 486 486
Partials 59 59 Continue to review full report at Codecov.
|
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
No description provided.