From b681b58e8ab21cef87aa025a9b9ae4336ba2a41a Mon Sep 17 00:00:00 2001 From: Kenta Tada Date: Wed, 11 Jul 2018 17:50:47 +0900 Subject: [PATCH] Fix the problem TESTFLAGS is not to be used in Makefile correctly This commit modifies Makefile to handle test targets correctly. Signed-off-by: Kenta Tada --- Makefile | 10 +++++----- tests/integration/README.md | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/Makefile b/Makefile index d490a3f03..c392bd534 100644 --- a/Makefile +++ b/Makefile @@ -66,25 +66,25 @@ localtest: make localunittest localintegration localrootlessintegration unittest: runcimage - docker run -e TESTFLAGS -t --privileged --rm -v /lib/modules:/lib/modules:ro -v $(CURDIR):/go/src/$(PROJECT) $(RUNC_IMAGE) make localunittest + docker run -t --privileged --rm -v /lib/modules:/lib/modules:ro -v $(CURDIR):/go/src/$(PROJECT) $(RUNC_IMAGE) make localunittest TESTFLAGS=${TESTFLAGS} localunittest: all $(GO) test -timeout 3m -tags "$(BUILDTAGS)" ${TESTFLAGS} -v $(allpackages) integration: runcimage - docker run -e TESTFLAGS -t --privileged --rm -v /lib/modules:/lib/modules:ro -v $(CURDIR):/go/src/$(PROJECT) $(RUNC_IMAGE) make localintegration + docker run -t --privileged --rm -v /lib/modules:/lib/modules:ro -v $(CURDIR):/go/src/$(PROJECT) $(RUNC_IMAGE) make localintegration TESTPATH=${TESTPATH} localintegration: all - bats -t tests/integration${TESTFLAGS} + bats -t tests/integration${TESTPATH} rootlessintegration: runcimage - docker run -e TESTFLAGS -t --privileged --rm -v $(CURDIR):/go/src/$(PROJECT) $(RUNC_IMAGE) make localrootlessintegration + docker run -t --privileged --rm -v $(CURDIR):/go/src/$(PROJECT) $(RUNC_IMAGE) make localrootlessintegration localrootlessintegration: all tests/rootless.sh shell: runcimage - docker run -e TESTFLAGS -ti --privileged --rm -v $(CURDIR):/go/src/$(PROJECT) $(RUNC_IMAGE) bash + docker run -ti --privileged --rm -v $(CURDIR):/go/src/$(PROJECT) $(RUNC_IMAGE) bash install: install -D -m0755 runc $(BINDIR)/runc diff --git a/tests/integration/README.md b/tests/integration/README.md index c58788af8..8ee6ebf28 100644 --- a/tests/integration/README.md +++ b/tests/integration/README.md @@ -27,7 +27,7 @@ $ sudo bats tests/integration ``` To run a single test bucket: ``` -$ make integration TESTFLAGS="/checkpoint.bats" +$ make integration TESTPATH="/checkpoint.bats" ```