Put mypy into strict mode, adapt code to still pass.

[plomtask] / tests / utils.py

diff --git a/tests/utils.py b/tests/utils.py
index cd0c457c0b0affb0034f5e81de2bc5d4a6139263..9964201bbc7d6312ed1e8cb29e1a5a359608e86f 100644 (file)
--- a/tests/utils.py
+++ b/tests/utils.py
@@ -1,6 +1,7 @@
 """Shared test utilities."""
 from unittest import TestCase
 from threading import Thread
+from http.client import HTTPConnection
 from datetime import datetime
 from os import remove as remove_file
 from plomtask.db import DatabaseFile, DatabaseConnection
@@ -10,13 +11,13 @@ from plomtask.http import TaskHandler, TaskServer
 class TestCaseWithDB(TestCase):
     """Module tests not requiring DB setup."""
 
-    def setUp(self):
+    def setUp(self) -> None:
         timestamp = datetime.now().timestamp()
         self.db_file = DatabaseFile(f'test_db:{timestamp}')
         self.db_file.remake()
         self.db_conn = DatabaseConnection(self.db_file)
 
-    def tearDown(self):
+    def tearDown(self) -> None:
         self.db_conn.close()
         remove_file(self.db_file.path)
 
@@ -24,14 +25,16 @@ class TestCaseWithDB(TestCase):
 class TestCaseWithServer(TestCaseWithDB):
     """Module tests against our HTTP server/handler (and database)."""
 
-    def setUp(self):
+    def setUp(self) -> None:
         super().setUp()
         self.httpd = TaskServer(self.db_file, ('localhost', 0), TaskHandler)
         self.server_thread = Thread(target=self.httpd.serve_forever)
         self.server_thread.daemon = True
         self.server_thread.start()
+        self.conn = HTTPConnection(str(self.httpd.server_address[0]),
+                                   self.httpd.server_address[1])
 
-    def tearDown(self):
+    def tearDown(self) -> None:
         self.httpd.shutdown()
         self.httpd.server_close()
         self.server_thread.join()