test_dnszone.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.
This commit is contained in:
Thomas Woerner
2021-05-20 13:16:12 +02:00
parent 3d436677a5
commit 8eaa362732

View File

@@ -17,7 +17,7 @@
name: testzone.local name: testzone.local
state: present state: present
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Ensure zone is present, again. - name: Ensure zone is present, again.
ipadnszone: ipadnszone:
@@ -25,7 +25,7 @@
name: testzone.local name: testzone.local
state: present state: present
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Ensure zone is disabled. - name: Ensure zone is disabled.
ipadnszone: ipadnszone:
@@ -33,7 +33,7 @@
name: testzone.local name: testzone.local
state: disabled state: disabled
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Ensure zone is disabled, again. - name: Ensure zone is disabled, again.
ipadnszone: ipadnszone:
@@ -41,7 +41,7 @@
name: testzone.local name: testzone.local
state: disabled state: disabled
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Ensure zone is enabled. - name: Ensure zone is enabled.
ipadnszone: ipadnszone:
@@ -49,7 +49,7 @@
name: testzone.local name: testzone.local
state: enabled state: enabled
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Ensure zone is enabled, again. - name: Ensure zone is enabled, again.
ipadnszone: ipadnszone:
@@ -57,7 +57,7 @@
name: testzone.local name: testzone.local
state: enabled state: enabled
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Ensure forward_policy is none. - name: Ensure forward_policy is none.
ipadnszone: ipadnszone:
@@ -65,7 +65,7 @@
name: testzone.local name: testzone.local
forward_policy: none forward_policy: none
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Ensure forward_policy is none, again. - name: Ensure forward_policy is none, again.
ipadnszone: ipadnszone:
@@ -73,7 +73,7 @@
name: testzone.local name: testzone.local
forward_policy: none forward_policy: none
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Ensure forward_policy is first. - name: Ensure forward_policy is first.
ipadnszone: ipadnszone:
@@ -81,7 +81,7 @@
name: testzone.local name: testzone.local
forward_policy: first forward_policy: first
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Ensure forward_policy is first, again. - name: Ensure forward_policy is first, again.
ipadnszone: ipadnszone:
@@ -89,7 +89,7 @@
name: testzone.local name: testzone.local
forward_policy: first forward_policy: first
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Ensure first forwarder is set. - name: Ensure first forwarder is set.
ipadnszone: ipadnszone:
@@ -99,7 +99,7 @@
- ip_address: 8.8.8.8 - ip_address: 8.8.8.8
port: 53 port: 53
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Ensure first and second forwarder are set. - name: Ensure first and second forwarder are set.
ipadnszone: ipadnszone:
@@ -110,7 +110,7 @@
port: 53 port: 53
- ip_address: 2001:4860:4860::8888 - ip_address: 2001:4860:4860::8888
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Ensure first and second forwarder are set, again. - name: Ensure first and second forwarder are set, again.
ipadnszone: ipadnszone:
@@ -121,7 +121,7 @@
port: 53 port: 53
- ip_address: 2001:4860:4860::8888 - ip_address: 2001:4860:4860::8888
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Ensure only second forwarder is set. - name: Ensure only second forwarder is set.
ipadnszone: ipadnszone:
@@ -130,14 +130,14 @@
forwarders: forwarders:
- ip_address: 2001:4860:4860::8888 - ip_address: 2001:4860:4860::8888
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Nothing changes. - name: Nothing changes.
ipadnszone: ipadnszone:
ipaadmin_password: SomeADMINpassword ipaadmin_password: SomeADMINpassword
name: testzone.local name: testzone.local
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Ensure no forwarders are set. - name: Ensure no forwarders are set.
ipadnszone: ipadnszone:
@@ -145,7 +145,7 @@
name: testzone.local name: testzone.local
forwarders: [] forwarders: []
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Create zones test1 - name: Create zones test1
ipadnszone: ipadnszone:
@@ -198,7 +198,7 @@
- test3.testzone.local - test3.testzone.local
state: absent state: absent
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Ensure multiple zones are absent, again - name: Ensure multiple zones are absent, again
ipadnszone: ipadnszone:
@@ -209,7 +209,7 @@
- test3.testzone.local - test3.testzone.local
state: absent state: absent
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
# Teardown # Teardown
- name: Teardown testing environment - name: Teardown testing environment