ci: coverage report on test failure (#43)

* ci: coverage report on test failure

* ci(perf): add mod and build caching

* ci: fix continue on error

* ci: remove dont cover and mod based cache key

* ci: handle failed test

* ci: remove test envs

.github/workflows/test.yml

@@ -12,29 +12,31 @@ jobs:
   test-coverage-upload:
     runs-on: ubuntu-latest
     steps:
-
       - uses: actions/setup-go@v3
         with:
-          go-version: 1.16
+          go-version: 1.17
 
       - uses: actions/checkout@v3
 
-      - run: make test
-
-      - name: filter out DONTCOVER
-        if: github.event_name == 'pull_request'
+      - uses: actions/cache@v3
+        with:
+          path: |
+            ~/.cache/go-build
+            ~/go/pkg/mod
+          key: ${{ runner.os }}-go-${{ hashFiles('**/go.mod') }}
+          restore-keys: |
+            ${{ runner.os }}-go-
+
+      - name: Test
         run: |
-          excludelist="$(find ./ -type f -name '*.go' | xargs grep -l 'DONTCOVER')"
-          excludelist+=" $(find ./ -type f -name '*.pb.go')"
-          excludelist+=" $(find ./ -type f -path './tests/mocks/*.go')"
-          for filename in ${excludelist}; do
-            filename=$(echo $filename | sed 's/^./github.com\/cosmos\/cosmos-sdk/g')
-            echo "Excluding ${filename} from coverage report..."
-            sed -i "/$(echo $filename | sed 's/\//\\\//g')/d" ./coverage.out
-          done
+          set +e -x
+          make test
+          TEST_FAILED=false
+          [[ $? == 0 ]] && echo "TEST_FAILED=true" >> $GITHUB_ENV
+        id: test_status
 
       - name: Convert coverage to lcov
-        if: github.evnte_name == 'pull_request'
+        if: github.event_name == 'pull_request'
         uses: jandelgado/gcov2lcov-action@v1
 
       - name: Code Coverage Report
@@ -43,3 +45,8 @@ jobs:
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}
           lcov-file: coverage.lcov
+
+      - name: Fail CI if test was failed
+        if: ${{ env.TEST_FAILED }}
+        run: |
+          false

Makefile

@@ -44,6 +44,8 @@ distclean:
 .PHONY: distclean
 
 ### Testing
+TEST_TARGET ?= ./...
+TEST_ARGS ?= -timeout 12m -race -coverprofile=./coverage.out -covermode=atomic -v
 
 test: test-unit
 test-all: test-unit test-ledger-mock test-race test-cover
@@ -55,7 +57,7 @@ test-ledger: test-ledger-mock
 	@go test -mod=readonly -v `go list github.com/cosmos/cosmos-sdk/crypto` -tags='cgo ledger'
 
 test-unit:
-	@VERSION=$(VERSION) go test -mod=readonly $(PACKAGES_NOSIMULATION) -tags='ledger test_ledger_mock'
+	@VERSION=$(VERSION) go test -mod=readonly -tags='ledger test_ledger_mock' $(PACKAGES_NOSIMULATION) $(TEST_TARGET) $(TEST_ARGS)
 
 test-race:
 	@VERSION=$(VERSION) go test -mod=readonly -race $(PACKAGES_NOSIMULATION)
@@ -150,4 +152,4 @@ lintci-update: lintci-remove lintci-install
 .PHONY: lintci-update
 
 goimports:
-	goimports -local="github.com/AssetMantle/modules" -w .
+	goimports -local="github.com/AssetMantle/modules" -w .