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

added reportPassedTest, takeScreenshotsOnExpectFailures and additional prefix options #53

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

ryan-ohern
Copy link

@ryan-ohern ryan-ohern commented May 9, 2017

reportPassedTest:
With my automated tests running frequently, I was looking for a way to only generate the html report if a spec/suite had a failure. Hopefully this option will be of use to someone else as well.

takeScreenshotsOnExpectFailures:
Allows for grabbing a screenshot as soon as an expect statement fails vs just at end of spec. Included in this was modifying the html report to write an array of screenshots per spec vs just one.

specDoneScreenshotPrefix & expectFailedScreenshotPrefix:
Allows for customizing a prefix to the filenames for these screenshots. I found this helpful when viewing multiple screenshots within one spec to quickly identify which one was the spec done screenshots vs expect failure screenshots.

@ryan-ohern ryan-ohern changed the title added reportPassedTest option added reportPassedTest, takeScreenshotsOnExpectFailures and additional prefix options May 11, 2017
@davidzaque
Copy link

@ryan-ohern it will be merged? i need that!!

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