ci: run Go tests during build (#161)

* Enable build failures on lint failures, since the problems have
  now been resolved
* Install Go and run automated tests during build

Author: jawnsy

.github/workflows/build.yml

@@ -38,21 +38,23 @@ jobs:
 
       - name: Lint Helm Chart
         run: make lint/helm
-        # TODO(jawnsy): fix linter warnings and remove this
-        continue-on-error: true
 
       - name: Check formatting and docs
-        if: always()
         run: ./scripts/fmt.sh
 
       - name: Lint Shell Scripts
-        if: always()
         run: make lint/shellcheck
 
       - name: Lint Kubernetes Templates
-        if: always()
         run: make lint/kubernetes
 
+      - name: Install Go
+        uses: actions/setup-go@v2
+        with:
+          go-version: '^1.17'
+
+      - name: Unit tests
+        run: ./scripts/test_go.sh
+
       - name: Package Helm Chart
-        if: always()
         run: ./scripts/package.sh

scripts/test_go.sh

@@ -0,0 +1,60 @@
+#!/usr/bin/env bash
+#
+# Run Go-based unit tests
+
+set -o errexit
+set -o nounset
+set -o pipefail
+
+CI=${CI:-""}
+PROJECT_ROOT=$(git rev-parse --show-toplevel)
+# shellcheck source=lib.sh
+source "$PROJECT_ROOT/scripts/lib.sh"
+
+echo "--- Running go test"
+export FORCE_COLOR=true
+
+test_args=(
+  -v
+  -failfast
+  "${TEST_ARGS:-}"
+)
+
+REPORTDIR="/tmp/testreports"
+mkdir -p "$REPORTDIR"
+TESTREPORT_JSON="$REPORTDIR/test_go.json"
+TESTREPORT_XML="$REPORTDIR/test_go.xml"
+COVERAGE="$REPORTDIR/test_go.coverage"
+
+test_args+=(
+  "-covermode=set"
+  "-coverprofile=$COVERAGE"
+)
+
+declare test_status=0
+
+pushd "$PROJECT_ROOT/tests" >/dev/null 2>&1
+  # Allow failures to ensure that we can report on slow tests
+  set +o errexit
+
+  run_trace false gotestsum \
+    --debug \
+    --jsonfile="$TESTREPORT_JSON" \
+    --junitfile="$TESTREPORT_XML" \
+    --hide-summary=skipped \
+    --packages="./..." \
+    -- "${test_args[@]}"
+  test_status=$?
+
+  # Re-enable failures if steps fail
+  set -o errexit
+popd >/dev/null 2>&1
+
+# These unit tests should all be fast, so this report should be empty
+threshold="5s"
+echo "--- ಠ_ಠ The following tests took longer than $threshold to complete:"
+run_trace false gotestsum tool slowest \
+  --jsonfile="$TESTREPORT_JSON" \
+  --threshold="$threshold"
+
+exit "$test_status"