From 34d23c617c89861e0d4ee1aad6a0acf1892502df Mon Sep 17 00:00:00 2001
From: Massimiliano Culpo <massimiliano.culpo@googlemail.com>
Date: Sun, 1 Jan 2017 22:05:33 +0100
Subject: [PATCH] tests: `coverage combine` run only if tests succeeded (#2712)

---
 share/spack/qa/run-unit-tests | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/share/spack/qa/run-unit-tests b/share/spack/qa/run-unit-tests
index 9cace0dfd6..d2ce9647af 100755
--- a/share/spack/qa/run-unit-tests
+++ b/share/spack/qa/run-unit-tests
@@ -44,8 +44,7 @@ spack config get compilers
 # Run unit tests with code coverage
 if [[ "$TRAVIS_PYTHON_VERSION" == 2.7 ]]; then
     coverage run bin/spack install -v libdwarf
-    coverage run bin/spack test "$@"
-    coverage combine
+    coverage run bin/spack test "$@" && coverage combine
 else
     spack install -v libdwarf
     spack test "$@"
-- 
GitLab