Skip to content

Commit c3997b6

Browse files
authored
Merge pull request kubernetes-sigs#497 from mihivagyok/measure-test-coverage
Enable coverage measurement
2 parents 0b0dc50 + b223112 commit c3997b6

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

.gitignore

+6
Original file line numberDiff line numberDiff line change
@@ -12,3 +12,9 @@
1212

1313
# macOS paraphernalia
1414
.DS_Store
15+
16+
# coverage files
17+
konnectivity.out
18+
konnectivity.html
19+
konnectivity-client/client.out
20+
konnectivity-client/client.html

Makefile

+3-3
Original file line numberDiff line numberDiff line change
@@ -57,8 +57,8 @@ mock_gen:
5757

5858
.PHONY: test
5959
test:
60-
go test -race ./...
61-
cd konnectivity-client && go test -race ./...
60+
go test -race -covermode=atomic -coverprofile=konnectivity.out ./... && go tool cover -html=konnectivity.out -o=konnectivity.html
61+
cd konnectivity-client && go test -race -covermode=atomic -coverprofile=client.out ./... && go tool cover -html=client.out -o=client.html
6262

6363
## --------------------------------------
6464
## Binaries
@@ -297,4 +297,4 @@ release-alias-tag: # Adds the tag to the last build tag. BASE_REF comes from the
297297
.PHONY: clean
298298
clean:
299299
go clean -testcache
300-
rm -rf proto/agent/agent.pb.go proto/agent/agent_grpc.pb.go konnectivity-client/proto/client/client.pb.go konnectivity-client/proto/client/client_grpc.pb.go konnectivity-client/proto/client/client_grpc.licensed.go proto/agent/agent_grpc.licensed.go easy-rsa.tar.gz easy-rsa cfssl cfssljson certs bin proto/agent/mocks
300+
rm -rf proto/agent/agent.pb.go proto/agent/agent_grpc.pb.go konnectivity-client/proto/client/client.pb.go konnectivity-client/proto/client/client_grpc.pb.go konnectivity-client/proto/client/client_grpc.licensed.go proto/agent/agent_grpc.licensed.go easy-rsa.tar.gz easy-rsa cfssl cfssljson certs bin proto/agent/mocks konnectivity.html konnectivity.out konnectivity-client/client.html konnectivity-client/client.out

0 commit comments

Comments
 (0)