From 47436177010e124fdb21c6a119ed877b91aacef9 Mon Sep 17 00:00:00 2001 From: JulesB40 Date: Tue, 27 Oct 2026 18:08:11 +0100 Subject: [PATCH] test(tools): skip the configure seed CMake round trip without Ninja The round trip configures with -G Ninja but only skipped when CMake was missing, so it errored on hosts without Ninja. Require both, like the other configure seed project tests. Co-Authored-By: Claude Opus 5.5 --- tools/tests/test_configure_seed.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/tests/test_configure_seed.py b/tools/tests/test_configure_seed.py index 373ce43764..d6f1b74c3d 100644 --- a/tools/tests/test_configure_seed.py +++ b/tools/tests/test_configure_seed.py @@ -312,7 +312,7 @@ class ChecksParsingTests(unittest.TestCase): "SINGLE": ("", "Test SINGLE"), }) - @unittest.skipUnless(shutil.which("cmake"), "CMake is required for cache round trips") + @unittest.skipUnless(shutil.which("cmake") and shutil.which("ninja"), "CMake and Ninja are required for cache round trips") def test_multiline_help_from_real_cmake_survives_seed_round_trip(self): with tempfile.TemporaryDirectory() as directory: root = Path(directory)