From f1c697c29997c72952eefaa5d0ecc04b557d2c0d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Christian=20K=C3=BChnel?= <kuhnel@google.com>
Date: Wed, 6 May 2020 17:24:05 +0200
Subject: [PATCH] only run tests if build has passed

this relates to #176
---
 Jenkins/BETA-phabricator-linux/Jenkinsfile | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/Jenkins/BETA-phabricator-linux/Jenkinsfile b/Jenkins/BETA-phabricator-linux/Jenkinsfile
index 32301ba..eadfb31 100644
--- a/Jenkins/BETA-phabricator-linux/Jenkinsfile
+++ b/Jenkins/BETA-phabricator-linux/Jenkinsfile
@@ -110,12 +110,15 @@ pipeline {
                 dir("${LLVM_DIR}")
                 {
                     script {
-                        try {
-                            sh(script: "${SCRIPT_DIR}/run_ninja.py check-all")
-                        } catch (e) {
-                            success = false;
-                            failure_message += "\n'ninja check-all' failed" // append as build might already be broken
-                            echo e.toString()
+                        if (success) {
+                            // only run tests if build has passed, see #176
+                            try {
+                                sh(script: "${SCRIPT_DIR}/run_ninja.py check-all")
+                            } catch (e) {
+                                success = false;
+                                failure_message += "\n'ninja check-all' failed" // append as build might already be broken
+                                echo e.toString()
+                            }
                         }
                     }
                 }