Skip to content

Commit

Permalink
fix: update Makefile test (#86)
Browse files Browse the repository at this point in the history
Signed-off-by: Junjie Gao <[email protected]>
  • Loading branch information
JeyJeyGao authored Mar 13, 2023
1 parent eccd2c5 commit 8406300
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 7 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/workflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ jobs:
- name: Check out code into the Go module directory
uses: actions/checkout@v3
- name: Run unit tests
run: go test -race -v -coverprofile=coverage.txt -covermode=atomic ./...
run: make test
- name: Build testing
run: make build
- name: Upload coverage reports to Codecov
Expand Down
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@

# Test binary, built with `go test -c`
*.test
coverage.txt

# Output of the go coverage tool, specifically when used with LiteIDE
*.out
Expand Down
9 changes: 3 additions & 6 deletions Makefile
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
# Notation Azure KeyVault Plugin Makefile for Linux
MODULE = github.com/Azure/notation-azure-kv
COMMANDS = notation-azure-kv
GIT_TAG = $(shell git describe --tags --abbrev=0 --exact-match 2>/dev/null)
Expand Down Expand Up @@ -30,13 +31,9 @@ download: ## download dependencies via go mod
.PHONY: build
build: $(addprefix bin/,$(COMMANDS)) ## builds binaries

.PHONY: clean
clean:
git status --ignored --short | grep '^!! ' | sed 's/!! //' | xargs rm -rf

.PHONY: test
test:
go test ./... -coverprofile cover.out
test: ## run unit test
go test -race -v -coverprofile=coverage.txt -covermode=atomic ./...

.PHONY: install
install: bin/notation-azure-kv ## installs the plugin
Expand Down

0 comments on commit 8406300

Please sign in to comment.