Merge lp:~gz/juju-ci-tools/run_tests_race into lp:juju-ci-tools

[Martin Packman]

Add --race flag to run-unit-test script

Will run the unit tests with go's -race flag, which we expect to make some additional tests fail.

There is one behaviour change here for previous cases, passing both --safe and --retry will use -p 2 for both test runs.

[Curtis Hovey (sinzui)]

Thank you.

review: Approve (code)

=== modified file 'run-unit-tests'
--- run-unit-tests	2015-05-19 15:23:38 +0000
+++ run-unit-tests	2015-05-26 15:35:19 +0000
@@ -9,10 +9,10 @@
 FIXUP_SOURCES=""
 IS_LOCAL="false"
 RETRY="false"
-SAFE="false"
 REVISION_BUILD=${revision_build:-}
 USE_PPA=""
 USE_TMPFS="false"
+GOTESTCMD="go test"
 LOG_FILE_PATH="/usr/share/distro-info/ubuntu.csv"
 LOG_TXT="17.04,Angsty Antelope,angsty,2016-10-23,2017-04-30,2018-01-29"
 
@@ -25,7 +25,7 @@
             RETRY="true"
             ;;
         --safe)
-            SAFE="true"
+            GOTESTCMD="$GOTESTCMD -p 2"
             ;;
         --local)
             IS_LOCAL="true"
@@ -47,6 +47,9 @@
         --use-tmpfs)
             USE_TMPFS="true"
             ;;
+        --race)
+            GOTESTCMD="$GOTESTCMD -race"
+            ;;
     esac
     shift
 done
@@ -160,9 +163,9 @@
 fi
 
 if [[ "$RETRY" == "true" ]]; then
-    go test ./... || go test -p 2 ./...
-elif [[  "$SAFE" == "true"  ]]; then
-    go test -p 2 ./...
+    $GOTESTCMD ./... || $GOTESTCMD ./...
+elif [[  "$GOTESTCMD" != "go test"  ]]; then
+    $GOTESTCMD ./...
 else
     make check
 fi