Skip to content
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

refactored AllInvocationsFinder and VerifiableInvocationsFinder to st… #515

Merged
merged 1 commit into from Jul 31, 2016
Merged

refactored AllInvocationsFinder and VerifiableInvocationsFinder to st… #515

merged 1 commit into from Jul 31, 2016

Conversation

PascalSchumacher
Copy link
Contributor

…atic utility classes (#426)

@codecov-io
Copy link

Current coverage is 87.64% (diff: 71.42%)

Merging #515 into master will decrease coverage by 0.04%

@@             master       #515   diff @@
==========================================
  Files           268        268          
  Lines          4818       4816     -2   
  Methods           0          0          
  Messages          0          0          
  Branches        793        793          
==========================================
- Hits           4225       4221     -4   
- Misses          423        425     +2   
  Partials        170        170          

Sunburst

Powered by Codecov. Last update b1a7090...e7d0449

@raphw raphw merged commit 19457cc into mockito:master Jul 31, 2016
@ChristianSchwarz
Copy link
Contributor

@PascalSchumacher
These 2 classes have only one public method, they can be merged into:

  • VerificationFinder::findVerified(..) / ::findAll(..)

What do you think?

@PascalSchumacher
Copy link
Contributor Author

PascalSchumacher commented Aug 1, 2016

@ChristianSchwarz
sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants