Drop catchError and use try / catch instead,

to fail the build on test run and / or ingestion.
elastic · Jun 15, 2020 · 448bab6 · 448bab6
1 parent 52c8e96
commit 448bab6
Showing 1 changed file with 4 additions and 2 deletions.
diff --git 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()