Bug 1792178 [wpt PR 36048] - Request upload: test synchronous feature detect, a=testonly

Automatic update from web-platform-tests
Request upload: test synchronous feature detect

For whatwg/fetch#1493.
--

wpt-commits: 9841e79d6f1301738125a2fbfbc6992c7b8ae366
wpt-pr: 36048

Author: jakearchibald

testing/web-platform/tests/fetch/api/basic/request-upload.h2.any.js

@@ -5,17 +5,21 @@
 
 const duplex = "half";
 
-function testUpload(desc, url, method, createBody, expectedBody) {
+async function assertUpload(url, method, createBody, expectedBody) {
   const requestInit = {method};
+  const body = createBody();
+  if (body) {
+    requestInit["body"] = body;
+    requestInit.duplex = "half";
+  }
+  const resp = await fetch(url, requestInit);
+  const text = await resp.text();
+  assert_equals(text, expectedBody);
+}
+
+function testUpload(desc, url, method, createBody, expectedBody) {
   promise_test(async () => {
-    const body = createBody();
-    if (body) {
-      requestInit["body"] = body;
-      requestInit.duplex = "half";
-    }
-    const resp = await fetch(url, requestInit);
-    const text = await resp.text();
-    assert_equals(text, expectedBody);
+    await assertUpload(url, method, createBody, expectedBody);
   }, desc);
 }
 
@@ -98,6 +102,63 @@ promise_test(async (test) => {
   assert_equals(await response.text(), 'test', `Response has correct body`);
 }, "Feature detect for POST with ReadableStream, using request object");
 
+test(() => {
+  let duplexAccessed = false;
+
+  const request = new Request("", {
+    body: new ReadableStream(),
+    method: "POST",
+    get duplex() {
+      duplexAccessed = true;
+      return "half";
+    },
+  });
+
+  assert_equals(
+    request.headers.get("Content-Type"),
+    null,
+    `Request should not have a content-type set`
+  );
+  assert_true(duplexAccessed, `duplex dictionary property should be accessed`);
+}, "Synchronous feature detect");
+
+// The asserts the synchronousFeatureDetect isn't broken by a partial implementation.
+// An earlier feature detect was broken by Safari implementing streaming bodies as part of Request,
+// but it failed when passed to fetch().
+// This tests ensures that UAs must not implement RequestInit.duplex and streaming request bodies without also implementing the fetch() parts.
+promise_test(async () => {
+  let duplexAccessed = false;
+
+  const request = new Request("", {
+    body: new ReadableStream(),
+    method: "POST",
+    get duplex() {
+      duplexAccessed = true;
+      return "half";
+    },
+  });
+
+  const supported =
+    request.headers.get("Content-Type") === null && duplexAccessed;
+
+  // If the feature detect fails, assume the browser is being truthful (other tests pick up broken cases here)
+  if (!supported) return false;
+
+  await assertUpload(
+    url,
+    "POST",
+    () =>
+      new ReadableStream({
+        start: (controller) => {
+          const encoder = new TextEncoder();
+          controller.enqueue(encoder.encode("Test"));
+          controller.close();
+        },
+      }),
+    "Test"
+  );
+}, "Synchronous feature detect fails if feature unsupported");
+
 promise_test(async (t) => {
   const body = createStream(["hello"]);
   const method = "POST";