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

Trigger deferred test on retrieving result #51

Merged

Conversation

martijnspaan
Copy link
Contributor

In our case we chill our selenium UI tests. As a feature a screenshot is created when the test fails. Currently this is only possible by including the given and when inside the actual test method (then) because when an exception occurs in the constructor it is not possible to catch it for building the screenshot.

Solution would be use a deffered test, but that is currently only possible when expecting an exception. This is now supported through the Result property which will execute the deferred test first.

@martijnspaan
Copy link
Contributor Author

@Erwinvandervalk, can you please review this change?

@martijnspaan martijnspaan changed the base branch from develop to master December 20, 2016 10:31
In our case we chill our selenium UI tests. As a feature a screenshot is created when the test fails. Currently this is only possible by including the given and when inside the actual test method (then) because when an exception occurs in the constructor it is not possible to catch it for building the screenshot.

Solution would be use a deffered test, but that is currently only possible when expecting an exception. This is now supported through the Result property which will execute the deferred test first.
@Erwinvandervalk Erwinvandervalk merged commit 265b902 into ChillBDD:master Dec 20, 2016
@martijnspaan martijnspaan deleted the Trigger_test_on_result branch December 20, 2016 12:31
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.

None yet

2 participants