fix: apply fixes based on tests

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>

Author: henryiii

mesonpy/__init__.py

@@ -995,7 +995,7 @@ def get_requires_for_build_wheel(
     config_settings: Optional[Dict[str, str]] = None,
 ) -> List[str]:
     dependencies = [_depstr.wheel, *_get_ninja_if_needed()]
-    if sys.platform.startswith("linux"):
+    if sys.platform.startswith('linux'):
         # we may need patchelf
         if not shutil.which('patchelf'):
             # patchelf not already accessible on the system

tests/test_pep517.py

@@ -14,7 +14,7 @@
 
 
 @pytest.mark.parametrize('package', ['pure', 'library'])
-@pytest.mark.parametrize('system_patchelf', [True, False], ids=['patchelf', 'nopatchelf'])
+@pytest.mark.parametrize('system_patchelf', ['patchelf', None], ids=['patchelf', 'nopatchelf'])
 @pytest.mark.parametrize('ninja', [None, '1.8.1', '1.8.3'], ids=['noninja', 'oldninja', 'newninja'])
 def test_get_requires_for_build_wheel(monkeypatch, package, system_patchelf, ninja):
     def which(prog: str) -> bool:
@@ -37,7 +37,7 @@ def run(cmd: List[str], *args: object, **kwargs: object) -> subprocess.Completed
     monkeypatch.setattr(subprocess, 'run', run)
 
     expected = {mesonpy._depstr.wheel}
-    if system_patchelf and sys.platform.startswith('linux'):
+    if system_patchelf is None and sys.platform.startswith('linux'):
         expected |= {mesonpy._depstr.patchelf}
     if ninja is None or [int(x) for x in ninja.split('.')] < [1, 8, 2]:
         expected |= {mesonpy._depstr.ninja}

tests/test_pep518.py

@@ -17,11 +17,11 @@
 def test_pep518(pep518, virtual_env, package, build_args, tmp_path):
     dist = tmp_path / 'dist'
 
-    result = subprocess.run([virtual_env, '-m', 'pip', 'install', 'build'], check=False)
+    result = subprocess.run([str(virtual_env), '-m', 'pip', 'install', 'build'], check=False)
     assert result.returncode == 0
 
     with cd_package(package):
         result = subprocess.run(
-            [virtual_env, '-m', 'build', f'--outdir={dist}', *build_args], check=False
+            [str(virtual_env), '-m', 'build', f'--outdir={dist}', *build_args], check=False
         )
         assert result.returncode == 0