Used const instead of var

src/test/testrunner.js

@@ -1,48 +1,57 @@
-var
+const webpage = require('webpage');
-  webpage = require('webpage'),
+const urls = ['src/test/index.html'];
-  urls = [
-    'src/test/index.html'
-  ]
-;
 
-(function test() {'use strict';
+async function runTests() {
-  var page, url = urls.shift();
+    for (const url of urls) {
-  if (!url) return phantom.exit(0);
+        console.log(`Loading: ${url}`);
-  console.log('Loading: ' + url);
+        const page = await new Promise((resolve, reject) => {
-  page = webpage.create();
+            const page = webpage.create();
-  page.open(url, function (status) {
+            page.onLoadFinished = status => {
-    if (status === 'success') {
+                if (status === 'success') resolve(page);
-      setTimeout(function () {
+                else reject(new Error(`Failed to load: ${url}`));
-        var results = page.evaluate(function() {
+            };
-          // remove the first node with the total from the following counts
+            page.open(url);
-          var passed = Math.max(0, document.querySelectorAll('.pass').length - 1);
+        });
-          var resultHeader = document.querySelector('#wru strong');
+
-          return {
+        try {
-            // retrieve the total executed tests number
+            const results = await evaluateTestResults(page);
-            total: ''.concat(
+            printTestResults(results);
-              passed,
+            if (results.passed === 0 || results.failed + results.errored > 0) {
-              ' tests (',
+                console.error('Tests failed.');
-              resultHeader ? resultHeader.textContent.replace(/\D/g, '') : 'no',
+                process.exit(1);
-              ' assertions)'
+            }
-            ),
+        } catch (error) {
+            console.error(error);
+            process.exit(1);
+        } finally {
+            page.close();
+        }
+    }
+    console.log('All tests completed successfully.');
+    process.exit(0);
+}
+
+async function evaluateTestResults(page) {
+    return await page.evaluate(() => {
+        const passed = Math.max(0, document.querySelectorAll('.pass').length - 1);
+        const resultHeader = document.querySelector('#wru strong');
+        return {
+            total: `${passed} tests (${resultHeader ? resultHeader.textContent.replace(/\D/g, '') : 'no'} assertions)`,
             passed: passed,
             failed: Math.max(0, document.querySelectorAll('.fail').length - 1),
             errored: Math.max(0, document.querySelectorAll('.error').length - 1)
-          };
+        };
-        });
+    });
-        page.stop();
+}
-        page.close();
+
-        console.log('- - - - - - - - - -');
+function printTestResults(results) {
-        console.log('total:   ' + results.total);
+    console.log('- - - - - - - - - -');
-        console.log('- - - - - - - - - -');
+    console.log(`total:   ${results.total}`);
-        console.log('passed:  ' + results.passed);
+    console.log('- - - - - - - - - -');
-        console.log('failed:  ' + results.failed);
+    console.log(`passed:  ${results.passed}`);
-        console.log('errored: ' + results.errored);
+    console.log(`failed:  ${results.failed}`);
-        console.log('- - - - - - - - - -');
+    console.log(`errored: ${results.errored}`);
-        if (results.passed === 0 || 0 < results.failed + results.errored) {
+    console.log('- - - - - - - - - -');
-          phantom.exit(1);
+}
-        } else test();
+
-      }, 1000);
+runTests();
-    } else phantom.exit(1);
-  });
-}());