From f789c132dabc3ba91fe3695fc6ceb91697e92b06 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nicol=C3=B2=20Ribaudo?= Date: Mon, 9 Jun 2025 18:45:33 +0200 Subject: [PATCH] Log test number and id in reftests The new output looks like ``` Harness thinks this browser is firefox Fetching manifest "/test/test_manifest.json"... done [1/1152] bug1755201: Skipping file "pdfs/bug1755201.pdf" [2/1152] issue14864: Skipping file "pdfs/issue14864.pdf" [3/1152] issue18503: Skipping file "pdfs/issue18503.pdf" [4/1152] issue19835: Skipping file "pdfs/issue19835.pdf" [5/1152] filled-background-range: Loading file "pdfs/filled-background.pdf" Loading page 1/3... Snapshotting... done Loading page 2/3... Snapshotting... done Loading page 3/3... Snapshotting... done [6/1152] tracemonkey-eq: Loading file "pdfs/tracemonkey.pdf" Loading page 1/14... Snapshotting... done Loading page 2/14... Snapshotting... done ``` This gives an idea of how many tests are left while they are running --- test/driver.js | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/test/driver.js b/test/driver.js index a991fa984..67fda3c51 100644 --- a/test/driver.js +++ b/test/driver.js @@ -604,10 +604,14 @@ class Driver { md5FileMap.set(task.md5, task.file); } + this._log( + `[${this.currentTask + 1}/${this.manifest.length}] ${task.id}:\n` + ); + // Support *linked* test-cases for the other suites, e.g. unit- and // integration-tests, without needing to run them as reference-tests. if (task.type === "other") { - this._log(`Skipping file "${task.file}"\n`); + this._log(` Skipping file "${task.file}"\n`); if (!task.link) { this._nextPage(task, 'Expected "other" test-case to be linked.'); @@ -618,7 +622,7 @@ class Driver { return; } - this._log('Loading file "' + task.file + '"\n'); + this._log(` Loading file "${task.file}"\n`); try { let xfaStyleElement = null; @@ -849,7 +853,7 @@ class Driver { if (task.pageNum > this._getLastPageNumber(task)) { if (++task.round < task.rounds) { - this._log(" Round " + (1 + task.round) + "\n"); + this._log(` Round ${1 + task.round}\n`); task.pageNum = task.firstPage || 1; } else { this.currentTask++; @@ -860,7 +864,7 @@ class Driver { if (task.skipPages?.includes(task.pageNum)) { this._log( - " Skipping page " + task.pageNum + "/" + task.pdfDoc.numPages + "...\n" + ` Skipping page ${task.pageNum}/${task.pdfDoc.numPages}...\n` ); task.pageNum++; this._nextPage(task); @@ -870,7 +874,7 @@ class Driver { if (!failure) { try { this._log( - " Loading page " + task.pageNum + "/" + task.pdfDoc.numPages + "... " + ` Loading page ${task.pageNum}/${task.pdfDoc.numPages}... ` ); ctx = this.canvas.getContext("2d", { alpha: false }); task.pdfDoc.getPage(task.pageNum).then(