fetch: Try closing connections more aggressively

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

index c19e4dba303d2fe0da536d17539f1f6ae6ae38ed..fafd1c37284c80486f0859aa33c39e0e2728153d 100644 (file)
--- a/paperdoorknob_test.py
+++ b/paperdoorknob_test.py
@@ -8,17 +8,24 @@
  import unittest
  import threading
  from http.server import BaseHTTPRequestHandler, HTTPServer
+import requests
  import paperdoorknob
  
-TEST_PORT = 8080
  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()
@@ -27,7 +34,8 @@ class FakeGlowficHTTPRequestHandler(BaseHTTPRequestHandler):
  
  class TestFetch(unittest.TestCase):
      def setUp(self) -> None:
-        web_server = HTTPServer(('', TEST_PORT), FakeGlowficHTTPRequestHandler)
+        web_server = HTTPServer(('', 0), FakeGlowficHTTPRequestHandler)
+        self._port = web_server.socket.getsockname()[1]
          threading.Thread(target=web_server.serve_forever).start()
          self._stop_server = web_server.shutdown
  
@@ -35,7 +43,15 @@ class TestFetch(unittest.TestCase):
          self._stop_server()
  
      def testFetch(self) -> None:
-        paperdoorknob.fetch(f"http://localhost:{TEST_PORT}", TIMEOUT)
+        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__':