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

Discuss if we need the script for vitest run --coverage on frontend and implement solution #1027

Open
3 tasks
NabbeunNabi opened this issue Feb 11, 2025 · 0 comments
Labels
code quality question Further information is requested research possibility and viability of features

Comments

@NabbeunNabi
Copy link
Contributor

Problem

We currently have a script that checks the coverage of our tests for vitest on the frontend that runs with the script yarn test:pinia:coverage. We are not sure how helpful this is as we use Cypress within our Vue files for E2E testing. Is it worth keeping? Is it worth updating configs for this or the script to ignore any files ending with .vue so we can still see the test coverage with pinia and utility files.

Success

The necessity of this script is researched and the solution is implemented.

Requirements

  • Research was vitest run --coverage does in detail and how accurate or helpful it actually is.
  • Provide a solution as to whether it is worth updating the script to ignore .vue files or to remove the script completely
  • Implement the solution to either remove the script or ignore the .vue files, with quality documentation and support for the decision.

Implementation notes

Here are the Vitest Docs for coverage

@NabbeunNabi NabbeunNabi added code quality question Further information is requested research possibility and viability of features labels Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code quality question Further information is requested research possibility and viability of features
Projects
None yet
Development

No branches or pull requests

1 participant