Skip to content
This repository has been archived by the owner on Nov 15, 2021. It is now read-only.

PathMap support #1048

Open
manuc66 opened this issue Jul 22, 2021 · 5 comments
Open

PathMap support #1048

manuc66 opened this issue Jul 22, 2021 · 5 comments

Comments

@manuc66
Copy link

manuc66 commented Jul 22, 2021

Is your feature request related to a problem? Please describe.
On CI server, we use msbuild switch pathmap to achieve deterministic builds (with deterministic ). But by doing that, we aren't able to collect coverage on those assemblies.

Describe the solution you'd like
Could it be possible to add a command parameter to add a pathmap argument to do the same kind of mapping and allow collecting coverage

@sawilde
Copy link
Member

sawilde commented Jul 26, 2021

Are you storing your PDBs in a seperate location to the assemplies? if perhaps the -searchdirs argument is what you are looking for

@manuc66
Copy link
Author

manuc66 commented Jul 30, 2021

In fact coverage is reported, but the paths are those mapped with pathmap so when producing reports with a tool such as https://github.com/danielpalme/ReportGenerator "no coverage is found" because there is no matching file at the location

A new parameter would allow to map the other way round so that whatever tooling is being used in the chain it will be able to find the source file and show proper coverage for files in the solution

@sawilde
Copy link
Member

sawilde commented Aug 1, 2021

So a workaround be for the user to reverse the substitutions manually in the XML file before submitting to ReportGenerator?

@manuc66
Copy link
Author

manuc66 commented Aug 2, 2021

Yes, I think it should work

@sawilde
Copy link
Member

sawilde commented Aug 2, 2021

I would say for now then do that via script (I don't know what your platform is) rather than wait for OpenCover to be updated -my focus is elsewhere nowadays.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants