-
Notifications
You must be signed in to change notification settings - Fork 203
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
Generate HTML report? #13
Comments
Yes! We did get a solution for this that we are using internally and will be open sourcing that HTML reporter shortly (new week or so). It was a bummer to me that passing custom metadata is not currently supported by jest but that's something that needs to be take up with jest maintainers and I found an ugly workaround for the time being. |
Awesome! Can't wait :) I was already thinking of doing a global map of test names with the match results in the matcher and reading it in a reporter. I opened an issue on Jest jestjs/jest#4547 |
Thank you! I will keep this open until I publish the reporter. |
…ix to master * commit '98e8b991d5afba865b6ff4dc79ff8134dec9f04f': docs(readme): remove default value for customSnapshotsDir
Hi, I find the idea of an HTML report very interesting. Maybe with tools similar to the ones in bitbucket? Any updates on this? |
@Prior99 @epeli let me give an update on this since it has been awhile: I know I previously said I would open source a simple html reporter but I decided not too until jestjs/jest#4547 is resolved. What we have internally is an ugly hack I don't wish to open source (think https://github.com/Hargne/jest-html-reporter but with a regex to check for a path to a png image in the failure message so we can embed that image into html). If jestjs/jest#4547 can be resolved I can easily create an HTML reporter that could even integrate some cool diffing like what Bitbucket has if wanted. But I have not found the time to work on that jest issue. If you guys have the time and can help out with that it would be great! Otherwise it may be awhile. |
I think we are thinking about different HTML reporters here. I will have a look at the issue and maybe send a pull request if I can find some time on a weekend. |
@Prior99 what are you thinking of then? Do you think it should be its own feature request in another issue? |
I was thinking of instead of or in addition to generating an image with a
pixel-diff, generating an HTML page with a slider for side-by-side comparison
or fading both images.
|
Closing this as I am not sure this is going to be worked on anytime soon and is really not a |
Any plans for this?
I explored if it would have been possible to do via custom jest reporter but it seems that it's not possible to pass custom metadata (name of the diff file) from the
toMatchImageSnapshot
matcher to the reporters :/The text was updated successfully, but these errors were encountered: