search

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

Intra-line code coverage indications #90

Closed swingtempo closed 6 years ago

swingtempo commented 7 years ago

Hi,

when I have a ternary operation and tests that only execute one code path, I would like an indication that only certain lines/columns were executed. Instead I see that the whole line is green.

Similarly if I have an if statement and an else. Possible to have an option to show squiggly lines instead of gutters?

(low priority :) )

ryanluker commented 7 years ago

Hey @jayongg thanks for the issue 😄 . currently you can do line coverage instead of gutter coverage through the settings (see my post here for info https://github.com/ryanluker/vscode-coverage-gutters/issues/87#issuecomment-316275141). Could this be related to your other typescript issue? I as able to have partial (orange) indicators for javascript ternaries. save2 for extra info on what the lcov codes mean check out this link: https://github.com/linux-test-project/lcov/blob/master/man/geninfo.1#L481

swingtempo commented 7 years ago

Hey hey! No not related to typescript issue. You are able to see coverage of .ts files?

As for ternary operators out of curiosity are you able to see which particular conditions were covered?

Get Outlook for iOShttps://aka.ms/o0ukef

From: Ryan Luker notifications@github.com Sent: Monday, August 7, 2017 8:23:27 PM To: ryanluker/vscode-coverage-gutters Cc: Jay Ongg; Mention Subject: Re: [ryanluker/vscode-coverage-gutters] Intra-line code coverage indications (#90)

Hey @jayongghttps://github.com/jayongg thanks for the issue 😄 . currently you can do line coverage instead of gutter coverage through the settings (see my post here for info #87 (comment)https://github.com/ryanluker/vscode-coverage-gutters/issues/87#issuecomment-316275141). Could this be related to your other typescript issue? I as able to have partial (orange) indicators for javascript ternaries. [save2]https://user-images.githubusercontent.com/1335972/29054937-429b5dc0-7bae-11e7-9138-3b238d49effe.PNG

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ryanluker/vscode-coverage-gutters/issues/90#issuecomment-320839514, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKhX-QrfxgrrMQuydZSWC9pTmAvlUVCvks5sV9SvgaJpZM4OuosE.

ryanluker commented 7 years ago

ahh I see what you mean now about the coverage. no the coverage does not show for the specific condition that is "uncovered" it simply shows you that something on that line is not. Currently I can not think of a clean way (other then the green squiggles you mentioned)... in other extensions I have seen other people use symbols to try to better show the extent of the coverage but I do not think that would match what the extension currently offers. One option I could investigate would be to draw two colors on a line to try to show the difference in coverage but that would be quite the project 😃 .

swingtempo commented 7 years ago

Hmm is it possible to have vscode highlight part of a line (as opposed to swuiggles)? Actually I think the real question is - is it possible to read intra line info from a lcov file? I presume it is :)

ryanluker commented 6 years ago

Closing this one as it has gotten quite stale, could potentially work on this again after the caching enhancements #106 but without more research into drawing different colours on lines I am not sure how much of a project this one is.