Bug 1614579 [wpt PR 21718] - Fetch: improve ReadableStream upload tes…

…t, a=testonly

Automatic update from web-platform-tests
Fetch: improve ReadableStream upload test

See whatwg/fetch#88 (comment) onward.
--

wpt-commits: 867cf376fd8ec8e39a5025e2dccb307c1c2773bb
wpt-pr: 21718

UltraBlame original commit: d5e0c700360058cd2d46d713c5c457cc8a1feb47

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

@@ -1,11 +1,12 @@
 
 
 function testUpload(desc, url, method, createBody, expectedBody) {
-  var requestInit = {"method": method}
+  const requestInit = {"method": method}
   promise_test(function(test){
-    let body = createBody();
-    if (body)
+    const body = createBody();
+    if (body) {
       requestInit["body"] = body;
+    }
     return fetch(url, requestInit).then(function(resp) {
       return resp.text().then((text)=> {
         assert_equals(text, expectedBody);
@@ -16,15 +17,16 @@ function testUpload(desc, url, method, createBody, expectedBody) {
 
 function testUploadFailure(desc, url, method, createBody) {
   const requestInit = {"method": method};
-  promise_test(test => {
-    let body = createBody();
-    if (body)
+  promise_test(t => {
+    const body = createBody();
+    if (body) {
       requestInit["body"] = body;
-    return promise_rejects(new TypeError(), fetch(url, requestInit));
+    }
+    return promise_rejects_js(t, TypeError, fetch(url, requestInit));
   }, desc);
 }
 
-var url = RESOURCES_DIR + "echo-content.py"
+const url = RESOURCES_DIR + "echo-content.py"
 
 testUpload("Fetch with PUT with body", url,
   "PUT",
@@ -73,7 +75,7 @@ testUpload("Fetch with POST with Blob body with mime type", url,
 testUpload("Fetch with POST with ReadableStream", url,
   "POST",
   () => {
-    new ReadableStream({start: controller => {
+    return new ReadableStream({start: controller => {
       const encoder = new TextEncoder();
       controller.enqueue(encoder.encode("Test"));
       controller.close();
@@ -83,39 +85,39 @@ testUpload("Fetch with POST with ReadableStream", url,
 testUploadFailure("Fetch with POST with ReadableStream containing String", url,
   "POST",
   () => {
-    new ReadableStream({start: controller => {
+    return new ReadableStream({start: controller => {
       controller.enqueue("Test");
       controller.close();
     }})
   });
 testUploadFailure("Fetch with POST with ReadableStream containing null", url,
   "POST",
   () => {
-    new ReadableStream({start: controller => {
+    return new ReadableStream({start: controller => {
       controller.enqueue(null);
       controller.close();
     }})
   });
 testUploadFailure("Fetch with POST with ReadableStream containing number", url,
   "POST",
   () => {
-    new ReadableStream({start: controller => {
+    return new ReadableStream({start: controller => {
       controller.enqueue(99);
       controller.close();
     }})
   });
 testUploadFailure("Fetch with POST with ReadableStream containing ArrayBuffer", url,
   "POST",
   () => {
-    new ReadableStream({start: controller => {
+    return new ReadableStream({start: controller => {
       controller.enqueue(new ArrayBuffer());
       controller.close();
     }})
   });
 testUploadFailure("Fetch with POST with ReadableStream containing Blob", url,
   "POST",
   () => {
-    new ReadableStream({start: controller => {
+    return new ReadableStream({start: controller => {
       controller.enqueue(new Blob());
       controller.close();
     }})