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.

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

proposal: enable code coverage #1462

Open
deer opened this issue Jul 14, 2023 · 0 comments
Open

proposal: enable code coverage #1462

deer opened this issue Jul 14, 2023 · 0 comments

Comments

@deer
Copy link
Contributor

deer commented Jul 14, 2023

I created a branch to enable coverage in my fork and added this commit: deer@c92e910

I sort of did it wrong, because each test is uploading coverage which is a bit wasteful. But the results were interesting nonetheless: https://coveralls.io/github/deer/fresh. 75% isn't exactly great. There's a bit of noise here since it's including the tests folder itself, which is a bit silly. Two key parts stand out:

The biggest thing we could do to increase our coverage is to finally start testing all the CSP stuff, which I highlighted here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant