-
Notifications
You must be signed in to change notification settings - Fork 141
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
Issue #218 - Update to Spring-Data 2.1.2.RELEASE and Spring Framework 5.1.2.RELEAS #223
Conversation
… regression on live site.
Merge upstream
This required enabling Mockito mocking of final classes, adapting to a single changed Spring-Data class name, and updating unit tests to compensate.
Codecov Report
@@ Coverage Diff @@
## master #223 +/- ##
===========================================
+ Coverage 65.21% 65.4% +0.19%
- Complexity 598 600 +2
===========================================
Files 77 77
Lines 2038 2038
Branches 374 374
===========================================
+ Hits 1329 1333 +4
+ Misses 554 550 -4
Partials 155 155
Continue to review full report at Codecov.
|
Just a note, you dont need that block of code in PartTreeDynamoDBQueryUnitTest.java It will increase coverage as well. |
@boostchicken I'm sorry I'm not following. |
This PR merges all code & all changes from all parties to give credit to everyone!