Skip to content

Cobertura parser: Extract source path OS-agnostically

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

Superceded by !102295 (merged)

What does this MR do and why?

Resolves #336403 (closed).

Due to a merge conflict with !79866 (merged), I opted to close the previous MR !77024 (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.

Thanks to @fh1ch for advice in the previous MR!

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