The Mocha vs mocha test failed, here is a more robust handling of the pr...

[rohni]

Tried using grunt-mocha and ran into a problem where it failed because there was no test to see whether Mocha was defined, and so it failed.

This fixes the problem. I tried a try..catch solution initially, but it was pretty ugly, although I don't think this is much better.

The whitespace changes seem to have been made by my editor, but it is a bit cleaner this way.

Thanks, and hope this is helpful.

Rohan

[kmiyashiro]

Oh, I see. Instead of all those if statements, could you just change the original test to Mocha and mocha to window.Mocha and window.mocha instead? Silly mistake.

[rohni]

Okay, I have made the changes and pushed, they should be waiting in the branch.

[kmiyashiro]

Hmm, don't see it in your branch, are you sure you pushed?

[rohni]

Oops, how embarrassing. It is now pushed.

[rohni]

Yeah, that was another issue. The intermediary check is definitely needed, will adjust the code. Actually that was my first attempt to get it working, which failed because of the undefined Mocha. The window. solved this, of course, but not the second.

I am assuming from what i see above, and that GruntReporter has no callback, that synchronous code is safe here.

[kmiyashiro]

Thanks!