Skip to content

Support for Cobertura XML report format#812

Merged
sebastianbergmann merged 1 commit intosebastianbergmann:masterfrom
smmccabe:cobertura
Sep 26, 2020
Merged

Support for Cobertura XML report format#812
sebastianbergmann merged 1 commit intosebastianbergmann:masterfrom
smmccabe:cobertura

Conversation

@smmccabe
Copy link
Copy Markdown
Contributor

Updated version of #734

I combined some work I had done originally with Rask's work. The result should be fully compliant with the cobertura format dtd.

The one problem I have not resolved is one that Rask noted that cobertura doesn't seem to support non-class based lines. I am not sure if this should even be a blocking issue, but I will check what some other test runners like Jest have done.

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 22, 2020

Codecov Report

Merging #812 into master will increase coverage by 0.42%.
The diff coverage is 95.79%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #812      +/-   ##
============================================
+ Coverage     83.09%   83.51%   +0.42%     
- Complexity     1072     1093      +21     
============================================
  Files            61       62       +1     
  Lines          3448     3567     +119     
============================================
+ Hits           2865     2979     +114     
- Misses          583      588       +5     
Impacted Files Coverage Δ Complexity Δ
src/Report/Cobertura.php 95.79% <95.79%> (ø) 21.00 <21.00> (?)

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 c9394cb...28f0b51. Read the comment docs.

@sebastianbergmann sebastianbergmann merged commit 0a5b281 into sebastianbergmann:master Sep 26, 2020
@sebastianbergmann sebastianbergmann changed the title Add Cobertura report format Support for Cobertura XML report format Sep 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants