Merge pull request #95 from OnePunMan/bug/go-unit-test-false-positive

Fix go-unit-test script falsely succeeding if go list ./... fails
dnephin · Sep 29, 2022 · fb24a63 · fb24a63
2 parents 74f6978 + 2f9ab51
commit fb24a63
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/run-go-unit-tests.sh b/run-go-unit-tests.sh
@@ -1,10 +1,9 @@
 #!/usr/bin/env bash
-FILES=$(go list ./... | grep -v /vendor/)
 
-go test -tags=unit -timeout 30s -short -v ${FILES}
-
-returncode=$?
-if [ $returncode -ne 0 ]; then
+fail() {
  echo "unit tests failed"
  exit 1
-fi
+}
+
+FILES=$(go list ./... | grep -v /vendor/) || fail
+go test -tags=unit -timeout 30s -short -v ${FILES} || fail