From e546374f8ff889b308b3af86c2891d5753ef5ae9 Mon Sep 17 00:00:00 2001
From: Thomas Woerner <twoerner@redhat.com>
Date: Thu, 20 May 2021 13:16:31 +0200
Subject: [PATCH] test_hostgroup_rename.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/hostgroup/test_hostgroup_rename.yml | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tests/hostgroup/test_hostgroup_rename.yml b/tests/hostgroup/test_hostgroup_rename.yml
index 98007175..d3dd33b2 100644
--- a/tests/hostgroup/test_hostgroup_rename.yml
+++ b/tests/hostgroup/test_hostgroup_rename.yml
@@ -24,7 +24,7 @@
name: databases
state: present
register: result
- failed_when: not result.changed
+ failed_when: not result.changed or result.failed
- name: Rename host-group from `databases` to `datalake`
ipahostgroup:
@@ -33,7 +33,7 @@
rename: datalake
state: renamed
register: result
- failed_when: not result.changed
+ failed_when: not result.changed or result.failed
- name: Ensure host-group database was already absent
ipahostgroup:
@@ -41,7 +41,7 @@
name: database
state: absent
register: result
- failed_when: result.changed
+ failed_when: result.changed or result.failed
- name: Rename host-group from `databases` to `datalake`, again
ipahostgroup:
@@ -59,7 +59,7 @@
rename: datalake
state: renamed
register: result
- failed_when: result.changed
+ failed_when: result.changed or result.failed
- name: Ensure testing hostgroups do not exist.
ipahostgroup:
--
GitLab