lib,test: enable wasm/webapi/empty-body WPT

tniessen · tniessen · commit 26a6c360a787 · 2022-05-03T21:16:58.000Z
Refs: nodejs#42701 Refs: nodejs/undici#1346 Refs: nodejs#42939
diff --git a/lib/internal/bootstrap/pre_execution.js b/lib/internal/bootstrap/pre_execution.js
@@ -250,8 +250,11 @@ function setupFetch() {
       }
 
       // Pass all data from the response body to the WebAssembly compiler.
-      for await (const chunk of response.body) {
-        streamState.push(chunk);
+      const { body } = response;
+      if (body != null) {
+        for await (const chunk of body) {
+          streamState.push(chunk);
+        }
       }
     })().then(() => {
       // No error occurred. Tell the implementation that the stream has ended.
diff --git a/test/wpt/status/wasm/webapi.json b/test/wpt/status/wasm/webapi.json
@@ -12,9 +12,6 @@
   "contenttype.any.js": {
     "skip": "WPTRunner does not support fetch()"
   },
-  "empty-body.any.js": {
-    "skip": "Bug in undici, see https://github.com/nodejs/undici/issues/1345"
-  },
   "idlharness.any.js": {
     "skip": "not configured"
   },

Original file line number	Diff line number	Diff line change
`@@ -250,8 +250,11 @@ function setupFetch() {`
`250`	`250`	`}`
`251`	`251`
`252`	`252`	`// Pass all data from the response body to the WebAssembly compiler.`
`253`		`- for await (const chunk of response.body) {`
`254`		`- streamState.push(chunk);`
	`253`	`+ const { body } = response;`
	`254`	`+ if (body != null) {`
	`255`	`+ for await (const chunk of body) {`
	`256`	`+ streamState.push(chunk);`
	`257`	`+ }`
`255`	`258`	`}`
`256`	`259`	`})().then(() => {`
`257`	`260`	`// No error occurred. Tell the implementation that the stream has ended.`