Closed learn-more closed 5 years ago
@learn-more Thanks for submitting such a detailed bug ticket! When I get some more time to work on my OSS I will take a look at this issue.
Quick note: the coverage-gutters.xmlname
configuration is only meant to be a filename not path but I believe this wouldn't effect the extension like you are seeing (having just clover.xml
will still have the extension search for the coverage file within the open workspaces). Although if your build
folder is not in an open workspace you may get issues like you are seeing.
@ryanluker I am not 100% clear on what would be called a workspace
, but I have opened the root folder of the repository using File->Open Folder
.
The file build/logs/clover.xml
does show up in the Explorer
window in VS Code.
@learn-more thanks for the extra info, if it appears in the explorer then yes it will be in "workspace".
@learn-more this appears to be an issue with the upstream package clover-json
, I am going to open a ticket there to see if a fix can be applied but will move this out of the 2.3.1 release in the meantime.
It seemed to be caused by not having package
defined in the xml output as an immediate child of project https://github.com/coverage-report/clover-json/blob/master/src/index.js#L11.
@learn-more this is fixed in the upstream clover-json 0.3.0
release. I have pulled this version into master 1d96e001872e39652b3ca1ac570a07419ae35337 and tested that it fixes your issue with the provided example project, thanks again for that! This should get released to the marketplace in the next couple days.
Awesome, thanks!
Describe the bug When trying to run this on my repo, the following error is shown:
To Reproduce Attached is the coverage file, and a snapshot of the repo. (It is expected to move owner soon, hence the snapshot as attachment) The coverage file is inside the zip, at
build/logs/clover.xml
. mattermost.zipI have seen the related issues with the same title, but from what I can see, there are no empty classes in my project.
Desktop (please complete the following information):
The only configuration set for this extension is: