Closed fpavageau closed 3 years ago
Merging #2604 (fb40156) into master (7231b7d) will decrease coverage by
7.22%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #2604 +/- ##
============================================
- Coverage 81.29% 74.06% -7.23%
+ Complexity 2396 2168 -228
============================================
Files 267 267
Lines 7798 7800 +2
Branches 808 808
============================================
- Hits 6339 5777 -562
- Misses 1113 1644 +531
- Partials 346 379 +33
Flag | Coverage Δ | Complexity Δ | |
---|---|---|---|
integration | ? |
? |
|
unittests | 74.06% <100.00%> (-0.02%) |
0.00 <1.00> (ø) |
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 7231b7d...278d09c. Read the comment docs.
@fpavageau BTW, no need to squash or force-push in branches/PRs because we always squash commits on merge to master. Do you want to add your name to DatastoreTemplate.java
? If not, I'm happy to merge.
I added my name to the test class, I guess that's enough.
The new
DatastoreTemplate
instance should have the sameApplicationEventPublisher
as the original one, so thatAuditingHandler
can be called.Fixes #2603