Merge "Attempt to copy the coverage report even if job fails"

roles/fetch-coverage-output/tasks/main.yaml

@@ -1,20 +1,19 @@
-- when: zuul_success | default(true) | bool
-  block:
-
-    - name: Check to see if coverage report exists
+- name: Check to see if coverage report exists
   stat:
     path: "{{ coverage_output_src }}"
     register: coverage_report_stat
 
-    - name: Collect coverage details output
+- when:
+    - zuul_success | default(true) | bool
+    - not coverage_report_stat.stat.exists
+  name: Fail if coverage job succeeded and report is not present
+  fail:
+    msg: "Coverage report was not found even though tests succeeded"
+
+- when: coverage_report_stat.stat.exists
+  name: Collect coverage details output
   synchronize:
     dest: "{{ zuul_executor_dest }}"
     mode: pull
     src: "{{ coverage_output_src }}"
     verify_host: true
-      when: coverage_report_stat.stat.exists
-
-    - name: Fail if coverage report not found
-      fail:
-        msg: "Coverage report was not found even though tests succeeded"
-      when: not coverage_report_stat.stat.exists