mirror of
https://github.com/freeipa/ansible-freeipa.git
synced 2026-07-30 03:14:42 +00:00
test_user_certmapdata.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:
@@ -28,7 +28,7 @@
|
|||||||
first: test
|
first: test
|
||||||
last: test
|
last: test
|
||||||
register: result
|
register: result
|
||||||
failed_when: not result.changed
|
failed_when: not result.changed or result.failed
|
||||||
|
|
||||||
- name: User test certmapdata members present
|
- name: User test certmapdata members present
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -40,7 +40,7 @@
|
|||||||
- certificate: "{{ lookup('file', 'cert3.b64', rstrip=False) }}"
|
- certificate: "{{ lookup('file', 'cert3.b64', rstrip=False) }}"
|
||||||
action: member
|
action: member
|
||||||
register: result
|
register: result
|
||||||
failed_when: not result.changed
|
failed_when: not result.changed or result.failed
|
||||||
|
|
||||||
- name: User test certmapdata members present again
|
- name: User test certmapdata members present again
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -52,7 +52,7 @@
|
|||||||
- certificate: "{{ lookup('file', 'cert3.b64', rstrip=False) }}"
|
- certificate: "{{ lookup('file', 'cert3.b64', rstrip=False) }}"
|
||||||
action: member
|
action: member
|
||||||
register: result
|
register: result
|
||||||
failed_when: result.changed
|
failed_when: result.changed or result.failed
|
||||||
|
|
||||||
- name: User test certmapdata members absent
|
- name: User test certmapdata members absent
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -65,7 +65,7 @@
|
|||||||
action: member
|
action: member
|
||||||
state: absent
|
state: absent
|
||||||
register: result
|
register: result
|
||||||
failed_when: not result.changed
|
failed_when: not result.changed or result.failed
|
||||||
|
|
||||||
- name: User test certmapdata members absent again
|
- name: User test certmapdata members absent again
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -78,7 +78,7 @@
|
|||||||
action: member
|
action: member
|
||||||
state: absent
|
state: absent
|
||||||
register: result
|
register: result
|
||||||
failed_when: result.changed
|
failed_when: result.changed or result.failed
|
||||||
|
|
||||||
- name: User test certmapdata members present
|
- name: User test certmapdata members present
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -93,7 +93,7 @@
|
|||||||
subject: CN=subject3
|
subject: CN=subject3
|
||||||
action: member
|
action: member
|
||||||
register: result
|
register: result
|
||||||
failed_when: not result.changed
|
failed_when: not result.changed or result.failed
|
||||||
|
|
||||||
- name: User test certmapdata members present again
|
- name: User test certmapdata members present again
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -108,7 +108,7 @@
|
|||||||
subject: CN=subject3
|
subject: CN=subject3
|
||||||
action: member
|
action: member
|
||||||
register: result
|
register: result
|
||||||
failed_when: result.changed
|
failed_when: result.changed or result.failed
|
||||||
|
|
||||||
- name: User test certmapdata members absent
|
- name: User test certmapdata members absent
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -122,7 +122,7 @@
|
|||||||
action: member
|
action: member
|
||||||
state: absent
|
state: absent
|
||||||
register: result
|
register: result
|
||||||
failed_when: not result.changed
|
failed_when: not result.changed or result.failed
|
||||||
|
|
||||||
- name: User test certmapdata members absent again
|
- name: User test certmapdata members absent again
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -136,7 +136,7 @@
|
|||||||
action: member
|
action: member
|
||||||
state: absent
|
state: absent
|
||||||
register: result
|
register: result
|
||||||
failed_when: result.changed
|
failed_when: result.changed or result.failed
|
||||||
|
|
||||||
- name: User test certmapdata members absent
|
- name: User test certmapdata members absent
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -148,7 +148,7 @@
|
|||||||
action: member
|
action: member
|
||||||
state: absent
|
state: absent
|
||||||
register: result
|
register: result
|
||||||
failed_when: not result.changed
|
failed_when: not result.changed or result.failed
|
||||||
|
|
||||||
- name: User test certmapdata members absent again
|
- name: User test certmapdata members absent again
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -160,7 +160,7 @@
|
|||||||
action: member
|
action: member
|
||||||
state: absent
|
state: absent
|
||||||
register: result
|
register: result
|
||||||
failed_when: result.changed
|
failed_when: result.changed or result.failed
|
||||||
|
|
||||||
- name: User test certmapdata member present
|
- name: User test certmapdata member present
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -171,7 +171,7 @@
|
|||||||
subject: CN=test,dc=example,dc=com
|
subject: CN=test,dc=example,dc=com
|
||||||
action: member
|
action: member
|
||||||
register: result
|
register: result
|
||||||
failed_when: not result.changed
|
failed_when: not result.changed or result.failed
|
||||||
|
|
||||||
- name: User test certmapdata member present again
|
- name: User test certmapdata member present again
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -182,7 +182,7 @@
|
|||||||
subject: CN=test,dc=example,dc=com
|
subject: CN=test,dc=example,dc=com
|
||||||
action: member
|
action: member
|
||||||
register: result
|
register: result
|
||||||
failed_when: result.changed
|
failed_when: result.changed or result.failed
|
||||||
|
|
||||||
- name: User test certmapdata member (data) present again
|
- name: User test certmapdata member (data) present again
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -192,7 +192,7 @@
|
|||||||
- data: X509:<I>dc=com,dc=example,CN=ca<S>dc=com,dc=example,CN=test
|
- data: X509:<I>dc=com,dc=example,CN=ca<S>dc=com,dc=example,CN=test
|
||||||
action: member
|
action: member
|
||||||
register: result
|
register: result
|
||||||
failed_when: result.changed
|
failed_when: result.changed or result.failed
|
||||||
|
|
||||||
- name: User test certmapdata member absent
|
- name: User test certmapdata member absent
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -204,7 +204,7 @@
|
|||||||
action: member
|
action: member
|
||||||
state: absent
|
state: absent
|
||||||
register: result
|
register: result
|
||||||
failed_when: not result.changed
|
failed_when: not result.changed or result.failed
|
||||||
|
|
||||||
- name: User test certmapdata member (data) absent again
|
- name: User test certmapdata member (data) absent again
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -215,7 +215,7 @@
|
|||||||
action: member
|
action: member
|
||||||
state: absent
|
state: absent
|
||||||
register: result
|
register: result
|
||||||
failed_when: result.changed
|
failed_when: result.changed or result.failed
|
||||||
|
|
||||||
- name: User test absent
|
- name: User test absent
|
||||||
ipauser:
|
ipauser:
|
||||||
@@ -223,7 +223,7 @@
|
|||||||
name: test
|
name: test
|
||||||
state: absent
|
state: absent
|
||||||
register: result
|
register: result
|
||||||
failed_when: not result.changed
|
failed_when: not result.changed or result.failed
|
||||||
|
|
||||||
- name: Remove certificate files.
|
- name: Remove certificate files.
|
||||||
shell:
|
shell:
|
||||||
|
|||||||
Reference in New Issue
Block a user