Bug 1729382 [wpt PR 30366] - Update files in wpt/webtransport, a=testonly

Automatic update from web-platform-tests
Update files in wpt/webtransport

This CL just updates the formatting of files in wpt/webtransport by
using:
- test helper functions where possible
- single quotations for strings
- PEP8 style for python handlers

Bug: 1201569
Change-Id: I39f475a9de1522fea8bf5a3fb231bfdff2294878
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3142703
Reviewed-by: Kenichi Ishibashi <[email protected]>
Commit-Queue: Nidhi Jaju <[email protected]>
Cr-Commit-Position: refs/heads/main@{#918675}

--

wpt-commits: e9162bcc6fa2449b55004f9b4b040f60b744f1ae
wpt-pr: 30366

Author: nidhijaju

testing/web-platform/tests/webtransport/constructor.any.js

@@ -1,7 +1,6 @@
 // META: global=window,worker
 // META: script=/common/get-host-info.sub.js
-
-const HOST = get_host_info().ORIGINAL_HOST;
+// META: script=resources/webtransport-test-helpers.sub.js
 
 const BAD_URLS = [
   null,
@@ -21,12 +20,10 @@ for (const url of BAD_URLS) {
   }, `WebTransport constructor should reject URL '${url}'`);
 }
 
-// TODO(bashi): Test CSP.
-
 promise_test(t => {
   const wt = new WebTransport(`https://${HOST}:0/`);
   return Promise.all([
-    promise_rejects_js(t, TypeError, wt.ready, 'ready promise rejects'),
-    promise_rejects_js(t, TypeError, wt.closed, 'closed promise rejects'),
+    promise_rejects_js(t, TypeError, wt.ready, 'ready promise should be rejected'),
+    promise_rejects_js(t, TypeError, wt.closed, 'closed promise should be rejected'),
   ]);
-}, 'connection to port 0 should fail');
+}, 'Connection to port 0 should fail');

testing/web-platform/tests/webtransport/csp-fail.sub.window.js

@@ -1,13 +1,10 @@
 // META: global=window,worker
 // META: script=/common/get-host-info.sub.js
-
-const HOST = get_host_info().ORIGINAL_HOST;
-const PORT = '{{ports[webtransport-h3][0]}}';
-const BASE = `https://${HOST}:${PORT}`;
+// META: script=resources/webtransport-test-helpers.sub.js
 
 function set_csp(destination) {
-  let meta = document.createElement("meta");
-  meta.httpEquiv = "Content-Security-Policy";
+  let meta = document.createElement('meta');
+  meta.httpEquiv = 'Content-Security-Policy';
   meta.content = `connect-src ${destination}`;
   return meta;
 }
@@ -16,7 +13,7 @@ promise_test(async t => {
   let meta = set_csp("'none'");
   document.head.appendChild(meta);
 
-  let wt = new WebTransport(`${BASE}/handlers/custom-response.py?:status=200`);
+  let wt = new WebTransport(webtransport_url('custom-response.py?:status=200'));
   await promise_rejects_dom(t, 'SecurityError', wt.ready, 'ready promise should be rejected');
   await promise_rejects_dom(t, 'SecurityError', wt.closed, 'closed promise should be rejected');
-}, "WebTransport connection should fail when CSP connect-src is set to none and reject the promises");
+}, 'WebTransport connection should fail when CSP connect-src is set to none and reject the promises');

testing/web-platform/tests/webtransport/csp-pass.sub.window.js

@@ -1,13 +1,10 @@
 // META: global=window,worker
 // META: script=/common/get-host-info.sub.js
-
-const HOST = get_host_info().ORIGINAL_HOST;
-const PORT = '{{ports[webtransport-h3][0]}}';
-const BASE = `https://${HOST}:${PORT}`;
+// META: script=resources/webtransport-test-helpers.sub.js
 
 function set_csp(destination) {
-  let meta = document.createElement("meta");
-  meta.httpEquiv = "Content-Security-Policy";
+  let meta = document.createElement('meta');
+  meta.httpEquiv = 'Content-Security-Policy';
   meta.content = `connect-src ${destination}`;
   return meta;
 }
@@ -16,6 +13,6 @@ promise_test(async t => {
  let meta = set_csp(`${BASE}`);
  document.head.appendChild(meta);
 
-  let wt = new WebTransport(`${BASE}/webtransport/handlers/custom-response.py?:status=200`);
+  let wt = new WebTransport(webtransport_url('custom-response.py?:status=200'));
   await wt.ready;
-}, "WebTransport connection should succeed when CSP connect-src destination is set to the page");
+}, 'WebTransport connection should succeed when CSP connect-src destination is set to the page');

testing/web-platform/tests/webtransport/handlers/custom-response.py

@@ -1,13 +1,14 @@
 from urllib.parse import urlsplit, parse_qsl
 
+
 def connect_received(request_headers, response_headers):
-  for data in request_headers:
-    if data[0] == b':path':
-      path = data[1].decode('utf-8')
+    for data in request_headers:
+        if data[0] == b':path':
+            path = data[1].decode('utf-8')
 
-      qs = dict(parse_qsl(urlsplit(path).query))
-      for key, value in qs.items():
-        response_headers.append((key.encode('utf-8'), value.encode('utf-8')))
+            qs = dict(parse_qsl(urlsplit(path).query))
+            for key, value in qs.items():
+                response_headers.append((key.encode('utf-8'), value.encode('utf-8')))
 
-      break
-  return
+            break
+    return