From 23312f256672de757c744a48e838a2c4a2aff5c5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tor=20Arne=20Vestb=C3=B8?= <tor.arne.vestbo@qt.io>
Date: Mon, 6 Jan 2025 15:17:38 +0100
Subject: [PATCH] Make qtbase and all other modules significant on macOS 15 by
 default

Unless a module overrides it by setting the environment variable
COIN_CTEST_FORCE_IGNORE_EXIT_CODE.

As this change trickles through submodule updates we'll mark modules
as COIN_CTEST_FORCE_IGNORE_EXIT_CODE if needed.

Change-Id: Ia37d6e4c4dee41a3dd7446fa23ab96593dab9402
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
---
 .../cmake_regular_test_instructions.yaml      | 26 +++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/coin/instructions/cmake_regular_test_instructions.yaml b/coin/instructions/cmake_regular_test_instructions.yaml
index b24b892ea13..91593df6b4f 100644
--- a/coin/instructions/cmake_regular_test_instructions.yaml
+++ b/coin/instructions/cmake_regular_test_instructions.yaml
@@ -16,4 +16,30 @@ instructions:
   - type: ChangeDirectory
     directory: "{{.SourceDir}}_standalone_tests"
 
+  # For configurations with granular test significance,
+  # make the tests significant by default, unless a module
+  # explicitly overrides it.
+  - type: Group
+    enable_if:
+      condition: and
+      conditions:
+        - condition: property
+          property: features
+          not_contains_value: "InsignificantTests"
+        - condition: runtime
+          env_var: COIN_CTEST_IGNORE_EXIT_CODE
+          equals_value: "1"
+        - condition: or
+          conditions:
+            - condition: property
+              property: id
+              contains_value: "macos-15"
+        - condition: runtime
+          env_var: COIN_CTEST_FORCE_IGNORE_EXIT_CODE
+          equals_value: null
+    instructions:
+      - type: EnvironmentVariable
+        variableName: COIN_CTEST_IGNORE_EXIT_CODE
+        variableValue: "0"
+
   - !include "{{qt/qtbase}}/cmake_run_ctest.yaml"