nodejs · targos · Mar 6, 2021 · aduh95 · Mar 6, 2021 · targos
diff --git a/test/common/wpt.js b/test/common/wpt.js
@@ -299,6 +299,7 @@ class WPTRunner {
 
     this.results = {};
     this.inProgress = new Set();
+    this.workers = new Map();
     this.unexpectedFailures = [];
   }
 
@@ -376,6 +377,7 @@ class WPTRunner {
           scriptsToRun,
         },
       });
+      this.workers.set(testFileName, worker);
 
       worker.on('message', (message) => {
         switch (message.type) {
@@ -496,6 +498,9 @@ class WPTRunner {
       this.resultCallback(filename, { status: 2, name: 'Unknown' });
     }
     this.inProgress.delete(filename);
+    // Always force termination of the worker. Some tests allocate resources
+    // that would otherwise keep it alive.
+    this.workers.get(filename).terminate();
-    this.workers.get(filename).terminate();
+    this.workers.get(filename).terminate().catch(() => {});
   if (this[kHandle] === null) return PromiseResolve(); 
   this[kHandle].stopThread(); 
   // Do not use events.once() here, because the 'exit' event will always be 
   // emitted regardless of any errors, and the point is to only resolve 
   // once the thread has actually stopped. 
   return new Promise((resolve) => { 
     this.once('exit', resolve); 
   }); 
 } 
-    this.workers.get(filename).terminate();
+    this.workers.get(filename).terminate().catch(() => {});
   if (this[kHandle] === null) return PromiseResolve(); 
  
   this[kHandle].stopThread(); 
  
   // Do not use events.once() here, because the 'exit' event will always be 
   // emitted regardless of any errors, and the point is to only resolve 
   // once the thread has actually stopped. 
   return new Promise((resolve) => { 
     this.once('exit', resolve); 
   }); 
 } 
   }
 
   addTestResult(filename, item) {

diff --git a/test/wpt/status/FileAPI/blob.json b/test/wpt/status/FileAPI/blob.json
@@ -1,6 +1,6 @@
 {
   "Blob-constructor.any.js": {
-    "skip": "https://github.com/nodejs/node/issues/37358"
+    "skip": "Depends on File API"
   },
   "Blob-constructor-dom.window.js": {
     "skip": "Depends on DOM API"