Add test.coveragePathIgnorePatterns
option
#12238
Open
+242
−26
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.
What does this PR do?
bun:test
option similar to Jest'scoveragePathIgnorePatterns
#4021.Adds a
coveragePathIgnorePatterns
option tobunfig.toml
similar to Jest's option of the same name.Per #4021 (comment), the value can be a glob pattern or an array of glob patterns. File paths matching any of the globs will be ignored for the purpose of coverage % calculation and reporting.
I do have a couple of questions/concerns:
If all tested file paths are ignored (e.g.,
"**"
) and the coverage reporter is "lcov", thelcov.info
file is created but left empty. Seems maybe technically correct, but LCOV processing tools might error (genhtml
definitely errors; I'm not sure about others).The glob matcher seems strict to me, at least in this context. For example,
"*.js"
won't match anything but"**/*.js"
will. Probably not a big deal, but we might consider relaxing it a little. I don't have an opinion either way.Documentation or TypeScript types
Code changes
How did you verify your code works?
I wrote automated tests.