fix(test): treat failure to launch tests as a test failure

The previous formulation of the test runner was not catching promise
rejections coming from the webdriver / browser. Node would print
warnings to the console, but the test runner would exit with code 0 and
CI would treat this as a passing build. Now the rejection is caught and
reported, and the test runner exits with code 1.

Existing functionality to look for failed test results has not changed.

Author: cwillisf

tests/jsunit/test_runner.js

@@ -24,14 +24,23 @@ var testHtml = function (htmlString) {
 
 var path = process.cwd();
 
-browser
-  .get("file://" + path + "/tests/jsunit/vertical_tests.html")
-  .then(function () { return browser.sleep(5000) })
-  .then(function () { return browser.findElement({id: "closureTestRunnerLog"}) })
-  .then(function (e) { return e.getText() })
-  .then(testHtml)
-  .then(function () { return browser.get("file://" + path + "/tests/jsunit/horizontal_tests.html")})
-  .then(function () { return browser.sleep(5000) })
-  .then(function () { return browser.findElement({id: "closureTestRunnerLog"}) })
-  .then(function (e) { return e.getText() })
-  .then(testHtml);
+var runTests = async function () {
+  var element, text;
+
+  await browser.get("file://" + path + "/tests/jsunit/vertical_tests.html");
+  await browser.sleep(5000);
+  element = await browser.findElement({id: "closureTestRunnerLog"});
+  text = await element.getText();
+  testHtml(text);
+
+  await browser.get("file://" + path + "/tests/jsunit/horizontal_tests.html");
+  await browser.sleep(5000);
+  element = await browser.findElement({id: "closureTestRunnerLog"});
+  text = await element.getText();
+  testHtml(text);
+};
+
+runTests().catch(e => {
+    console.error(e);
+    process.exit(1);
+});