Closed obuchtala closed 5 years ago
Merging #1300 into master will increase coverage by
0.55%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #1300 +/- ##
==========================================
+ Coverage 89.03% 89.59% +0.55%
==========================================
Files 400 400
Lines 9023 9023
==========================================
+ Hits 8034 8084 +50
+ Misses 989 939 -50
Impacted Files | Coverage Δ | |
---|---|---|
src/article/shared/TableComponent.js | 91.82% <0%> (+0.37%) |
:arrow_up: |
src/dar/PersistedDocumentArchive.js | 78.12% <0%> (+2.5%) |
:arrow_up: |
src/kit/ui/ScrollPane.js | 54.79% <0%> (+2.73%) |
:arrow_up: |
src/kit/app/SurfaceManager.js | 91.83% <0%> (+4.08%) |
:arrow_up: |
src/article/ArticleSerializer.js | 100% <0%> (+11.11%) |
:arrow_up: |
src/article/shared/DOIInputComponent.js | 90.24% <0%> (+24.39%) |
:arrow_up: |
src/kit/ui/Overlay.js | 100% <0%> (+25%) |
:arrow_up: |
src/kit/ui/AbstractScrollPane.js | 70.68% <0%> (+29.31%) |
:arrow_up: |
src/article/shared/GraphicComponent.js | 100% <0%> (+31.25%) |
:arrow_up: |
src/article/shared/InlineGraphicComponent.js | 100% <0%> (+33.33%) |
: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 4fbc34f...e258300. Read the comment docs.
Why
The current setup is not sufficient for several reasons:
nyc
andesm
What
This PR introduces a more sophisticated configuration, using explicit code instrumentation and different test execution environment producing different coverage outputs, which are eventually combined into one report using
nyc report
.