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

On test failure we want to keep all other tests running (setup should be done in grunt to stop on failure) #5

Closed
wants to merge 5 commits into from

Conversation

lucboutier
Copy link

I'm just logging the error as warn but keep selenium open so other tests are processed.

seckardt added a commit that referenced this pull request Sep 23, 2014
@seckardt
Copy link
Owner

Rejected this implementation. Used keepAlive flag to implement the desired behavior.

@seckardt seckardt closed this Sep 23, 2014
@lucboutier
Copy link
Author

Hi,
Your implementation is much better, however I think it's helpful to have the stack trace displayed:
grunt.warn(out);
Luc

seckardt added a commit that referenced this pull request Sep 23, 2014
@seckardt
Copy link
Owner

In case you are hunting down connection issues, etc. on your side, just use grunt {yourtarget} --verbose. This should give you all the information you need as the grunt-protractor-webdriver logs out everything in verbose mode. And it's much more reliable than a simple grunt.warn(out) as the out pretty often only contains portions of a multi-line statement.

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

Successfully merging this pull request may close these issues.

2 participants