mirror of
https://github.com/freeipa/ansible-freeipa.git
synced 2026-08-01 12:24:43 +00:00
test_sudocmd.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:
@@ -21,7 +21,7 @@
|
|||||||
name: /usr/bin/su
|
name: /usr/bin/su
|
||||||
state: present
|
state: present
|
||||||
register: result
|
register: result
|
||||||
failed_when: not result.changed
|
failed_when: not result.changed or result.failed
|
||||||
|
|
||||||
- name: Ensure sudocmd is present again
|
- name: Ensure sudocmd is present again
|
||||||
ipasudocmd:
|
ipasudocmd:
|
||||||
@@ -29,7 +29,7 @@
|
|||||||
name: /usr/bin/su
|
name: /usr/bin/su
|
||||||
state: present
|
state: present
|
||||||
register: result
|
register: result
|
||||||
failed_when: result.changed
|
failed_when: result.changed or result.failed
|
||||||
|
|
||||||
- name: Ensure sudocmd is absent
|
- name: Ensure sudocmd is absent
|
||||||
ipasudocmd:
|
ipasudocmd:
|
||||||
@@ -37,7 +37,7 @@
|
|||||||
name: /usr/bin/su
|
name: /usr/bin/su
|
||||||
state: absent
|
state: absent
|
||||||
register: result
|
register: result
|
||||||
failed_when: not result.changed
|
failed_when: not result.changed or result.failed
|
||||||
|
|
||||||
- name: Ensure sudocmd is absent again
|
- name: Ensure sudocmd is absent again
|
||||||
ipasudocmd:
|
ipasudocmd:
|
||||||
@@ -45,7 +45,7 @@
|
|||||||
name: /usr/bin/su
|
name: /usr/bin/su
|
||||||
state: absent
|
state: absent
|
||||||
register: result
|
register: result
|
||||||
failed_when: result.changed
|
failed_when: result.changed or result.failed
|
||||||
|
|
||||||
- name: Ensure multiple sudocmd are present
|
- name: Ensure multiple sudocmd are present
|
||||||
ipasudocmd:
|
ipasudocmd:
|
||||||
@@ -55,7 +55,7 @@
|
|||||||
- /usr/sbin/iwlist
|
- /usr/sbin/iwlist
|
||||||
state: present
|
state: present
|
||||||
register: result
|
register: result
|
||||||
failed_when: not result.changed
|
failed_when: not result.changed or result.failed
|
||||||
|
|
||||||
- name: Ensure multiple sudocmd are present again
|
- name: Ensure multiple sudocmd are present again
|
||||||
ipasudocmd:
|
ipasudocmd:
|
||||||
@@ -65,7 +65,7 @@
|
|||||||
- /usr/sbin/iwlist
|
- /usr/sbin/iwlist
|
||||||
state: present
|
state: present
|
||||||
register: result
|
register: result
|
||||||
failed_when: result.changed
|
failed_when: result.changed or result.failed
|
||||||
|
|
||||||
- name: Ensure multiple sudocmd are absent
|
- name: Ensure multiple sudocmd are absent
|
||||||
ipasudocmd:
|
ipasudocmd:
|
||||||
@@ -75,7 +75,7 @@
|
|||||||
- /usr/sbin/iwlist
|
- /usr/sbin/iwlist
|
||||||
state: absent
|
state: absent
|
||||||
register: result
|
register: result
|
||||||
failed_when: not result.changed
|
failed_when: not result.changed or result.failed
|
||||||
|
|
||||||
- name: Ensure multiple sudocmd are absent again
|
- name: Ensure multiple sudocmd are absent again
|
||||||
ipasudocmd:
|
ipasudocmd:
|
||||||
@@ -85,7 +85,8 @@
|
|||||||
- /usr/sbin/iwlist
|
- /usr/sbin/iwlist
|
||||||
state: absent
|
state: absent
|
||||||
register: result
|
register: result
|
||||||
failed_when: result.changed
|
failed_when: result.changed or result.failed
|
||||||
|
|
||||||
- name: Ensure sudocmds are absent
|
- name: Ensure sudocmds are absent
|
||||||
ipasudocmd:
|
ipasudocmd:
|
||||||
ipaadmin_password: SomeADMINpassword
|
ipaadmin_password: SomeADMINpassword
|
||||||
@@ -117,4 +118,4 @@
|
|||||||
- /usr/sbin/iwlist
|
- /usr/sbin/iwlist
|
||||||
state: absent
|
state: absent
|
||||||
register: result
|
register: result
|
||||||
failed_when: not result.changed
|
failed_when: not result.changed or result.failed
|
||||||
|
|||||||
Reference in New Issue
Block a user