From 448bab607cf41d03cffd339bd4848bde905b0b3c Mon Sep 17 00:00:00 2001
From: Tre' Seymour <wayne.seymour@elastic.co>
Date: Mon, 15 Jun 2020 13:52:14 -0600
Subject: [PATCH] Drop catchError and use try / catch instead, to fail the
 build on test run and / or ingestion.

---
 .ci/Jenkinsfile_coverage | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/.ci/Jenkinsfile_coverage b/.ci/Jenkinsfile_coverage
index 650ef94e1d3da..3db7e3d8fddd6 100644
--- a/.ci/Jenkinsfile_coverage
+++ b/.ci/Jenkinsfile_coverage
@@ -11,11 +11,13 @@ kibanaPipeline(timeoutMinutes: 240) {
       'CODE_COVERAGE=1', // Enables coverage.  Needed for multiple ci scripts, such as remote.ts, test/scripts/*.sh, schema.js, etc.
     ]) {
       workers.base(name: 'coverage-worker', size: 'l', ramDisk: false, bootstrapped: false) {
-        catchError {
+        try {
           kibanaCoverage.runTests()
           handleIngestion(TIME_STAMP)
+        } catch (err) {
+          currentBuild.result = 'FAILURE'
+          handleFail()
         }
-        handleFail()
       }
     }
     kibanaPipeline.sendMail()