ryanluker / vscode-coverage-gutters

Display test coverage generated by lcov and xml - works with many languages
https://marketplace.visualstudio.com/items?itemName=ryanluker.vscode-coverage-gutters
MIT License
460 stars 88 forks source link

coverage gutters do not show up sometimes #337

Closed stealthrabbi closed 2 years ago

stealthrabbi commented 3 years ago

Describe the bug I'm running pytests. I was previously getting gutter indicators to show up, but they no longer are. I have to do line coverage to see anything. I saw on the readme that there's a conflict with breakpoints. But, I've removed all breakpoints in my workspace and regenrated coverage, and I still don't see itin the gutter.

Also, the pytest I'm using creates a coverage.xml (default). The extension has a default of cov.xml, yet it still finds coverage.xml

Desktop (please complete the following information):

ryanluker commented 3 years ago

@stealthrabbi Thanks for the ticket! If you could try out the example python project and let me know if that works any better for you, that would be a great first step! https://github.com/ryanluker/vscode-coverage-gutters/tree/master/example/python

Regarding the extension finding coverage.xml, we have had people mention that a lot, so it was added as a default alongside the other popular coverage file names. The old way to specify the file names (for lcov and xml) is being deprecated eventually. https://github.com/ryanluker/vscode-coverage-gutters/blob/master/package.json#L136-L144 https://github.com/ryanluker/vscode-coverage-gutters/blob/master/src/extension/config.ts#L48

Regarding the Windows 10 -> Ubuntu 20 docker container setup, you may need to look into using the remotePathResolve if the coverage file paths that you get in the coverage.xml match correctly 🤔. https://github.com/ryanluker/vscode-coverage-gutters/blob/master/example/example.code-workspace#L10 https://github.com/ryanluker/vscode-coverage-gutters/blob/master/example/remote-node/lcov.info

ryanluker commented 2 years ago

Closing this due to lack of activity but can easily reopen!