Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use a for...of loop in the specDone handler in the test reporter

Browse Source 
Moreover, remove a left-over reference to `test.py` since that was
ported to JavaScript a long time ago.
This commit is contained in:
Tim van der Meij 2020-08-01 20:49:24 +02:00
parent 252f2ba529
commit c19d76f9b8
No known key found for this signature in database
GPG Key ID: 8C3FD2925A5F2762
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
test/unit/testreporter.js
View File

@ -59,10 +59,9 @@ var TestReporter = function (browser) {
if (result.failedExpectations.length === 0) { if (result.failedExpectations.length === 0) {
sendResult("TEST-PASSED", result.description); sendResult("TEST-PASSED", result.description);
} else { } else {
var failedMessages = ""; let failedMessages = "";
var items = result.failedExpectations; for (const item of result.failedExpectations) {
for (var i = 0, ii = items.length; i < ii; i++) { failedMessages += `${item.message} `;
failedMessages += items[i].message + " ";
} }
sendResult("TEST-UNEXPECTED-FAIL", result.description, failedMessages); sendResult("TEST-UNEXPECTED-FAIL", result.description, failedMessages);
} }
@ -71,7 +70,7 @@ var TestReporter = function (browser) {
this.suiteDone = function (result) {}; this.suiteDone = function (result) {};
this.jasmineDone = function () { 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); setTimeout(sendQuitRequest, 500);
}; };
}; };