Skip to content
Snippets Groups Projects
Commit ef9ddcc7 authored by Thomas Woerner's avatar Thomas Woerner
Browse files

test_pwpolicy.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.
parent e546374f
No related branches found
No related tags found
No related merge requests found
...@@ -28,7 +28,7 @@ ...@@ -28,7 +28,7 @@
name: ops name: ops
state: present state: present
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Ensure presence of pwpolicies for group ops - name: Ensure presence of pwpolicies for group ops
ipapwpolicy: ipapwpolicy:
...@@ -44,7 +44,7 @@ ...@@ -44,7 +44,7 @@
maxfail: 3 maxfail: 3
failinterval: 5 failinterval: 5
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Ensure presence of pwpolicies for group ops again - name: Ensure presence of pwpolicies for group ops again
ipapwpolicy: ipapwpolicy:
...@@ -60,21 +60,21 @@ ...@@ -60,21 +60,21 @@
maxfail: 3 maxfail: 3
failinterval: 5 failinterval: 5
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Ensure maxlife of 49 for global_policy - name: Ensure maxlife of 49 for global_policy
ipapwpolicy: ipapwpolicy:
ipaadmin_password: SomeADMINpassword ipaadmin_password: SomeADMINpassword
maxlife: 49 maxlife: 49
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Ensure maxlife of 49 for global_policy again - name: Ensure maxlife of 49 for global_policy again
ipapwpolicy: ipapwpolicy:
ipaadmin_password: SomeADMINpassword ipaadmin_password: SomeADMINpassword
maxlife: 49 maxlife: 49
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
- name: Ensure absence of pwpoliciy global_policy will fail - name: Ensure absence of pwpoliciy global_policy will fail
ipapwpolicy: ipapwpolicy:
...@@ -82,7 +82,7 @@ ...@@ -82,7 +82,7 @@
state: absent state: absent
register: result register: result
ignore_errors: True ignore_errors: True
failed_when: result is defined and result failed_when: (result is defined and result) or result.failed
- name: Ensure absence of pwpolicies for group ops - name: Ensure absence of pwpolicies for group ops
ipapwpolicy: ipapwpolicy:
...@@ -90,14 +90,14 @@ ...@@ -90,14 +90,14 @@
name: ops name: ops
state: absent state: absent
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Ensure maxlife of 90 for global_policy - name: Ensure maxlife of 90 for global_policy
ipapwpolicy: ipapwpolicy:
ipaadmin_password: SomeADMINpassword ipaadmin_password: SomeADMINpassword
maxlife: 90 maxlife: 90
register: result register: result
failed_when: not result.changed failed_when: not result.changed or result.failed
- name: Ensure absence of pwpolicies for group ops - name: Ensure absence of pwpolicies for group ops
ipapwpolicy: ipapwpolicy:
...@@ -105,4 +105,4 @@ ...@@ -105,4 +105,4 @@
name: ops name: ops
state: absent state: absent
register: result register: result
failed_when: result.changed failed_when: result.changed or result.failed
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment