Add Windows testing to CI

Author: bbc2

.github/workflows/test.yml

@@ -14,6 +14,12 @@ jobs:
           - ubuntu-latest
         python-version:
           ["3.9", "3.10", "3.11", "3.12", "3.13", "3.14", pypy3.9, pypy3.10]
+        include:
+          # Test lowest and highest Python versions on Windows
+          - os: windows-latest
+            python-version: "3.9"
+          - os: windows-latest
+            python-version: "3.14"
 
     steps:
       - uses: actions/checkout@v6

tests/test_cli.py

@@ -1,14 +1,17 @@
 import os
+import sys
 from pathlib import Path
 from typing import Optional
 
 import pytest
-import sh
 
 import dotenv
 from dotenv.cli import cli as dotenv_cli
 from dotenv.version import __version__
 
+if not sys.platform.startswith("win"):
+    import sh
+
 
 @pytest.mark.parametrize(
     "output_format,content,expected",
@@ -173,6 +176,9 @@ def test_set_no_file(cli):
     assert "Missing argument" in result.output
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="sh module doesn't support Windows"
+)
 def test_get_default_path(tmp_path):
     with sh.pushd(tmp_path):
         (tmp_path / ".env").write_text("a=b")
@@ -182,6 +188,9 @@ def test_get_default_path(tmp_path):
         assert result == "b\n"
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="sh module doesn't support Windows"
+)
 def test_run(tmp_path):
     with sh.pushd(tmp_path):
         (tmp_path / ".env").write_text("a=b")
@@ -191,6 +200,9 @@ def test_run(tmp_path):
         assert result == "b\n"
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="sh module doesn't support Windows"
+)
 def test_run_with_existing_variable(tmp_path):
     with sh.pushd(tmp_path):
         (tmp_path / ".env").write_text("a=b")
@@ -202,6 +214,9 @@ def test_run_with_existing_variable(tmp_path):
         assert result == "b\n"
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="sh module doesn't support Windows"
+)
 def test_run_with_existing_variable_not_overridden(tmp_path):
     with sh.pushd(tmp_path):
         (tmp_path / ".env").write_text("a=b")
@@ -213,6 +228,9 @@ def test_run_with_existing_variable_not_overridden(tmp_path):
         assert result == "c\n"
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="sh module doesn't support Windows"
+)
 def test_run_with_none_value(tmp_path):
     with sh.pushd(tmp_path):
         (tmp_path / ".env").write_text("a=b\nc")
@@ -222,6 +240,9 @@ def test_run_with_none_value(tmp_path):
         assert result == "b\n"
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="sh module doesn't support Windows"
+)
 def test_run_with_other_env(dotenv_path):
     dotenv_path.write_text("a=b")
 

tests/test_ipython.py

@@ -1,11 +1,15 @@
 import os
+import sys
 from unittest import mock
 
 import pytest
 
 pytest.importorskip("IPython")
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="This test assumes case-sensitive variables"
+)
 @mock.patch.dict(os.environ, {}, clear=True)
 def test_ipython_existing_variable_no_override(tmp_path):
     from IPython.terminal.embed import InteractiveShellEmbed
@@ -22,6 +26,9 @@ def test_ipython_existing_variable_no_override(tmp_path):
     assert os.environ == {"a": "c"}
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="This test assumes case-sensitive variables"
+)
 @mock.patch.dict(os.environ, {}, clear=True)
 def test_ipython_existing_variable_override(tmp_path):
     from IPython.terminal.embed import InteractiveShellEmbed
@@ -38,6 +45,9 @@ def test_ipython_existing_variable_override(tmp_path):
     assert os.environ == {"a": "b"}
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="This test assumes case-sensitive variables"
+)
 @mock.patch.dict(os.environ, {}, clear=True)
 def test_ipython_new_variable(tmp_path):
     from IPython.terminal.embed import InteractiveShellEmbed

tests/test_main.py

@@ -1,15 +1,18 @@
 import io
 import logging
 import os
+import stat
 import sys
 import textwrap
 from unittest import mock
 
 import pytest
-import sh
 
 import dotenv
 
+if not sys.platform.startswith("win"):
+    import sh
+
 
 def test_set_key_no_file(tmp_path):
     nx_path = tmp_path / "nx"
@@ -62,15 +65,25 @@ def test_set_key_encoding(dotenv_path):
 
 
 @pytest.mark.skipif(
-    os.geteuid() == 0, reason="Root user can access files even with 000 permissions."
+    not sys.platform.startswith("win") and os.geteuid() == 0,
+    reason="Root user can access files even with 000 permissions.",
 )
 def test_set_key_permission_error(dotenv_path):
-    dotenv_path.chmod(0o000)
+    if sys.platform.startswith("win"):
+        # On Windows, make file read-only
+        dotenv_path.chmod(stat.S_IREAD)
+    else:
+        # On Unix, remove all permissions
+        dotenv_path.chmod(0o000)
 
     with pytest.raises(PermissionError):
         dotenv.set_key(dotenv_path, "a", "b")
 
-    dotenv_path.chmod(0o600)
+    # Restore permissions
+    if sys.platform.startswith("win"):
+        dotenv_path.chmod(stat.S_IWRITE | stat.S_IREAD)
+    else:
+        dotenv_path.chmod(0o600)
     assert dotenv_path.read_text() == ""
 
 
@@ -170,16 +183,6 @@ def test_unset_encoding(dotenv_path):
     assert dotenv_path.read_text(encoding=encoding) == ""
 
 
-@pytest.mark.skipif(
-    os.geteuid() == 0, reason="Root user can access files even with 000 permissions."
-)
-def test_set_key_unauthorized_file(dotenv_path):
-    dotenv_path.chmod(0o000)
-
-    with pytest.raises(PermissionError):
-        dotenv.set_key(dotenv_path, "a", "x")
-
-
 def test_unset_non_existent_file(tmp_path):
     nx_path = tmp_path / "nx"
     logger = logging.getLogger("dotenv.main")
@@ -312,6 +315,9 @@ def test_load_dotenv_disabled_notification(dotenv_path, flag_value):
         )
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="This test assumes case-sensitive variables"
+)
 @pytest.mark.parametrize(
     "flag_value",
     [
@@ -395,6 +401,9 @@ def test_load_dotenv_no_file_verbose():
     )
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="This test assumes case-sensitive variables"
+)
 @mock.patch.dict(os.environ, {"a": "c"}, clear=True)
 def test_load_dotenv_existing_variable_no_override(dotenv_path):
     dotenv_path.write_text("a=b")
@@ -405,6 +414,9 @@ def test_load_dotenv_existing_variable_no_override(dotenv_path):
     assert os.environ == {"a": "c"}
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="This test assumes case-sensitive variables"
+)
 @mock.patch.dict(os.environ, {"a": "c"}, clear=True)
 def test_load_dotenv_existing_variable_override(dotenv_path):
     dotenv_path.write_text("a=b")
@@ -415,6 +427,9 @@ def test_load_dotenv_existing_variable_override(dotenv_path):
     assert os.environ == {"a": "b"}
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="This test assumes case-sensitive variables"
+)
 @mock.patch.dict(os.environ, {"a": "c"}, clear=True)
 def test_load_dotenv_redefine_var_used_in_file_no_override(dotenv_path):
     dotenv_path.write_text('a=b\nd="${a}"')
@@ -425,6 +440,9 @@ def test_load_dotenv_redefine_var_used_in_file_no_override(dotenv_path):
     assert os.environ == {"a": "c", "d": "c"}
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="This test assumes case-sensitive variables"
+)
 @mock.patch.dict(os.environ, {"a": "c"}, clear=True)
 def test_load_dotenv_redefine_var_used_in_file_with_override(dotenv_path):
     dotenv_path.write_text('a=b\nd="${a}"')
@@ -435,6 +453,9 @@ def test_load_dotenv_redefine_var_used_in_file_with_override(dotenv_path):
     assert os.environ == {"a": "b", "d": "b"}
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="This test assumes case-sensitive variables"
+)
 @mock.patch.dict(os.environ, {}, clear=True)
 def test_load_dotenv_string_io_utf_8():
     stream = io.StringIO("a=à")
@@ -456,6 +477,9 @@ def test_load_dotenv_file_stream(dotenv_path):
     assert os.environ == {"a": "b"}
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="sh module doesn't support Windows"
+)
 def test_load_dotenv_in_current_dir(tmp_path):
     dotenv_path = tmp_path / ".env"
     dotenv_path.write_bytes(b"a=b")
@@ -484,6 +508,9 @@ def test_dotenv_values_file(dotenv_path):
     assert result == {"a": "b"}
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="This test assumes case-sensitive variables"
+)
 @pytest.mark.parametrize(
     "env,string,interpolate,expected",
     [

tests/test_zip_imports.py

@@ -5,7 +5,10 @@
 from unittest import mock
 from zipfile import ZipFile
 
-import sh
+import pytest
+
+if not sys.platform.startswith("win"):
+    import sh
 
 
 def walk_to_root(path: str):
@@ -62,6 +65,9 @@ def test_load_dotenv_gracefully_handles_zip_imports_when_no_env_file(tmp_path):
     import child1.child2.test  # noqa
 
 
+@pytest.mark.skipif(
+    sys.platform.startswith("win"), reason="sh module doesn't support Windows"
+)
 def test_load_dotenv_outside_zip_file_when_called_in_zipfile(tmp_path):
     zip_file_path = setup_zipfile(
         tmp_path,