Merge pull request #146 from fuweid/me-enable-root-for-testing

dmcgowan · web-flow · commit f65d91d395eb · 2019-11-26T16:54:31.000-08:00
test: enable root for RequiresRoot cases
diff --git a/.travis.yml b/.travis.yml
@@ -30,4 +30,5 @@ script:
   - ../project/script/validate/vendor
   - make build binaries
   - if [ "$GOOS" = "linux" ]; then make vet test; fi
+  - if [ "$GOOS" = "linux" ]; then sudo PATH=$PATH GOPATH=$GOPATH make root-test; fi
   - if [ "$GOOS" != "linux" ]; then make test-compile; fi
diff --git a/Makefile b/Makefile
@@ -20,7 +20,18 @@ VERSION=$(shell git describe --match 'v[0-9]*' --dirty='.m' --always)
 
 GO_LDFLAGS=-ldflags "-X `go list ./version`.Version=$(VERSION)"
 
+PKG=github.com/containerd/continuity
+
 PACKAGES=$(shell go list ./... | grep -v /vendor/)
+TEST_REQUIRES_ROOT_PACKAGES=$(filter \
+    ${PACKAGES}, \
+    $(shell \
+    for f in $$(git grep -l testutil.RequiresRoot | grep -v Makefile); do \
+        d="$$(dirname $$f)"; \
+        [ "$$d" = "." ] && echo "${PKG}" && continue; \
+        echo "${PKG}/$$d"; \
+    done | sort -u) \
+    )
 
 .PHONY: clean all fmt vet lint build test binaries setup
 .DEFAULT: default
@@ -68,6 +79,10 @@ test:
 	@echo "+ $@"
 	@go test $(PACKAGES)
 
+root-test:
+	@echo "+ $@"
+	@go test ${TEST_REQUIRES_ROOT_PACKAGES} -test.root
+
 test-compile:
 	@echo "+ $@"
 	@for pkg in $(PACKAGES); do go test -c $$pkg; done
