Skip to content
This repository has been archived by the owner on Dec 18, 2019. It is now read-only.

Pending reason does not display in test results #29

Open
patthiel opened this issue Mar 17, 2017 · 3 comments
Open

Pending reason does not display in test results #29

patthiel opened this issue Mar 17, 2017 · 3 comments

Comments

@patthiel
Copy link
Contributor

patthiel commented Mar 17, 2017

When calling pending('My reason here') in a jasmine spec, the reason is never displayed in WDIO (standalone mode) results. Small change in https://github.com/webdriverio/wdio-jasmine-framework/blob/master/lib/reporter.js is likely all that's needed. I can PR this change next week.

Reproduceable Spec:
https://gist.github.com/patthiel/7f68db05a21fdbd3cd80a2b7b66f8eb1

@christian-bromann
Copy link
Contributor

christian-bromann commented Mar 17, 2017

Well according to the package.json you should use jasmine >=2.5 😉

@patthiel
Copy link
Contributor Author

patthiel commented Mar 17, 2017

Sorry, i meant if you run the example through the version of jasmine (eg. node_modules/.bin/jasmine /path/to/pending.spec.js) that is installed via package.json, it properly displays the reason. However, if you run the same spec through WDIO it does not display. Does that make sense?

@christian-bromann
Copy link
Contributor

Small change in https://github.com/webdriverio/wdio-jasmine-framework/blob/master/lib/reporter.js is likely all that's needed.

I am not sure about that. Other frameworks also don't have messages when a test in pending. So all reporter have to get an update in order to support that. Happy to merge any PRs that direct this "issue".

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants