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

Html report is green despite scenario has failed #835

Closed
balpet opened this issue Apr 14, 2015 · 8 comments
Closed

Html report is green despite scenario has failed #835

balpet opened this issue Apr 14, 2015 · 8 comments
Labels
🐛 bug Defect / Bug good first issue Good for newcomers 🚦 needs tests Needs tests

Comments

@balpet
Copy link

balpet commented Apr 14, 2015

Hi,

I'm investigating an error that might be a cucumber bug as well. Could you please also check?
So our framework checks before starting executing a scenario's steps if the server that the tests should be running against is up and available. If it's not then the scenario's steps does not even get executed and our framework raises different kind of exceptions.
And the html report of it says correctly that the scneario has failed but still it's color will be green instead of red.

Have you ever experienced it, do you know what might be the root cause?

Thanks,
Peter

2015-04-14_1815
2015-04-14_1817

@mattwynne
Copy link
Member

This could easily be a bug in the HTML formatter, yes. What we need is a new spec for the case where the before hook fails.

@mattwynne mattwynne added 🐛 bug Defect / Bug 🚦 needs tests Needs tests good first issue Good for newcomers labels Apr 14, 2015
@brasmusson
Copy link
Contributor

@balpet Are you using Cucumber v2.0 or Cucumber v1.3.x?

@balpet
Copy link
Author

balpet commented Apr 15, 2015

@brasmusson I was using 1.3.17

@brasmusson
Copy link
Contributor

It is likely that this is a bug in the HTML formatter in Cucumber v1.3.x. This problem is maybe fixed in Cucumber v2.0, but to be sure I need to test it there.

@balpet
Copy link
Author

balpet commented Apr 15, 2015

Thanks @brasmusson and @mattwynne for looking into this

@mattwynne
Copy link
Member

@balpet are you able to upgrade to 2.0 and try with that?

@mattwynne
Copy link
Member

Thanks Bjorn.

@lock
Copy link

lock bot commented Oct 25, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Oct 25, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐛 bug Defect / Bug good first issue Good for newcomers 🚦 needs tests Needs tests
Projects
None yet
Development

No branches or pull requests

3 participants