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

fix: improved test by adding the image to the expected result #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: improved test by adding the image to the expected result #33

wants to merge 1 commit into from

Conversation

miumum
Copy link

@miumum miumum commented Apr 6, 2020

I was investigating how the reporter works with the screen shots and did not see it in documentation. So I looked at the unit test and I was hoping I will find the example. I was also not successful.
I found the screens hots are handled in index.js So I changed unit test to see the actual result.
When I had the unit test updated decided to create pull request and contribute humbly in your reporter.

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.

2 participants