Browse Source

Merge pull request #5477 from vmarmol/add-testdirs

Add a TESTDIRS variable to the test bundle to control which directories are unit tested
Tianon Gravi 11 years ago
parent
commit
a96cac4d7d
2 changed files with 7 additions and 2 deletions
  1. 1 1
      Makefile
  2. 6 1
      hack/make/test

+ 1 - 1
Makefile

@@ -10,7 +10,7 @@ DOCKER_IMAGE := docker$(if $(GIT_BRANCH),:$(GIT_BRANCH))
 DOCKER_DOCS_IMAGE := docker-docs$(if $(GIT_BRANCH),:$(GIT_BRANCH))
 DOCKER_MOUNT := $(if $(BINDDIR),-v "$(CURDIR)/$(BINDDIR):/go/src/github.com/dotcloud/docker/$(BINDDIR)")
 
-DOCKER_RUN_DOCKER := docker run --rm -it --privileged -e TESTFLAGS -e DOCKER_GRAPHDRIVER -e DOCKER_EXECDRIVER $(DOCKER_MOUNT) "$(DOCKER_IMAGE)"
+DOCKER_RUN_DOCKER := docker run --rm -it --privileged -e TESTFLAGS -e TESTDIRS -e DOCKER_GRAPHDRIVER -e DOCKER_EXECDRIVER $(DOCKER_MOUNT) "$(DOCKER_IMAGE)"
 # to allow `make DOCSDIR=docs docs-shell`
 DOCKER_RUN_DOCS := docker run --rm -it $(if $(DOCSDIR),-v $(CURDIR)/$(DOCSDIR):/$(DOCSDIR)) -e AWS_S3_BUCKET
 

+ 6 - 1
hack/make/test

@@ -17,8 +17,13 @@ bundle_test() {
 	{
 		date
 
+		# Run all the tests if no TESTDIRS were specified.
+		if [ -z "$TESTDIRS" ]; then
+			TESTDIRS=$(find_dirs '*_test.go')
+		fi
+
 		TESTS_FAILED=()
-		for test_dir in $(find_dirs '*_test.go'); do
+		for test_dir in $TESTDIRS; do
 			echo
 
 			if ! LDFLAGS="$LDFLAGS $LDFLAGS_STATIC_DOCKER" go_test_dir "$test_dir"; then