Update Tests with Fresh Py Environment

Author: Behemyth

tests/fixtures/cli.py

@@ -1,5 +1,8 @@
 """Fixtures for interfacing with the CLI."""
 
+import os
+import platform
+
 import pytest
 from typer.testing import CliRunner
 
@@ -13,3 +16,21 @@ def fixture_typer_runner() -> CliRunner:
     runner = CliRunner()
 
     return runner
+
+
+@pytest.fixture(
+    name='fresh_environment',
+    scope='session',
+)
+def fixture_fresh_environment() -> dict[str, str]:
+    """Create a fresh environment for subprocess calls."""
+    # Start with a minimal environment
+    new_env = {}
+
+    # Copy only variables you need
+    if platform.system() == 'Windows':
+        new_env['SystemRoot'] = os.environ['SystemRoot']  # noqa: SIM112
+
+    # Provide a PATH that doesn't contain venv references
+    new_env['PATH'] = os.environ['PATH']
+    return new_env

tests/integration/examples/test_conan_cmake.py

@@ -9,50 +9,44 @@
 
 from typer.testing import CliRunner
 
-from cppython.console.entry import app
-
 pytest_plugins = ['tests.fixtures.example']
 
 
 class TestConanCMake:
     """Test project variation of conan and CMake"""
 
     @staticmethod
-    def test_simple(example_runner: CliRunner) -> None:
+    def test_simple(example_runner: CliRunner, fresh_environment: dict[str, str]) -> None:
         """Simple project"""
-        result = example_runner.invoke(
-            app,
-            [
-                'install',
-            ],
-        )
+        # By nature of running the test, we require PDM to develop the project and so it will be installed
+        result = subprocess.run(['pdm', 'install'], capture_output=True, text=True, env=fresh_environment, check=False)
 
-        assert result.exit_code == 0, result.output
+        assert result.returncode == 0, f'PDM install failed: {result.stderr}'
 
         # Run the CMake configuration command
-        cmake_result = subprocess.run(['cmake', '--preset=default'], capture_output=True, text=True, check=False)
+        result = subprocess.run(
+            ['cmake', '--preset=default'], capture_output=True, text=True, env=fresh_environment, check=False
+        )
 
-        assert cmake_result.returncode == 0, f'CMake configuration failed: {cmake_result.stderr}'
+        assert result.returncode == 0, f'Cmake failed: {result.stderr}'
 
         # Verify that the build directory contains the expected files
         assert (Path('build') / 'CMakeCache.txt').exists(), 'build/CMakeCache.txt not found'
 
     @staticmethod
-    def test_inject(example_runner: CliRunner) -> None:
+    def test_inject(example_runner: CliRunner, fresh_environment: dict[str, str]) -> None:
         """Inject"""
-        result = example_runner.invoke(
-            app,
-            [
-                'install',
-            ],
-        )
+        # By nature of running the test, we require PDM to develop the project and so it will be installed
+        result = subprocess.run(['pdm', 'install'], capture_output=True, text=True, env=fresh_environment, check=False)
 
-        assert result.exit_code == 0, result.output
+        assert result.returncode == 0, f'PDM install failed: {result.stderr}'
 
         # Run the CMake configuration command
-        cmake_result = subprocess.run(['cmake', '--preset=default'], capture_output=True, text=True, check=False)
+        result = subprocess.run(
+            ['cmake', '--preset=default'], capture_output=True, text=True, env=fresh_environment, check=False
+        )
 
-        assert cmake_result.returncode == 0, f'CMake configuration failed: {cmake_result.stderr}'
+        assert result.returncode == 0, f'Cmake failed: {result.stderr}'
 
         # Verify that the build directory contains the expected files
         assert (Path('build') / 'CMakeCache.txt').exists(), 'build/CMakeCache.txt not found'

tests/integration/examples/test_vcpkg_cmake.py

@@ -10,8 +10,6 @@
 import pytest
 from typer.testing import CliRunner
 
-from cppython.console.entry import app
-
 pytest_plugins = ['tests.fixtures.example']
 
 
@@ -20,23 +18,19 @@ class TestVcpkgCMake:
 
     @staticmethod
     @pytest.mark.skip(reason='TODO')
-    def test_simple(example_runner: CliRunner) -> None:
+    def test_simple(example_runner: CliRunner, fresh_environment: dict[str, str]) -> None:
         """Simple project"""
-        result = example_runner.invoke(
-            app,
-            [
-                'install',
-            ],
-        )
+        # By nature of running the test, we require PDM to develop the project and so it will be installed
+        result = subprocess.run(['pdm', 'install'], capture_output=True, text=True, env=fresh_environment, check=False)
 
-        assert result.exit_code == 0, result.output
+        assert result.returncode == 0, f'PDM install failed: {result.stderr}'
 
         # Run the CMake configuration command
-        cmake_result = subprocess.run(
-            ['cmake', '--preset=default', '-B', 'build'], capture_output=True, text=True, check=False
+        result = subprocess.run(
+            ['cmake', '--preset=default'], capture_output=True, text=True, env=fresh_environment, check=False
         )
 
-        assert cmake_result.returncode == 0, f'CMake configuration failed: {cmake_result.stderr}'
+        assert result.returncode == 0, f'Cmake failed: {result.stderr}'
 
         # Verify that the build directory contains the expected files
         assert (Path('build') / 'CMakeCache.txt').exists(), 'build/CMakeCache.txt not found'