Fix exception with 'skipped' test (#4406)

Co-authored-by: MARGERIE Gilles <[email protected]>
ansible · Nov 20, 2024 · d06911b · d06911b
1 parent d58af09
commit d06911b
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/examples/playbooks/rule-jinja-pass.yml b/examples/playbooks/rule-jinja-pass.yml
@@ -88,3 +88,18 @@
     - name: Bug https://github.com/ansible/ansible-lint/issues/3908
       ansible.builtin.debug:
         msg: "{{ foo | ansible.builtin.mandatory(msg='My message') }}"
+
+- name: Bugs https://github.com/ansible/ansible-lint/issues/4338
+  hosts: localhost
+  tasks:
+    - name: Test task
+      ansible.builtin.debug:
+        msg: "Hello, world!"
+      when: false
+      register: result
+
+    - name: Test task 2
+      vars:
+        counter: "{{ result is skipped | ternary(1, 2) }}"
+      ansible.builtin.debug:
+        msg: "Hello, world {{ counter }}!"
diff --git a/src/ansiblelint/rules/jinja.py b/src/ansiblelint/rules/jinja.py
@@ -65,6 +65,8 @@ class Token(NamedTuple):
             r"Unrecognized type <<class 'ansible.template.AnsibleUndefined'>> for (.*) filter <value>$",
             # https://github.com/ansible/ansible-lint/issues/3155
             r"^The '(.*)' test expects a dictionary$",
+            # https://github.com/ansible/ansible-lint/issues/4338
+            r"An unhandled exception occurred while templating (.*). Error was a <class 'ansible.errors.AnsibleFilterError'>, original message: The (.*) test expects a dictionary$",
         ],
     ),
     flags=re.MULTILINE | re.DOTALL,