tests: rework skips to show uv vs. build

Signed-off-by: Henry Schreiner <[email protected]>

Author: henryiii

test/conftest.py

@@ -160,13 +160,21 @@ def build_frontend_env_nouv(request: pytest.FixtureRequest) -> dict[str, str]:
     return {"CIBW_BUILD_FRONTEND": frontend}
 
 
-@pytest.fixture
-def build_frontend_env(build_frontend_env_nouv: dict[str, str]) -> dict[str, str]:
-    frontend = build_frontend_env_nouv["CIBW_BUILD_FRONTEND"]
-    if frontend != "build" or get_platform() == "pyodide" or find_uv() is None:
-        return build_frontend_env_nouv
+@pytest.fixture(params=["pip", "build", "build[uv]"])
+def build_frontend_env(request: pytest.FixtureRequest) -> dict[str, str]:
+    frontend = request.param
+    platform = get_platform()
+    if platform in {"pyodide", "ios", "android"} and frontend == "pip":
+        pytest.skip("Can't use pip as build frontend for pyodide/ios/android platform")
+    if platform == "pyodide" and frontend == "build[uv]":
+        pytest.skip("Can't use uv with pyodide yet")
+    uv_path = find_uv()
+    if uv_path is None and frontend == "build[uv]":
+        pytest.skip("Can't find uv, so skipping uv tests")
+    if uv_path is not None and frontend == "build" and platform not in {"android", "ios"}:
+        pytest.skip("No need to check build when uv is present")
 
-    return {"CIBW_BUILD_FRONTEND": "build[uv]"}
+    return {"CIBW_BUILD_FRONTEND": frontend}
 
 
 @pytest.fixture

test/test_android.py

@@ -103,12 +103,11 @@ def test_expected_wheels(tmp_path):
     )
 
 
-@pytest.mark.parametrize("frontend", ["build[uv]", "pip"])
-def test_frontend_good(tmp_path, frontend):
+def test_frontend_good(tmp_path, build_frontend_env):
     new_c_project().generate(tmp_path)
     wheels = cibuildwheel_run(
         tmp_path,
-        add_env={**cp313_env, "CIBW_BUILD_FRONTEND": frontend},
+        add_env={**cp313_env, **build_frontend_env},
     )
     assert wheels == [f"spam-0.1.0-cp313-cp313-android_21_{native_arch.android_abi}.whl"]