From 94afce76d34b1007c0e354cf1d024e0466e044b3 Mon Sep 17 00:00:00 2001
From: Bill Havanki <bill.havanki@segment.com>
Date: Tue, 14 May 2024 15:03:10 -0400
Subject: [PATCH] build: Add fmt and coverage make targets

Running fmt becomes a prerequisite for building chamber, helping to keep
the code formatted.

The coverage target runs basic code coverage tests, and sets the stage
for reporting.
---
 .gitignore |  2 ++
 Makefile   | 11 +++++++++--
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/.gitignore b/.gitignore
index 602691a9..bf94b7c5 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,5 @@
 *~
 *.sw[a-z]
 dist/
+/chamber
+/coverage.out
diff --git a/Makefile b/Makefile
index 744a84ac..065882dc 100644
--- a/Makefile
+++ b/Makefile
@@ -24,6 +24,10 @@ SRC := $(shell find . -name '*.go')
 test: store/awsapi_mock.go
 	go test -v ./...
 
+.PHONY: coverage
+coverage:
+	go test -coverpkg ./... -coverprofile coverage.out ./...
+
 store/awsapi_mock.go: store/awsapi.go
 ifdef MOQ
 	rm -f $@
@@ -41,9 +45,12 @@ clean:
 dist/:
 	mkdir -p dist
 
+fmt:
+	go fmt ./...
+
 build: chamber
 
-chamber: $(SRC)
+chamber: fmt $(SRC)
 	CGO_ENABLED=0 go build -trimpath $(LDFLAGS) -o $@
 
 dist/chamber-$(VERSION)-darwin-amd64: | dist/
@@ -64,4 +71,4 @@ dist/chamber-$(VERSION)-linux-arm64 dist/chamber-$(VERSION)-linux-aarch64: | dis
 dist/chamber-$(VERSION)-windows-amd64.exe: | dist/
 	GOOS=windows GOARCH=amd64 CGO_ENABLED=0 go build -trimpath $(LDFLAGS) -o $@
 
-.PHONY: clean all linux
+.PHONY: clean all fmt build linux