Allow pending reason - Githubissues

webdriverio-boneyard / wdio-jasmine-framework

A WebdriverIO v4 plugin. Adapter for Jasmine testing framework.

http://webdriver.io/

MIT License

23 stars 23 forks source link

Closed patthiel closed 7 years ago

patthiel commented 7 years ago

This PR fixes a bug where adding a pending reason when calling pending('reason here') from within an it() caused the test to fail.
Added a unit test to cover this scenario

If the pending reason isn't available to reporters, we should probably figure out a way to make it available. This is just a quick fix.

cc. @christian-bromann

patthiel commented 7 years ago

The unit test failing above doesn't seem related to my PR.

Also - All tests passed locally.

christian-bromann commented 7 years ago

@patthiel all good, was just a flake, thanks for the PR!