Closed soywiz closed 7 years ago
That's not good in my opinion. The tests should NOT change for the coverage report imo.
Additionally I'm still not sure what the benefit of code coverage is? How does it help with all those different languages?
If I understand this correctly we are only testing weather all java based code is covered, right?
I'm not trying to get coverage of languages themselves. But we should get coverage of jtransc code and maybe java API calls tested? When running integration tests the jtransc code that generates final code is executed and we get coverage of that. Also tests like BigTest are run first in Java and we would get also coverage of which APIs are being executed. But not sure. At any rate, what I'm trying to get is coverage of jtransc code, which is enough for me to get value from this.
:exclamation: No coverage uploaded for pull request base (
master@09050d8
). Click here to learn what that means. The diff coverage is0%
.
@@ Coverage Diff @@
## master #235 +/- ##
=========================================
Coverage ? 15.78%
Complexity ? 304
=========================================
Files ? 159
Lines ? 12274
Branches ? 3121
=========================================
Hits ? 1938
Misses ? 9797
Partials ? 539
Impacted Files | Coverage Δ | Complexity Δ | |
---|---|---|---|
...ransc/plugin/service/ServiceLoaderJTranscPlugin.kt | 0% <0%> (ø) |
0 <0> (?) |
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 09050d8...3fe7b45. Read the comment docs.
Fixes #233