Responses to code review from @freakboy3742

Author: joerick

test/test_ios.py

@@ -171,7 +171,10 @@ def test_no_xbuild_tool_definition(tmp_path, capfd):
 
     # The expected wheels were produced.
     expected_wheels = utils.expected_wheels(
-        "spam", "0.1.0", platform="ios", python_abi_tags=["cp313-cp313"]
+        "spam",
+        "0.1.0",
+        platform="ios",
+        python_abi_tags=["cp313-cp313"],
     )
     assert set(actual_wheels) == set(expected_wheels)
 
@@ -214,6 +217,7 @@ def test_empty_xbuild_tool_definition(tmp_path, capfd):
     assert "Your project configuration does not define any cross-build tools." not in captured.err
 
 
+@pytest.mark.serial
 def test_ios_test_command_without_python_dash_m(tmp_path, capfd):
     """pytest should be able to run without python -m, but it should warn."""
     if utils.platform != "macos":

test/utils.py

@@ -157,8 +157,8 @@ def expected_wheels(
     package_version: str,
     manylinux_versions: list[str] | None = None,
     musllinux_versions: list[str] | None = None,
-    macosx_deployment_target: str = "10.9",
-    iphoneos_deployment_target: str = "13.0",
+    macosx_deployment_target: str | None = None,
+    iphoneos_deployment_target: str | None = None,
     machine_arch: str | None = None,
     platform: str = platform,
     python_abi_tags: list[str] | None = None,
@@ -174,6 +174,12 @@ def expected_wheels(
         if platform == "linux":
             machine_arch = arch_name_for_linux(machine_arch)
 
+    if macosx_deployment_target is None:
+        macosx_deployment_target = os.environ.get("MACOSX_DEPLOYMENT_TARGET", "10.9")
+
+    if iphoneos_deployment_target is None:
+        iphoneos_deployment_target = os.environ.get("IPHONEOS_DEPLOYMENT_TARGET", "13.0")
+
     architectures = [machine_arch]
     if not single_arch:
         if platform == "linux":