Closed louismillsdiffblue closed 5 years ago
Merging #16 into master will increase coverage by
0.18%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #16 +/- ##
============================================
+ Coverage 47% 47.19% +0.18%
- Complexity 637 639 +2
============================================
Files 56 56
Lines 4310 4310
Branches 790 790
============================================
+ Hits 2026 2034 +8
+ Misses 1854 1851 -3
+ Partials 430 425 -5
Impacted Files | Coverage Δ | Complexity Δ | |
---|---|---|---|
...va/com/alipay/test/acts/object/ActsObjectUtil.java | 42.1% <0%> (-3.95%) |
10% <0%> (-2%) |
|
.../main/java/com/alipay/test/acts/util/DateUtil.java | 41.45% <0%> (-0.2%) |
77% <0%> (ø) |
|
...in/java/com/alipay/test/acts/util/CSVApisUtil.java | 43.09% <0%> (+2.54%) |
46% <0%> (+4%) |
:arrow_up: |
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 241cd70...b6c27f5. Read the comment docs.
Hi,
Further to my initial PR for the acts module, I have written some tests that cover com.alipay.test.acts.util.CSVApisUtil with the help of Diffblue Cover.
Again, these tests should help detect any regressions caused by future code changes. If there are any other classes you would consider important to have tests for I would be happy to take a look.