]> git.scottworley.com Git - paperdoorknob/commitdiff
fetch: test: Hold reference to webserver
authorScott Worley <scottworley@scottworley.com>
Thu, 23 Nov 2023 20:49:40 +0000 (12:49 -0800)
committerScott Worley <scottworley@scottworley.com>
Wed, 20 Dec 2023 01:37:10 +0000 (17:37 -0800)
paperdoorknob_test.py

index b1d41d493574103bc5b22696d72454a43c33d07e..cf945d60faec978ef258eb3396224398944688e7 100644 (file)
@@ -33,27 +33,30 @@ class FakeGlowficHTTPRequestHandler(BaseHTTPRequestHandler):
 
 
 class TestFetch(unittest.TestCase):
 
 
 class TestFetch(unittest.TestCase):
+    def _port(self) -> int:
+        port = self._web_server.socket.getsockname()[1]
+        assert isinstance(port, int)
+        return port
+
     def setUp(self) -> None:
     def setUp(self) -> None:
-        web_server = HTTPServer(('', 0), FakeGlowficHTTPRequestHandler)
-        self._port = web_server.socket.getsockname()[1]
-        self._thread = threading.Thread(target=web_server.serve_forever)
+        self._web_server = HTTPServer(('', 0), FakeGlowficHTTPRequestHandler)
+        self._thread = threading.Thread(target=self._web_server.serve_forever)
         self._thread.start()
         self._thread.start()
-        self._stop_server = web_server.shutdown
 
     def tearDown(self) -> None:
 
     def tearDown(self) -> None:
-        self._stop_server()
+        self._web_server.shutdown()
         self._thread.join()
 
     def testFetch(self) -> None:
         self._thread.join()
 
     def testFetch(self) -> None:
-        paperdoorknob.fetch(f"http://localhost:{self._port}", TIMEOUT)
+        paperdoorknob.fetch(f"http://localhost:{self._port()}", TIMEOUT)
 
     def testFetchErrors(self) -> None:
         with self.assertRaises(requests.HTTPError):
             paperdoorknob.fetch(
 
     def testFetchErrors(self) -> None:
         with self.assertRaises(requests.HTTPError):
             paperdoorknob.fetch(
-                f"http://localhost:{self._port}/not_found", TIMEOUT)
+                f"http://localhost:{self._port()}/not_found", TIMEOUT)
         with self.assertRaises(requests.HTTPError):
             paperdoorknob.fetch(
         with self.assertRaises(requests.HTTPError):
             paperdoorknob.fetch(
-                f"http://localhost:{self._port}/server_error", TIMEOUT)
+                f"http://localhost:{self._port()}/server_error", TIMEOUT)
 
 
 if __name__ == '__main__':
 
 
 if __name__ == '__main__':