From dd0d02b7654abb63fdc698c495f65f161beea5db Mon Sep 17 00:00:00 2001 From: Thomas Woerner <twoerner@redhat.com> Date: Thu, 20 May 2021 13:16:44 +0200 Subject: [PATCH] test_user_certmapdata_issuer_subject.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. --- .../test_user_certmapdata_issuer_subject.yml | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/user/certmapdata/test_user_certmapdata_issuer_subject.yml b/tests/user/certmapdata/test_user_certmapdata_issuer_subject.yml index 0309c6af..2d323b49 100644 --- a/tests/user/certmapdata/test_user_certmapdata_issuer_subject.yml +++ b/tests/user/certmapdata/test_user_certmapdata_issuer_subject.yml @@ -18,7 +18,7 @@ first: test last: test register: result - failed_when: not result.changed + failed_when: not result.changed or result.failed - name: User test certmapdata members present ipauser: @@ -33,7 +33,7 @@ subject: CN=subject3 action: member register: result - failed_when: not result.changed + failed_when: not result.changed or result.failed - name: User test certmapdata members present again ipauser: @@ -48,7 +48,7 @@ subject: CN=subject3 action: member register: result - failed_when: result.changed + failed_when: result.changed or result.failed - name: User test certmapdata members absent ipauser: @@ -64,7 +64,7 @@ action: member state: absent register: result - failed_when: not result.changed + failed_when: not result.changed or result.failed - name: User test certmapdata members absent again ipauser: @@ -80,7 +80,7 @@ action: member state: absent register: result - failed_when: result.changed + failed_when: result.changed or result.failed - name: User test absent ipauser: @@ -88,4 +88,4 @@ name: test state: absent register: result - failed_when: not result.changed + failed_when: not result.changed or result.failed -- GitLab