Skip to content
This repository has been archived by the owner on Jul 8, 2019. It is now read-only.

Improve coverable-lines-of-code tracking #116

Open
Pablissimo opened this issue Feb 15, 2017 · 1 comment
Open

Improve coverable-lines-of-code tracking #116

Pablissimo opened this issue Feb 15, 2017 · 1 comment

Comments

@Pablissimo
Copy link
Owner

Pablissimo commented Feb 15, 2017

Need to filter LCOV output to ignore non-executable lines, since source mapping will frequently claim a line that is just a definition in TypeScript is actually executed (due to the fact that it really is executed in JS land).

@Pablissimo
Copy link
Owner Author

Current thinking goes along two lines:

  1. Implement a minimal TypeScript parser to track executable lines in the TypeScript sense
  2. Implement a custom tool in TypeScript using the TypeScript language service and walk the parse tree

First one's fragile and irritating, and likely to not catch all cases (but can be done cheaply for a 90%-right-probably solution). Second one's now a new dependency for the plugin, delivered... somehow, requires Node and the tool on the box running the scanner...

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

No branches or pull requests

1 participant