test: this test no longer needs a mock

nedbat · nedbat · commit 04ee0cac1025 · 2025-12-16T07:44:53.000-05:00
diff --git a/tests/test_sqlitedb.py b/tests/test_sqlitedb.py
@@ -5,12 +5,10 @@
 
 from __future__ import annotations
 
-from typing import NoReturn
 from unittest import mock
 
 import pytest
 
-import coverage.sqlitedb
 from coverage.exceptions import DataError
 from coverage.sqlitedb import SqliteDb
 
@@ -77,16 +75,10 @@ def test_retry_executemany_void_failure(self) -> None:
 
     def test_open_fails_on_bad_db(self) -> None:
         self.make_file("bad.db", "boogers")
-
-        def fake_failing_open(filename: str, mode: str) -> NoReturn:
-            assert (filename, mode) == ("bad.db", "rb")
-            raise RuntimeError("No you can't!")
-
-        with mock.patch.object(coverage.sqlitedb, "open", fake_failing_open):
-            msg = "Couldn't use data file 'bad.db': file is not a database"
-            with pytest.raises(DataError, match=msg):
-                with SqliteDb("bad.db", DebugControlString(options=["sql"])):
-                    pass  # pragma: not covered
+        msg = "Couldn't use data file 'bad.db': file is not a database"
+        with pytest.raises(DataError, match=msg):
+            with SqliteDb("bad.db", DebugControlString(options=["sql"])):
+                pass  # pragma: not covered
 
     def test_execute_void_can_allow_failure(self) -> None:
         with SqliteDb("fail.db", DebugControlString(options=["sql"])) as db:
