Remove the use of sh in tests

Author: bbc2

.github/workflows/test.yml

@@ -2,8 +2,6 @@ name: Run Tests
 
 on:
   push:
-    branches:
-      - main
   pull_request:
 
 concurrency:

tests/test_zip_imports.py

@@ -1,15 +1,11 @@
 import os
+import subprocess
 import sys
 import textwrap
 from typing import List
 from unittest import mock
 from zipfile import ZipFile
 
-import pytest
-
-if sys.platform != "win32":
-    import sh
-
 
 def walk_to_root(path: str):
     last_dir = None
@@ -65,7 +61,6 @@ def test_load_dotenv_gracefully_handles_zip_imports_when_no_env_file(tmp_path):
     import child1.child2.test  # noqa
 
 
-@pytest.mark.skipif(sys.platform == "win32", 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,
@@ -83,7 +78,7 @@ def test_load_dotenv_outside_zip_file_when_called_in_zipfile(tmp_path):
         ],
     )
     dotenv_path = tmp_path / ".env"
-    dotenv_path.write_bytes(b"a=b")
+    dotenv_path.write_bytes(b"A=B")
     code_path = tmp_path / "code.py"
     code_path.write_text(
         textwrap.dedent(
@@ -95,12 +90,16 @@ def test_load_dotenv_outside_zip_file_when_called_in_zipfile(tmp_path):
 
         import child1.child2.test
 
-        print(os.environ['a'])
+        print(os.environ['A'])
     """
         )
     )
-    os.chdir(str(tmp_path))
 
-    result = sh.Command(sys.executable)(code_path)
+    result = subprocess.run(
+        [sys.executable, str(code_path)],
+        capture_output=True,
+        text=True,
+        cwd=tmp_path,
+    )
 
-    assert result == "b\n"
+    assert result.stdout == "B\n"