Skip to content

[Testing]: increase coverage to 80% #5121

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.

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

Closed
jsjoeio opened this issue Apr 21, 2022 · 1 comment
Closed

[Testing]: increase coverage to 80% #5121

jsjoeio opened this issue Apr 21, 2022 · 1 comment
Assignees
Labels
testing Anything related to testing

Comments

@jsjoeio
Copy link
Contributor

jsjoeio commented Apr 21, 2022

Problem

We are unhappy with our current coverage (71% at the time of writing).

Solution

Write tests for code in the following files until we hit 80% coverage:

  • entry.ts
  • main.ts
  • heart.ts
  • http.ts
  • util.ts

We don't know how many new tests but we do know that the following files previously listed are lacking in file coverage or close to hitting 100% based on Codecov stats.

The graph below shows files and lines missing code coverage based on running yarn test:unit in code-server:
image

@jsjoeio jsjoeio added the testing Anything related to testing label Apr 21, 2022
@jsjoeio jsjoeio added this to the May 2022 milestone Apr 21, 2022
@jsjoeio jsjoeio added the Epic label May 2, 2022
@jsjoeio
Copy link
Contributor Author

jsjoeio commented May 2, 2022

Closing in favor of #5149

@jsjoeio jsjoeio closed this as completed May 2, 2022
@jsjoeio jsjoeio removed this from the May 2022 milestone May 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing Anything related to testing
Projects
None yet
Development

No branches or pull requests

2 participants