tests: more coverage

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "stapler"
-version = "1.2.6"
+version = "1.2.4"
 description = "Static pages as simple as a gzip file"
 requires-python = ">=3.14"
 dependencies = [

stapler/handlers.py

@@ -300,8 +300,6 @@ class RequestHandler(http.server.SimpleHTTPRequestHandler, BaseHandler):
         self.__target_spa: str | None = None
         try:
             super().__init__(*args, directory=params.data_dir, **kwargs, params=params)  # ty:ignore[unknown-argument]
-        except (BrokenPipeError, ConnectionResetError) as e:
-            self.logger.error("Connection lost: %s", str(e))  # noqa: TRY400
         except:
             self.logger.exception("Could not handle request")
 

tests/test_data_dir.py

@@ -59,6 +59,11 @@ class TestDataDir(BaseTestCase):
         self.__create_path("test_1")
         self.assertIsNone(self.data_dir.get_file("test_1", ".value"))
 
+    def test_get_file_cannot_read(self) -> None:
+        self.__create_path("test_1", {".value": "value"})
+        (self.tmp_path / "test_1" / ".value").chmod(0o300)
+        self.assertIsNone(self.data_dir.get_file("test_1", ".value"))
+
     def test_get_file_invalid_path(self) -> None:
         self.assertIsNone(self.data_dir.get_file("test_1", ".value"))
 

tests/test_handlers.py

@@ -177,21 +177,6 @@ class TestRequestHandler(BaseHandlerTestCase):
                 token_manager=self.token_manager,
             )
 
-    def test_handle_disconnect_silently(self) -> None:
-        with self.patch("http.server.BaseHTTPRequestHandler.__init__") as mock:
-            mock.side_effect = BrokenPipeError
-            logging.basicConfig(level=logging.CRITICAL)
-            RequestHandler(
-                unittest.mock.MagicMock(),
-                "127.0.0.1",
-                unittest.mock.MagicMock(),
-                params=Parameters(
-                    data_dir=self.get_tmp_dir(), certbot_www=str(self.certbot_www)
-                ),
-                registry=self.registry,
-                token_manager=self.token_manager,
-            )
-
     def test_do_head_forward(self) -> None:
         handler = self._get_handler()
         with (

uv.lock

@@ -203,7 +203,7 @@ wheels = [
 
 [[package]]
 name = "stapler"
-version = "1.2.6"
+version = "1.2.4"
 source = { editable = "." }
 dependencies = [
     { name = "requests" },