search

webdriverio-community / wdio-html-reporter

Fork of wdio-html-format-reporter
MIT License
18 stars 27 forks source link

Documentation issues #17

Closed goldengecko closed 4 years ago

goldengecko commented 4 years ago

Firstly, thanks for your work on making this available.

In the documentation it says See the report.test.js for configuration but there doesn't appear to be any file by that name. I believe it should be referring to reporter.spec.js

In the section To take a screenshot after test assert: it indicates you should both use browser.saveScreenshot(filepath); and process.emit('test:screenshot', filepath); but in the default settings where useOnAfterCommandForScreenshot is turned on, it already processes the screenshot and pushes the screenshot event. Having both commands results in the inclusion of two screenshots in the report.

Perhaps it would save others some time if these things were corrected.

rpii commented 4 years ago

I updated the documentation, but I really dont have much time for that. Feel free to make any improvements.