From c19d76f9b85aff8b5ec876d62c01bc845868ddec Mon Sep 17 00:00:00 2001 From: Tim van der Meij Date: Sat, 1 Aug 2020 20:49:24 +0200 Subject: [PATCH] Use a `for...of` loop in the `specDone` handler in the test reporter Moreover, remove a left-over reference to `test.py` since that was ported to JavaScript a long time ago. --- test/unit/testreporter.js | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/test/unit/testreporter.js b/test/unit/testreporter.js index c74825ae2..ab517d6f7 100644 --- a/test/unit/testreporter.js +++ b/test/unit/testreporter.js @@ -59,10 +59,9 @@ var TestReporter = function (browser) { if (result.failedExpectations.length === 0) { sendResult("TEST-PASSED", result.description); } else { - var failedMessages = ""; - var items = result.failedExpectations; - for (var i = 0, ii = items.length; i < ii; i++) { - failedMessages += items[i].message + " "; + let failedMessages = ""; + for (const item of result.failedExpectations) { + failedMessages += `${item.message} `; } sendResult("TEST-UNEXPECTED-FAIL", result.description, failedMessages); } @@ -71,7 +70,7 @@ var TestReporter = function (browser) { this.suiteDone = function (result) {}; this.jasmineDone = function () { - // Give the test.py some time process any queued up requests + // Give the test runner some time process any queued requests. setTimeout(sendQuitRequest, 500); }; };