-
Notifications
You must be signed in to change notification settings - Fork 91
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
Missing report configuration defaultSummarizer #547
Comments
If you feel like changing for another v8-driven code coverage tool, you can give One Double Zero a try. I just tested and it passes through any reporter options that it receives: {
"sources": [
"./src/main/{.,!(target)/**}/*.ts"
],
"reporters": [
"html",
"text"
],
"reporterOptions": {
"html": {
"defaultSummarizer": "nested"
}
}
} It is worth noting that the Note that I didn't know about that option and I have to thank you for that: the |
@ericmorand I haven't been putting work into open source, so I deserve this... But, using my repos to advertise your own open source projects is still a little gauche 😛
@miyasudokoro send me a patch and I will happily expose this 👏 |
@bcoe I've never done any testing with "snapshots" before, so I don't know how to finish. (See my commit above.)
|
@miyasudokoro I created a draft PR here you could use for a starting point, if your'e so inclined: |
There is a poorly-documented feature of the HTML Istanbul reporters (html, lcov, html-spa) called "defaultSummarizer". It would be nice if c8 could pass through this configuration option so it can be used.
I am willing to create a pull request for this issue myself.
Below is my documentation of what "defaultSummarizer" does from my pull request to @web/test-runner.
For some reporters, you can change the summary and hierarchy of reported files using the option
defaultSummarizer
.pkg
nested
:flat
:Example HTML report structures:
Actual file structure:
pkg:
nested:
flat:
The text was updated successfully, but these errors were encountered: