From defd6d2e08ee434dbd62c66ec865937acb343daf Mon Sep 17 00:00:00 2001
From: Thomas Woerner <twoerner@redhat.com>
Date: Thu, 20 May 2021 13:16:18 +0200
Subject: [PATCH] test_hbacsvc.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/hbacsvc/test_hbacsvc.yml | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/tests/hbacsvc/test_hbacsvc.yml b/tests/hbacsvc/test_hbacsvc.yml
index 87a6bbd5..c91cf39b 100644
--- a/tests/hbacsvc/test_hbacsvc.yml
+++ b/tests/hbacsvc/test_hbacsvc.yml
@@ -16,14 +16,14 @@
ipaadmin_password: SomeADMINpassword
name: http
register: result
- failed_when: not result.changed
+ failed_when: not result.changed or result.failed
- name: Ensure HBAC Service for http is present again
ipahbacsvc:
ipaadmin_password: SomeADMINpassword
name: http
register: result
- failed_when: result.changed
+ failed_when: result.changed or result.failed
- name: Ensure HBAC Service for tftp is present
ipahbacsvc:
@@ -31,7 +31,7 @@
name: tftp
description: TFTP service
register: result
- failed_when: not result.changed
+ failed_when: not result.changed or result.failed
- name: Ensure HBAC Service for tftp is present again
ipahbacsvc:
@@ -39,7 +39,7 @@
name: tftp
description: TFTP service
register: result
- failed_when: result.changed
+ failed_when: result.changed or result.failed
- name: Ensure HBAC Services for http and tftp are absent
ipahbacsvc:
@@ -47,7 +47,7 @@
name: http,tftp
state: absent
register: result
- failed_when: not result.changed
+ failed_when: not result.changed or result.failed
- name: Ensure HBAC Services for http and tftp are absent again
ipahbacsvc:
@@ -55,4 +55,4 @@
name: http,tftp
state: absent
register: result
- failed_when: result.changed
+ failed_when: result.changed or result.failed
--
GitLab