From 69925b2e1c73921b61c77880e1a2108143dd3080 Mon Sep 17 00:00:00 2001
From: Adam Reese <adam@reese.io>
Date: Wed, 22 Apr 2020 15:46:48 -0700
Subject: [PATCH] ref(scripts/coverage): simplify test coverage with coverpkg

Signed-off-by: Adam Reese <adam@reese.io>
---
 scripts/coverage.sh | 40 +++++++++++++++++-----------------------
 1 file changed, 17 insertions(+), 23 deletions(-)

diff --git a/scripts/coverage.sh b/scripts/coverage.sh
index bdbfaa991..7695f19b9 100755
--- a/scripts/coverage.sh
+++ b/scripts/coverage.sh
@@ -14,6 +14,17 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+# coverage.sh - Generate test coverage analysis
+# 
+# Coverage report is sent to coveralls.io if circleci is building the master
+# branch.
+#
+# Usage:
+#   coverage.sh [option]
+#
+# Options:
+#   --html         generate html coverage report
+
 set -euo pipefail
 
 covermode=${COVERMODE:-atomic}
@@ -24,31 +35,14 @@ pushd /
 hash goveralls 2>/dev/null || go get github.com/mattn/goveralls
 popd
 
-generate_cover_data() {
-  for d in $(go list ./...) ; do
-    (
-      local output="${coverdir}/${d//\//-}.cover"
-      go test -coverprofile="${output}" -covermode="$covermode" "$d"
-    )
-  done
+go test -cpu 4 -coverprofile="${profile}" -covermode="$covermode" -coverpkg=./... ./...
 
-  echo "mode: $covermode" >"$profile"
-  grep -h -v "^mode:" "$coverdir"/*.cover >>"$profile"
-}
+go tool cover -func "${profile}"
 
-push_to_coveralls() {
+if [[ "${CIRCLE_BRANCH-}" == 'master' ]]; then
   goveralls -coverprofile="${profile}" -service=circle-ci
-}
+fi
 
-generate_cover_data
-go tool cover -func "${profile}"
-
-case "${1-}" in
-  --html)
+if [[ "${1-}" == '--html' ]]; then
     go tool cover -html "${profile}"
-    ;;
-  --coveralls)
-    push_to_coveralls
-    ;;
-esac
-
+fi