-
Notifications
You must be signed in to change notification settings - Fork 664
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
Task "gulp test" fails when tests fail #498
Comments
Which options, which os? Can you test with this branch #499 ? |
no options, os x. |
Only change was made, 5 replaced by 500 here to make test fail: |
Cloned your project, npm install, bower install, gulp test, it works :/ Which version of Node? |
v0.12.0. |
Yep, but I don't see it...
|
I updated node to 0.12.2 and still see this bug. I will reinstall system from scratch to be sure all is ok with my setup, but it will be on weekend. |
I have the same problem.
|
I also has this error:
|
I had this error, but updating nodejs 0.12.2 makes it work good on windows 8.1 |
Maybe cutting some corners here, but the done callback is being passed with an argument which indicates failure to gulp. My quick fix was to wrap done in a function that does not pass any argument to the callback (in gulp/unit-tests.js):
Edit: I guess it's the singleRun option that is broken somehow. When singleRun is enabled, karma returns with exit code 1 meaning that gulp will show a stacktrace. |
@joelso solution works |
🎉 Just released v0.12.0 🎉 |
In newly-generated project (v0.11.0) change unit-test to make it fail (any expectation) and after error about failed test you will see also:
The text was updated successfully, but these errors were encountered: