axemclion/grunt-saucelabs — pull request #47 by pimterry (2013-06-19 19:25:42+00:00)
Title: Fix bug where we continue parsing version number after complaining it's not present, and crash

If the version number could not be found (which is a problem I'm currently seeing for some reason, but unrelated), grunt-saucelabs currently records the error and clearly explains it, registers the test as failed, and then goes on and tries to parse it.

This immediately fails (with `Fatal error: Cannot call method 'match' of undefined`), which crashes grunt, so the test is not recorded as failed on saucelabs (it times out 90 seconds later), and if you're running multiple browsers in parallel you can't even tell which failed.

This now fixes that, by failing the one test and then stopping that test entirely: a quick test shows test failures now being sensibly recorded by both grunt and saucelabs itself.