From b1edf574d74c8bc019beba46df5163da1f5ca91b Mon Sep 17 00:00:00 2001
From: Thomas Woerner <twoerner@redhat.com>
Date: Thu, 20 May 2021 13:16:43 +0200
Subject: [PATCH] test_users_certificate.yml: Use result.failed also for
failed_when
For failed_when result.failed should be used to make sure that
the task fails if there was an error.
---
tests/user/certificate/test_users_certificate.yml | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/tests/user/certificate/test_users_certificate.yml b/tests/user/certificate/test_users_certificate.yml
index 78198016..406e6030 100644
--- a/tests/user/certificate/test_users_certificate.yml
+++ b/tests/user/certificate/test_users_certificate.yml
@@ -41,7 +41,7 @@
- "{{ lookup('file', 'cert3.b64', rstrip=False) }}"
action: member
register: result
- failed_when: not result.changed
+ failed_when: not result.changed or result.failed
- name: User test cert members present again
ipauser:
@@ -54,7 +54,7 @@
- "{{ lookup('file', 'cert3.b64', rstrip=False) }}"
action: member
register: result
- failed_when: result.changed
+ failed_when: result.changed or result.failed
- name: User test cert members absent
ipauser:
@@ -68,7 +68,7 @@
state: absent
action: member
#register: result
- #failed_when: not result.changed
+ #failed_when: not result.changed or result.failed
- name: User test cert members absent again
ipauser:
@@ -82,7 +82,7 @@
state: absent
action: member
register: result
- failed_when: result.changed
+ failed_when: result.changed or result.failed
- name: User test absent
ipauser:
@@ -91,7 +91,7 @@
- name: test
state: absent
register: result
- failed_when: not result.changed
+ failed_when: not result.changed or result.failed
- name: Remove certificate files.
shell:
--
GitLab