Skip to content

Commit 3868f48

Browse files
committed
Include cmd/options in Makefile's test
1. Why is this change necessary ? Solves #163 2. How does this change address the issue ? Unit testing for node-problem-detector/cmd/options is yet to be done and the corresponding package path needs to be included in makefile test 3. How to verify this change ? make test command should run the test TestSetNodeNameOrDie with admin privileges. Signed-off-by: gkGaneshR <[email protected]>
1 parent 25b6c16 commit 3868f48

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ Dockerfile: Dockerfile.in
8787
sed -e 's|@BASEIMAGE@|$(BASEIMAGE)|g' $< >$@
8888

8989
test: vet fmt
90-
go test -timeout=1m -v -race ./pkg/... $(BUILD_TAGS)
90+
go test -timeout=1m -v -race ./cmd/options ./pkg/... $(BUILD_TAGS)
9191

9292
build-container: ./bin/node-problem-detector Dockerfile
9393
docker build -t $(IMAGE) .

0 commit comments

Comments
 (0)