From dc0bb555c1c03090296d5195fb9d3124154b2bb0 Mon Sep 17 00:00:00 2001
From: Kevin Veen-Birkenbach <kevin@veen.world>
Date: Mon, 18 Aug 2025 21:37:07 +0200
Subject: [PATCH] Added another group_names validation

---
 tasks/stages/01_constructor.yml                | 8 ++++++++
 tests/integration/test_variable_definitions.py | 2 +-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/tasks/stages/01_constructor.yml b/tasks/stages/01_constructor.yml
index f060b28b..ef789381 100644
--- a/tasks/stages/01_constructor.yml
+++ b/tasks/stages/01_constructor.yml
@@ -21,6 +21,14 @@
       applications: "{{ defaults_applications | merge_with_defaults(applications | default({}, true)) }}"
     no_log: "{{ MASK_CREDENTIALS_IN_LOGS | bool }}"
 
+  - name: "Sanity check: all group_names must exist in applications"
+    assert:
+      that:
+        - item in applications
+      fail_msg: "Group '{{ item }}' has no entry in 'applications'"
+      success_msg: "Group '{{ item }}' is defined in 'applications'"
+    loop: "{{ group_names }}"
+
   - name: Merge current play applications
     set_fact:
       current_play_applications: >-
diff --git a/tests/integration/test_variable_definitions.py b/tests/integration/test_variable_definitions.py
index f48afcd0..a37dfaa9 100644
--- a/tests/integration/test_variable_definitions.py
+++ b/tests/integration/test_variable_definitions.py
@@ -218,7 +218,7 @@ class TestVariableDefinitions(unittest.TestCase):
                                 if var in (
                                     'lookup', 'role_name', 'domains', 'item', 'host_type',
                                     'inventory_hostname', 'role_path', 'playbook_dir',
-                                    'ansible_become_password', 'inventory_dir', 'ansible_memtotal_mb', 'omit'
+                                    'ansible_become_password', 'inventory_dir', 'ansible_memtotal_mb', 'omit', 'group_names'
                                 ):
                                     continue