fix: keep running WPTs if a test times out (#2165)

nodejs · Jun 20, 2023 · 593c56c · 593c56c
1 parent c415fbb
commit 593c56c
Showing 1 changed file with 3 additions and 7 deletions.
diff --git a/test/wpt/runner/runner.mjs b/test/wpt/runner/runner.mjs
@@ -228,19 +228,15 @@ export class WPTRunner extends EventEmitter {
           if (variant) console.log('Variant:', variant)
           console.log(`Test took ${(performance.now() - start).toFixed(2)}ms`)
           console.log('='.repeat(96))
-
+        } catch (e) {
+          console.log(`${test} timed out after ${timeout}ms`)
+        } finally {
           if (result?.subtests.length > 0) {
             writeFileSync(this.#reportPath, JSON.stringify(report))
           }
 
           finishedFiles++
           activeWorkers.delete(worker)
-        } catch (e) {
-          console.log(`${test} timed out after ${timeout}ms`)
-          queueMicrotask(() => {
-            throw e
-          })
-          return
         }
       }
     }