Bug 1729187 [wpt PR 30349] - Remove tools/quic and aioquic, a=testonly

Automatic update from web-platform-tests
Remove tools/quic and aioquic (#30349)

For WebTransport over HTTP/3, we plan to use virtualenv and
requirements.txt for aioquic dependency.
--

wpt-commits: d4353831216071b21558800f4e824508b56a2a4a
wpt-pr: 30349

Author: bashi

testing/web-platform/tests/docs/writing-tests/server-features.md

@@ -22,8 +22,6 @@ precise details of the response:
 
 * *pywebsocket*, an existing websockets server
 
-* *tools/quic*, a custom Python 3 QUIC server using `aioquic`
-
 wptserve is a Python-based web server. By default it serves static
 files in the test suite. For more sophisticated requirements, several
 mechanisms are available to take control of the response. These are
@@ -134,25 +132,14 @@ supported in
 [OpenSSL 1.0.2](https://www.openssl.org/news/openssl-1.0.2-notes.html) and up.
 
 
-### Tests Requiring QUIC
-
-We do not support loading a test over QUIC yet, but a test can establish a QUIC
-connection to the test server (e.g. for WebTransport, similar to WebSocket).
-Since the QUIC server is not yet enabled by default, tests must explicitly
-declare that they need access to the QUIC server:
+### Tests Requiring WebTransport over HTTP/3
 
-* For HTML tests (including testharness.js and reference tests), add the
-  following element:
-```html
-<meta name="quic" content="true">
-```
-* For JavaScript tests (auto-generated tests), add the following comment:
-```js
-// META: quic=true
-```
+We do not support loading a test over WebTransport over HTTP/3 yet, but a test
+can establish a WebTransport session to the test server.
 
-The QUIC server is not yet enabled by default, so QUIC tests will be skipped
-unless `--enable-quic` is specified to `./wpt run`.
+The WebTransport over HTTP/3 server is not yet enabled by default, so
+WebTransport tests will fail unless `--enable-webtransport` is specified to
+ `./wpt run`.
 
 ### Test Features specified as query params
 

testing/web-platform/tests/tools/lint/lint.py

@@ -694,8 +694,7 @@ def check(self, root):
 
 def check_python_ast(repo_root, path, f):
     # type: (Text, Text, IO[bytes]) -> List[rules.Error]
-    # *.quic.py are Python 3 only and cannot be parsed by Python 2.
-    if not path.endswith(".py") or path.endswith(".quic.py"):
+    if not path.endswith(".py"):
         return []
 
     try:

testing/web-platform/tests/tools/manifest/item.py

@@ -198,11 +198,6 @@ def jsshell(self):
         # type: () -> Optional[Text]
         return self._extras.get("jsshell")
 
-    @property
-    def quic(self):
-        # type: () -> Optional[bool]
-        return self._extras.get("quic")
-
     @property
     def script_metadata(self):
         # type: () -> Optional[List[Tuple[Text, Text]]]
@@ -217,8 +212,6 @@ def to_json(self):
             rv[-1]["testdriver"] = self.testdriver
         if self.jsshell:
             rv[-1]["jsshell"] = True
-        if self.quic is not None:
-            rv[-1]["quic"] = self.quic
         if self.script_metadata:
             rv[-1]["script_metadata"] = [(k, v) for (k,v) in self.script_metadata]
         return rv

testing/web-platform/tests/tools/manifest/sourcefile.py

@@ -738,36 +738,6 @@ def has_testdriver(self):
             return None
         return bool(self.testdriver_nodes)
 
-    @cached_property
-    def quic_nodes(self):
-        # type: () -> List[ElementTree.Element]
-        """List of ElementTree Elements corresponding to nodes in a test that
-        specify whether it needs QUIC server."""
-        assert self.root is not None
-        return self.root.findall(".//{http://www.w3.org/1999/xhtml}meta[@name='quic']")
-
-    @cached_property
-    def quic(self):
-        # type: () -> Optional[bool]
-        """Boolean indicating whether a test requires QUIC server
-
-        Determined by <meta> elements (`quic_nodes()`) and "// META" comments
-        (`script_metadata()`).
-        """
-        if self.script_metadata:
-            if any(m == ("quic", "true") for m in self.script_metadata):
-                return True
-
-        if self.root is None:
-            return None
-
-        if self.quic_nodes:
-            quic_str = self.quic_nodes[0].attrib.get("content", "false")  # type: Text
-            if quic_str.lower() == "true":
-                return True
-
-        return None
-
     @cached_property
     def reftest_nodes(self):
         # type: () -> List[ElementTree.Element]
@@ -1031,7 +1001,6 @@ def manifest_items(self):
                     global_variant_url(self.rel_url, suffix) + variant,
                     timeout=self.timeout,
                     jsshell=jsshell,
-                    quic=self.quic,
                     script_metadata=self.script_metadata
                 )
                 for (suffix, jsshell) in sorted(global_suffixes(globals))
@@ -1048,7 +1017,6 @@ def manifest_items(self):
                     self.url_base,
                     test_url + variant,
                     timeout=self.timeout,
-                    quic=self.quic,
                     script_metadata=self.script_metadata
                 )
                 for variant in self.test_variants
