gh-131178: remove runtime tests for `http.server` CLI (#134287)

The runtime behavior of `http.server` CLI is hard to test on an arbitrary platform.
As such, tests asserting the correctness of `python -m http.server` are temporarily
removed and will be rewritten later once a universal solution has been found.

diff --git a/Lib/test/test_httpservers.py b/Lib/test/test_httpservers.py
index 429d9005bd7605129feb56db10eb51bd04687d30..0af1c45ecb2a8b8b8ce1c8c60b239121521125d5 100644 (file)
--- a/Lib/test/test_httpservers.py
+++ b/Lib/test/test_httpservers.py
@@ -21,7 +21,6 @@ import email.utils
 import html
 import http, http.client
 import urllib.parse
-import urllib.request
 import tempfile
 import time
 import datetime
@@ -34,8 +33,6 @@ from test import support
 from test.support import (
     is_apple, import_helper, os_helper, threading_helper
 )
-from test.support.script_helper import kill_python, spawn_python
-from test.support.socket_helper import find_unused_port
 
 try:
     import ssl
@@ -1455,86 +1452,6 @@ class CommandLineTestCase(unittest.TestCase):
         self.assertIn('error', stderr.getvalue())
 
 
-class CommandLineRunTimeTestCase(unittest.TestCase):
-    served_data = os.urandom(32)
-    served_file_name = 'served_filename'
-    tls_cert = certdata_file('ssl_cert.pem')
-    tls_key = certdata_file('ssl_key.pem')
-    tls_password = 'somepass'
-
-    def setUp(self):
-        super().setUp()
-        with open(self.served_file_name, 'wb') as f:
-            f.write(self.served_data)
-        self.addCleanup(os_helper.unlink, self.served_file_name)
-        self.tls_password_file = tempfile.mktemp()
-        with open(self.tls_password_file, 'wb') as f:
-            f.write(self.tls_password.encode())
-        self.addCleanup(os_helper.unlink, self.tls_password_file)
-
-    def fetch_file(self, path):
-        context = ssl.create_default_context()
-        # allow self-signed certificates
-        context.check_hostname = False
-        context.verify_mode = ssl.CERT_NONE
-        req = urllib.request.Request(path, method='GET')
-        with urllib.request.urlopen(req, context=context) as res:
-            return res.read()
-
-    def parse_cli_output(self, output):
-        matches = re.search(r'\((https?)://([^/:]+):(\d+)/?\)', output)
-        if matches is None:
-            return None, None, None
-        return matches.group(1), matches.group(2), int(matches.group(3))
-
-    def wait_for_server(self, proc, protocol, port, bind, timeout=50):
-        """Check the server process output.
-
-        Return True if the server was successfully started
-        and is listening on the given port and bind address.
-        """
-        while timeout > 0:
-            line = proc.stdout.readline()
-            if not line:
-                time.sleep(0.1)
-                timeout -= 1
-                continue
-            protocol_, host_, port_ = self.parse_cli_output(line)
-            if not protocol_ or not host_ or not port_:
-                time.sleep(0.1)
-                timeout -= 1
-                continue
-            if protocol_ == protocol and host_ == bind and port_ == port:
-                return True
-            break
-        return False
-
-    def test_http_client(self):
-        port = find_unused_port()
-        bind = '127.0.0.1'
-        proc = spawn_python('-u', '-m', 'http.server', str(port), '-b', bind,
-                            bufsize=1, text=True)
-        self.addCleanup(kill_python, proc)
-        self.addCleanup(proc.terminate)
-        self.assertTrue(self.wait_for_server(proc, 'http', port, bind))
-        res = self.fetch_file(f'http://{bind}:{port}/{self.served_file_name}')
-        self.assertEqual(res, self.served_data)
-
-    def test_https_client(self):
-        port = find_unused_port()
-        bind = '127.0.0.1'
-        proc = spawn_python('-u', '-m', 'http.server', str(port), '-b', bind,
-                            '--tls-cert', self.tls_cert,
-                            '--tls-key', self.tls_key,
-                            '--tls-password-file', self.tls_password_file,
-                            bufsize=1, text=True)
-        self.addCleanup(kill_python, proc)
-        self.addCleanup(proc.terminate)
-        self.assertTrue(self.wait_for_server(proc, 'https', port, bind))
-        res = self.fetch_file(f'https://{bind}:{port}/{self.served_file_name}')
-        self.assertEqual(res, self.served_data)
-
-
 def setUpModule():
     unittest.addModuleCleanup(os.chdir, os.getcwd())