Skip to content

Cobertura parser: Extract source path OS-agnostically

Katrin Leinweber requested to merge 336403c-os-independent-cobertura-path-split into master

What does this MR do and why?

Resolves #336403 (closed).

By replacing any Windows-backslashes first, we ensure that:

  1. the subsequent Unix-style path split works, and thus
  2. test coverage visualization works even when the report was generated on a Windows-based GitLab Runner.

Due to conflicts in !77024 (closed) & !82494 (closed), I opted to close the previous MRs.

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

I applied these changes with bundle exec guard active.

# Before:
… 
85 examples, 0 failures

# After
… 
86 examples, 0 failures

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Katrin Leinweber

Merge request reports