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

--reporter generates report after first worker finishes batch of tests #45

Closed
StephanBijzitter opened this issue Aug 14, 2017 · 6 comments

Comments

@StephanBijzitter
Copy link

StephanBijzitter commented Aug 14, 2017

With the following in the nightwatch config:

  "test_workers": {
    "enabled": true,
    "workers": "auto"
  }

The html report is created when the first worker finishes, and thus misses a lot of data.
Without workers, the report is created for all tests, but does not include the steps (as they're not in the XML)

@jls
Copy link
Owner

jls commented Aug 17, 2017

Hi Stephan, thanks for reporting this! I may need your help here because we don't use those config options so I don't have a good way to test.

Could you upload a zip containing a few sample tests and the nightwatch config along with info about which version of nightwatch you are using? I can use that to try and see what events are sent to the reporter and what the XML files contain.

Thank you!

@StephanBijzitter
Copy link
Author

StephanBijzitter commented Aug 17, 2017

I'll set up an example repository tomorrow, in the meantime also created a feature request at Nightwatch:
nightwatchjs/nightwatch#1557

Edit: example repo kinda got delayed and forgotten about, but issue should be enough since it's a fault in Nightwatch itself

@jls
Copy link
Owner

jls commented Aug 18, 2017

Ok perfect thank you!

@khenlevy
Copy link

khenlevy commented Sep 5, 2017

+1

@termleech
Copy link

Sadly I hope this isn't a result of this very glaring regression but that nightwatch has yet to fix nightwatchjs/nightwatch#1205

@jls
Copy link
Owner

jls commented Nov 29, 2017

Closing for comments above

@jls jls closed this as completed Nov 29, 2017
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

No branches or pull requests

4 participants