test: fix WPT runner result

daeyeon · RafaelGSS · commit 3a5fdacdc245 · 2022-09-05T11:32:56.000-03:00
This doesn't include the other tests for the result when running a specific test in WPT. Signed-off-by: Daeyeon Jeong daeyeon.dev@gmail.com PR-URL: #44238 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
diff --git a/test/common/wpt.js b/test/common/wpt.js
@@ -474,7 +474,6 @@ class WPTRunner {
     }
 
     process.on('exit', () => {
-      const total = this.specMap.size;
       if (this.inProgress.size > 0) {
         for (const filename of this.inProgress) {
           this.fail(filename, { name: 'Unknown' }, kIncomplete);
@@ -506,7 +505,9 @@ class WPTRunner {
       }
 
       const unexpectedPasses = [];
-      for (const [key, specMap] of this.specMap) {
+      for (const specMap of queue) {
+        const key = specMap.filename;
+
         // File has no expected failures
         if (!specMap.failedTests.length) {
           continue;
@@ -529,7 +530,8 @@ class WPTRunner {
         }
       }
 
-      const ran = total - skipped;
+      const ran = queue.length;
+      const total = ran + skipped;
       const passed = ran - expectedFailures - failures.length;
       console.log(`Ran ${ran}/${total} tests, ${skipped} skipped,`,
                   `${passed} passed, ${expectedFailures} expected failures,`,

Original file line number	Diff line number	Diff line change
`@@ -474,7 +474,6 @@ class WPTRunner {`
`474`	`474`	`}`
`475`	`475`
`476`	`476`	`process.on('exit', () => {`
`477`		`- const total = this.specMap.size;`
`478`	`477`	`if (this.inProgress.size > 0) {`
`479`	`478`	`for (const filename of this.inProgress) {`
`480`	`479`	`this.fail(filename, { name: 'Unknown' }, kIncomplete);`
`@@ -506,7 +505,9 @@ class WPTRunner {`
`506`	`505`	`}`
`507`	`506`
`508`	`507`	`const unexpectedPasses = [];`
`509`		`- for (const [key, specMap] of this.specMap) {`
	`508`	`+ for (const specMap of queue) {`
	`509`	`+ const key = specMap.filename;`
	`510`	`+`
`510`	`511`	`// File has no expected failures`
`511`	`512`	`if (!specMap.failedTests.length) {`
`512`	`513`	`continue;`
`@@ -529,7 +530,8 @@ class WPTRunner {`
`529`	`530`	`}`
`530`	`531`	`}`
`531`	`532`
`532`		`- const ran = total - skipped;`
	`533`	`+ const ran = queue.length;`
	`534`	`+ const total = ran + skipped;`
`533`	`535`	`const passed = ran - expectedFailures - failures.length;`
`534`	`536`	console.log(`Ran ${ran}/${total} tests, ${skipped} skipped,`,
`535`	`537`	`${passed} passed, ${expectedFailures} expected failures,`,