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

test: add front-end coverage reporting #2738

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

wescopeland
Copy link
Member

@wescopeland wescopeland commented Oct 2, 2024

Nearly all of this PR's diff is package-lock.json noise.


This PR enables front-end test coverage reporting, both in the terminal and via the Vitest official dashboard:

Screenshot 2024-10-02 at 6 15 53 PM

How to test this PR:

  • sail npm install
  • sail npm run test:coverage

This should cause tests to begin running and a tiny Vitest service to start at http://localhost:51204/__vitest__/.

Manually navigate your browser to http://localhost:51204/__vitest__/.

If npm run test:coverage is run without Sail, the browser window automatically pops. Needing to do manual navigation is a Sail limitation.

After tests are done running, coverage should also appear in the terminal itself. If you navigate to the URL above, you can click a button to view coverage:

Screenshot 2024-10-02 at 6 15 39 PM

@wescopeland wescopeland requested a review from a team October 2, 2024 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant