[buildstream] branch tristan/bst-1/properly-fail-tests created (now c856f47)

[tv...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

tvb pushed a change to branch tristan/bst-1/properly-fail-tests
in repository https://gitbox.apache.org/repos/asf/buildstream.git.


      at c856f47  .github/run-ci.sh: Fix test suite to properly fail in CI

This branch includes the following new commits:

     new c856f47  .github/run-ci.sh: Fix test suite to properly fail in CI

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

[buildstream] 01/01: .github/run-ci.sh: Fix test suite to properly fail in CI

[tv...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

tvb pushed a commit to branch tristan/bst-1/properly-fail-tests
in repository https://gitbox.apache.org/repos/asf/buildstream.git

commit c856f47ddb131c6ef778d15092fefe75261dae2a
Author: Tristan van Berkom <tr...@codethink.co.uk>
AuthorDate: Thu Feb 24 13:46:54 2022 +0900

    .github/run-ci.sh: Fix test suite to properly fail in CI
    
    Silly typo was causing tests to pass even when they fail.
---
 .github/run-ci.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/run-ci.sh b/.github/run-ci.sh
index 9a26ef9..6820a2c 100755
--- a/.github/run-ci.sh
+++ b/.github/run-ci.sh
@@ -62,7 +62,7 @@ function runTest() {
 
 # Lazily ensure that the script exits when a command fails
 #
-set -x
+set -e
 
 if [ -z "${test_names}" ]; then
     runTest "lint"