ci: take integration tests into account for front coverage #5976
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
At first I thought it would be fairly easy to have consistent code coverage for e2e tests and unit tests, but it's not:
Base problem
The same coverage system has to be used for both e2e tests and unit tests, so codecov can properly merge reports
Use istanbul everywhere
istanbul
coverage provider vitest-dev/vitest#1676 (comment)Pros:
Cons:
Use v8 everywhere
This would only work on chrome, but is easier to get working, as it does not require build instrumentation
Pros:
Cons: