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

Make reporter compatible with WebdriverIO v5 #12

Closed
christian-bromann opened this issue Jan 8, 2019 · 9 comments · Fixed by #14
Closed

Make reporter compatible with WebdriverIO v5 #12

christian-bromann opened this issue Jan 8, 2019 · 9 comments · Fixed by #14

Comments

@christian-bromann
Copy link
Collaborator

Hola 👋

WebdriverIO released v5 around Christmas, see also this blogpost. To make this reporter usable with the new version there are some small adjustments necessary. There are some minor changes that need to be made. You can find a general overview here. Most important ones to point out are:

  • set "@wdio/cli": "^5.0.0" as peerDependency
  • let the reporter extend from @wdio/reporter
  • change the constructor so it calls super()
    Note: if your reporter is designed to report to stdout have it the following way:
    /**
     * make reporter to write to output stream by default
     */
    options = Object.assign(options, { stdout: true });
    super(options)
    if your reporter should rather report to a file per default just call:
    super(options)
  • all file or stdout messages need to be propagated by this.write("my log message")

Feel free to drop by the webdriverio/v5 support channel in case you need support.

Thanks so much for contributing to the WebdriverIO community. We really appreciate it 🙏 !

@tagoro9
Copy link
Contributor

tagoro9 commented Jan 23, 2019

I would like to help with the update, as I'm using this reporter.

Looking at the current messages that are generated I think there is some information there that is no longer provided in the SuiteStats or TestStats classes.

I can try to create a PR with a possible update.

@fedecia
Copy link

fedecia commented Jan 23, 2019

I have a version working locally, will see if I can provide the PR

@VikingTristan
Copy link

Any updates on this?

@tagoro9
Copy link
Contributor

tagoro9 commented Feb 5, 2019

I didn't work on this as @fedecia mentioned that they already have a working version.

Happy to give it a try if they are unable to create the PR.

@mightyaleksey
Copy link
Member

Sorry, guys. Haven't noticed it. I'll check it this week and see what can i do.

@AnnikaNissen
Copy link
Contributor

Any news on this? I would really appreciate if the reporter would be updated to be compatible with WebdriverIO v5 :)

@simonwilson1985
Copy link

I have a version working locally, will see if I can provide the PR

@fedecia any chance you could provide a PR for this please?

@tagoro9
Copy link
Contributor

tagoro9 commented Mar 11, 2019

I just created a PR updating the reporter to support wdio 5. #14

@MykleNero
Copy link

Hello, any chances of getting this PR merged? #14

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

Successfully merging a pull request may close this issue.

8 participants