ci: run tests properly

- integration_test.go should only run with -tags=integration
- make: simplify "go test" call
- make: rename "test-integration" task just "integration"
- make: add "test" task to run all non-integration tests
- make: add "help" task to identify/describe tasks on CLI

Author: dmke

.github/workflows/tests.yaml

@@ -22,7 +22,7 @@ jobs:
           key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}
 
       - name: Run tests
-        run: make test-integration
+        run: make test
 
       - uses: codecov/codecov-action@v2
         with:

Makefile

@@ -1,14 +1,23 @@
-FILES_TO_FMT      ?= $(shell find . -path ./vendor -prune -o -name '*.go' -print)
+FILES_TO_FMT ?= $(shell find . -path ./vendor -prune -o -name '*.go' -print)
+
+.PHONY: help
+help: ## Print a short help message
+	@grep -hE '^[a-zA-Z_-]+:[^:]*?## .*$$' $(MAKEFILE_LIST) | \
+		awk 'BEGIN {FS = ":.*?## "}; {printf "\033[36m%-18s\033[0m %s\n", $$1, $$2}'
 
-## format: Formats Go code
 .PHONY: format
-format:
+format: ## Formats all Go code
 	@echo ">> formatting code"
 	@gofmt -s -w $(FILES_TO_FMT)
 
-## test-integration: Run all Go integration tests
-.PHONY: test-integration
-test-integration:
-	echo 'mode: atomic' > coverage.out
-	go list ./... | xargs -I{} sh -c 'go test -race -tags=integration -covermode=atomic -coverprofile=coverage.tmp -coverpkg $(go list ./... | tr "\n" ",") {} && tail -n +2 coverage.tmp >> coverage.out || exit 255'
-	rm coverage.tmp
+.PHONY: lint
+lint: ## Runs golangci-lint on source files
+	golangci-lint run
+
+.PHONY: test
+test: ## Run all Go tests (excluding integration tests)
+	go test -race -covermode=atomic -coverprofile=coverage.out ./...
+
+.PHONY: integration
+integration: ## Run Go tests (integration tests only)
+	go test -race -tags=integration -covermode=atomic -coverprofile=coverage.out ./...

integration_test.go

@@ -1,5 +1,5 @@
-//go:build !integration
-// +build !integration
+//go:build integration
+// +build integration
 
 package at