@@ -1064,7 +1032,6 @@ def manifest_items(self):
                     self.url_base,
                     test_url + variant,
                     timeout=self.timeout,
-                    quic=self.quic,
                     script_metadata=self.script_metadata
                 )
                 for variant in self.test_variants
@@ -1091,7 +1058,6 @@ def manifest_items(self):
                     self.url_base,
                     url,
                     timeout=self.timeout,
-                    quic=self.quic,
                     testdriver=testdriver,
                     script_metadata=self.script_metadata
                 ))
@@ -1105,7 +1071,6 @@ def manifest_items(self):
                     self.rel_url,
                     references=self.references,
                     timeout=self.timeout,
-                    quic=self.quic,
                     viewport_size=self.viewport_size,
                     dpi=self.dpi,
                     fuzzy=self.fuzzy

testing/web-platform/tests/tools/manifest/tests/test_sourcefile.py

@@ -170,7 +170,6 @@ def test_worker():
     for item, url in zip(items, expected_urls):
         assert item.url == url
         assert item.timeout is None
-        assert item.quic is None
 
 
 def test_window():
@@ -196,7 +195,6 @@ def test_window():
     for item, url in zip(items, expected_urls):
         assert item.url == url
         assert item.timeout is None
-        assert item.quic is None
 
 
 def test_worker_long_timeout():
@@ -262,7 +260,6 @@ def test_worker_with_variants():
     for item, url in zip(items, expected_urls):
         assert item.url == url
         assert item.timeout is None
-        assert item.quic is None
 
 
 def test_window_with_variants():
@@ -293,7 +290,6 @@ def test_window_with_variants():
     for item, url in zip(items, expected_urls):
         assert item.url == url
         assert item.timeout is None
-        assert item.quic is None
 
 
 def test_python_long_timeout():
@@ -338,7 +334,6 @@ def test_multi_global():
     for item, url in zip(items, expected_urls):
         assert item.url == url
         assert item.timeout is None
-        assert item.quic is None
 
 
 def test_multi_global_long_timeout():
@@ -398,7 +393,6 @@ def test_multi_global_with_custom_globals(input, expected):
         assert item.url == url
         assert item.jsshell is False
         assert item.timeout is None
-        assert item.quic is None
 
 
 def test_multi_global_with_jsshell_globals():
@@ -429,7 +423,6 @@ def test_multi_global_with_jsshell_globals():
         assert item.url == url
         assert item.jsshell == jsshell
         assert item.timeout is None
-        assert item.quic is None
 
 
 def test_multi_global_with_variants():
@@ -468,7 +461,6 @@ def test_multi_global_with_variants():
     for item, url in zip(items, expected_urls):
         assert item.url == url
         assert item.timeout is None
-        assert item.quic is None
 
 
 @pytest.mark.parametrize("input,expected", [
@@ -677,35 +669,6 @@ def test_relative_testdriver(ext):
     assert not s.has_testdriver
 
 
-@pytest.mark.parametrize("ext", ["htm", "html"])
-def test_quic_html(ext):
-    filename = "html/test." + ext
-
-    content = b'<meta name="quic" content="true">'
-    s = create(filename, content)
-    assert s.quic
-
-    content = b'<meta name="quic" content="false">'
-    s = create(filename, content)
-    assert s.quic is None
-
-
-def test_quic_js():
-    filename = "html/test.any.js"
-
-    content = b"// META: quic=true"
-    s = create(filename, content)
-    _, items = s.manifest_items()
-    for item in items:
-        assert item.quic
-
-    content = b"// META: quic=false"
-    s = create(filename, content)
-    _, items = s.manifest_items()
-    for item in items:
-        assert item.quic is None
-
-
 @pytest.mark.parametrize("ext", ["htm", "html"])
 def test_reftest(ext):
     content = b"<link rel=match href=ref.html>"

testing/web-platform/tests/tools/mypy.ini

@@ -24,9 +24,6 @@ warn_unused_ignores = True
 
 # Ignore missing or untyped libraries.
 
-[mypy-aioquic.*]
-ignore_missing_imports = True
-
 [mypy-github.*]
 ignore_missing_imports = True