fetch: test: Explicitly join webserver thread

[paperdoorknob] / paperdoorknob_test.py
diff --git a/paperdoorknob_test.py b/paperdoorknob_test.py

index 2bd354419c113d8d6e321bb8395278879ea65dcc..b1d41d493574103bc5b22696d72454a43c33d07e 100644 (file)
--- a/paperdoorknob_test.py
+++ b/paperdoorknob_test.py
@@ -8,6 +8,7 @@
  import unittest
  import threading
  from http.server import BaseHTTPRequestHandler, HTTPServer
+import requests
  import paperdoorknob
  
  TIMEOUT = 8
@@ -15,9 +16,16 @@ TIMEOUT = 8
  
  class FakeGlowficHTTPRequestHandler(BaseHTTPRequestHandler):
  
+    def _response_code(self) -> int:
+        if self.path == "/not_found":
+            return 404
+        if self.path == "/server_error":
+            return 500
+        return 200
+
      def do_GET(self) -> None:
          body = b'<html><body>This is glowfic</body></html>'
-        self.send_response(200)
+        self.send_response(self._response_code())
          self.send_header("Content-type", "text/html")
          self.send_header("Content-Length", str(len(body)))
          self.end_headers()
@@ -28,15 +36,25 @@ class TestFetch(unittest.TestCase):
      def setUp(self) -> None:
          web_server = HTTPServer(('', 0), FakeGlowficHTTPRequestHandler)
          self._port = web_server.socket.getsockname()[1]
-        threading.Thread(target=web_server.serve_forever).start()
+        self._thread = threading.Thread(target=web_server.serve_forever)
+        self._thread.start()
          self._stop_server = web_server.shutdown
  
      def tearDown(self) -> None:
          self._stop_server()
+        self._thread.join()
  
      def testFetch(self) -> None:
          paperdoorknob.fetch(f"http://localhost:{self._port}", TIMEOUT)
  
+    def testFetchErrors(self) -> None:
+        with self.assertRaises(requests.HTTPError):
+            paperdoorknob.fetch(
+                f"http://localhost:{self._port}/not_found", TIMEOUT)
+        with self.assertRaises(requests.HTTPError):
+            paperdoorknob.fetch(
+                f"http://localhost:{self._port}/server_error", TIMEOUT)
+
  
  if __name__ == '__main__':
      unittest.main()