worker: throw InvalidStateError in postMessage after close

This update addresses expected failures for invalid postMessage after close in WPT. PR-URL: nodejs#55206 Reviewed-By: Mattias Buelens <[email protected]> Reviewed-By: Matthew Aitken <[email protected]>
tpoisseau · Nov 21, 2024 · 396911c · 396911c
1 parent 02f60d5
commit 396911c
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 9 deletions.
diff --git a/lib/internal/worker/io.js b/lib/internal/worker/io.js
@@ -402,7 +402,7 @@ class BroadcastChannel extends EventTarget {
     if (arguments.length === 0)
       throw new ERR_MISSING_ARGS('message');
     if (this[kHandle] === undefined)
-      throw new DOMException('BroadcastChannel is closed.');
+      throw new DOMException('BroadcastChannel is closed.', 'InvalidStateError');
     if (this[kHandle].postMessage(message) === undefined)
       throw new DOMException('Message could not be posted.');
   }

diff --git a/test/wpt/status/webmessaging/broadcastchannel.json b/test/wpt/status/webmessaging/broadcastchannel.json
@@ -10,14 +10,6 @@
       ]
     }
   },
-  "interface.any.js": {
-    "fail": {
-      "expected": [
-        "postMessage after close should throw",
-        "postMessage should throw InvalidStateError after close, even with uncloneable data"
-      ]
-    }
-  },
   "origin.window.js": {
     "fail": {
       "expected": [