Alternate test for failure path in rewrite that does not require us to ignore type checking.

Author: theGOTOguy

tests/test_main.py

@@ -22,11 +22,6 @@ def test_set_key_no_file(tmp_path):
     assert os.path.exists(nx_file)
 
 
-def test_set_key_invalid_file():
-    with pytest.raises(TypeError):
-        dotenv.set_key(None, "foo", "bar")  # type: ignore
-
-
 @pytest.mark.parametrize(
     "before,key,value,expected,after",
     [
@@ -183,6 +178,13 @@ def test_unset_encoding(dotenv_file):
         assert f.read() == ""
 
 
+def test_set_key_unauthorized_file(dotenv_file):
+    os.chmod(dotenv_file, 0o000)
+
+    with pytest.raises(PermissionError):
+        dotenv.set_key(dotenv_file, "a", "x")
+
+
 def test_unset_non_existent_file(tmp_path):
     nx_file = str(tmp_path / "nx")
     logger = logging.getLogger("dotenv.main")