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

Summary is now being printed on run complete #29

Closed

Conversation

Asaf-S
Copy link

@Asaf-S Asaf-S commented Nov 11, 2021

No description provided.

@rbelev
Copy link

rbelev commented Jan 13, 2022

Would love to be able to use this. Any issue in this moving forward? 😄

@Asaf-S
Copy link
Author

Asaf-S commented Jan 14, 2022

I think the owner of this plugin is not maintaining this plugin anymore. Anyway, you can use my fork (which is waiting to be merged) by installing it directly from GitHub, as it was not registered independently to npm. I use it like that and it's really improved my testing experience...

Tell me if you'll need help setting it up.

@rickhanlonii
Copy link
Owner

@Asaf-S sorry for the delay. Printing the summary by default would defeat my purpose for this reporter. Can you add an option for this? Same as showWarnings but as showSummary?

@mrazauskas
Copy link

Since Jest v28.0.0-alpha.9 one can add summary to Silence Reporter simply like this:

reporters: ['jest-silent-reporter', 'summary']

The default reporter will be disabled, only summary will print. Thanks for idea @Asaf-S!

@rickhanlonii
Copy link
Owner

@mrazauskas that's really cool, thanks for doing that!

Closing this as that's now the preferred way to opt-into a summary at the end.

@Asaf-S
Copy link
Author

Asaf-S commented Mar 15, 2023

@mrazauskas - hey man, thx for the above only now have I noticed it!
It causes another problem now, I've opened an issue about it: here.
Would really appreciate it if you'll have a look.
Thx again.

@mrazauskas
Copy link

Sure. Saw your issue and looking at it already.

Interestingly Jest is using silent reporter with dummy and collects coverage in CI. See config: https://github.com/facebook/jest/blob/main/jest.config.ci.mjs

I will check if that really works and what is going on with your case.

@Asaf-S
Copy link
Author

Asaf-S commented Mar 15, 2023

I think this is the quickest response for an issue ever. Can you be more awesome?? 😆

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.

4 participants