Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Conversation

@uhafner
Copy link
Member

@uhafner uhafner commented Sep 18, 2022

Seems that our parser already handles that file correctly.

See https://github.com/jenkinsci/code-coverage-api-plugin/issues/473

@uhafner uhafner added the tests Enhancement of tests label Sep 18, 2022
@codecov
Copy link

codecov bot commented Sep 18, 2022

Codecov Report

Merging #474 (5418870) into master (d3e9c96) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #474      +/-   ##
============================================
- Coverage     72.46%   72.38%   -0.09%     
  Complexity     1005     1005              
============================================
  Files            87       87              
  Lines          3741     3741              
  Branches        435      435              
============================================
- Hits           2711     2708       -3     
- Misses          884      885       +1     
- Partials        146      148       +2     
Impacted Files Coverage Δ
...va/io/jenkins/plugins/coverage/CoverageAction.java 53.33% <0.00%> (-15.56%) ⬇️
...ns/coverage/adapter/parser/JavaCoverageParser.java 82.05% <0.00%> (+1.28%) ⬆️
...enkins/plugins/coverage/adapter/util/XMLUtils.java 58.57% <0.00%> (+4.28%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@uhafner uhafner linked an issue Sep 19, 2022 that may be closed by this pull request
@uhafner uhafner merged commit fd2bd6a into master Sep 19, 2022
@uhafner uhafner deleted the cobertura-npe branch September 19, 2022 06:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

tests Enhancement of tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NullPointerException on parsing Cobertura XML

2 participants