Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add code coverage task to scripts to measure test coverage #92

Open
smorrisj opened this issue Jan 18, 2023 · 1 comment · May be fixed by #93
Open

Add code coverage task to scripts to measure test coverage #92

smorrisj opened this issue Jan 18, 2023 · 1 comment · May be fixed by #93
Assignees
@clangsmith @2TomLi
Labels
code quality Code quality related issues

Comments

@smorrisj
Copy link
Contributor

Is your feature request related to a problem? Please describe.
We need the ability to measure the amount of code coverage that our automated tests achieve during runs.

Describe the solution you'd like
-Identify a coverage framework to use (c8, jest, istanbul, etc)
-A new task added to scripts called coverage that will generate line coverage at a minimum for our ts files.
-Task is integrated into the PR check action to provide additional insight at review time.

Additional context
This seems very doable based on this post.
@smorrisj
Copy link
Contributor Author

I think c8 seems attractive here. It does what we want and has the ability to enable minimum threshholds down the road at PR time.

@smorrisj smorrisj linked a pull request Jan 18, 2023 that will close this issue
feat: add code coverage #93
Open
@smorrisj smorrisj added the code quality Code quality related issues label Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@clangsmith clangsmith

@2TomLi 2TomLi

Labels
code quality Code quality related issues
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add code coverage sassoftware/vscode-sas-extension
3 participants
@clangsmith @2TomLi @smorrisj