Update test_httpservers.py

Author: picnixz

Lib/test/test_httpservers.py

@@ -1472,6 +1472,14 @@ def setUp(self):
             f.write(self.tls_password.encode())
         self.addCleanup(os_helper.unlink, self.tls_password_file)
 
+    @unittest.skipIf(ssl is None, "requires ssl")
+    def get_ssl_context(self):
+        context = ssl.create_default_context()
+        # allow self-signed certificates
+        context.check_hostname = False
+        context.verify_mode = ssl.CERT_NONE
+        return context
+
     def fetch_file(self, path, context=None):
         req = urllib.request.Request(path, method='GET')
         with urllib.request.urlopen(req, context=context) as res:
@@ -1506,8 +1514,7 @@ def wait_for_server(self, proc, protocol, port, bind, timeout=50):
         return False
 
     def test_http_client(self):
-        port = find_unused_port()
-        bind = '127.0.0.1'
+        _, (bind, port) = server._get_best_family(None, find_unused_port())
         proc = spawn_python('-u', '-m', 'http.server', str(port), '-b', bind,
                             bufsize=1, text=True)
         self.addCleanup(kill_python, proc)
@@ -1516,15 +1523,8 @@ def test_http_client(self):
         res = self.fetch_file(f'http://{bind}:{port}/{self.served_file_name}')
         self.assertEqual(res, self.served_data)
 
-    @unittest.skipIf(ssl is None, "requires ssl")
     def test_https_client(self):
-        context = ssl.create_default_context()
-        # allow self-signed certificates
-        context.check_hostname = False
-        context.verify_mode = ssl.CERT_NONE
-
-        port = find_unused_port()
-        bind = '127.0.0.1'
+        _, (bind, port) = server._get_best_family(None, find_unused_port())
         proc = spawn_python('-u', '-m', 'http.server', str(port), '-b', bind,
                             '--tls-cert', self.tls_cert,
                             '--tls-key', self.tls_key,
@@ -1533,8 +1533,8 @@ def test_https_client(self):
         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}',
-                              context=context)
+        url = f'https://{bind}:{port}/{self.served_file_name}'
+        res = self.fetch_file(url, context=self.get_ssl_context())
         self.assertEqual(res, self.served_data)