Merge pull request #22 from jingxu97/prow

Enable unit test

Author: k8s-ci-robot

.prow.sh

@@ -7,6 +7,4 @@
 
  . release-tools/prow.sh
 
-# TODO: Once we add windows build rules and a Makefile, we can enable
-# some basic unit testing
-# main
+main

Makefile

@@ -14,8 +14,9 @@ BUILD_TOOLS_DIR = $(BUILD_DIR)/tools
 
 GO_ENV_VARS = GO111MODULE=on GOOS=windows
 
+# TODO: temporarily disable generate and lint because they are not working.
 .PHONY: all
-all: generate compile lint
+all: compile test
 
 .PHONY: compile
 compile: compile-client compile-server compile-csi-proxy-api-gen
@@ -74,6 +75,14 @@ lint: $(GOLANGCI_LINT)
 	$(GO_ENV_VARS) $(GOLANGCI_LINT) run
 	git --no-pager diff --exit-code
 
+.PHONY: test-go
+test: test-go
+test-go:
+	@ echo; echo "### $@:"
+	# TODO: After issue https://github.com/microsoft/go-winio/pull/169 is resolved, remove the filter on the test path.
+	GO111MODULE=on go test `find ./internal/server/ -type d -not -name server`;\
+	cd client && GO111MODULE=on go test `go list ./... | grep -v group` && cd ../
+
 # see https://github.com/golangci/golangci-lint#binary-release
 $(GOLANGCI_LINT):
 	curl -sfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b "$$(dirname '$(GOLANGCI_LINT)')" '$(GOLANGCI_LINT_VERSION)'

release-tools/prow.sh

@@ -242,7 +242,7 @@ configvar CSI_PROW_DEP_VERSION v0.5.1 "golang dep version to be used for vendor
 # thus only makes sense in repos which provide their own CSI
 # driver. Repos can enable sanity testing by setting
 # CSI_PROW_TESTS_SANITY=sanity.
-configvar CSI_PROW_TESTS "unit parallel serial parallel-alpha serial-alpha sanity" "tests to run"
+#configvar CSI_PROW_TESTS "unit parallel serial parallel-alpha serial-alpha sanity" "tests to run"
 tests_enabled () {
     local t1 t2
     # We want word-splitting here, so ignore: Quote to prevent word splitting, or split robustly with mapfile or read -a.
@@ -960,7 +960,8 @@ main () {
             fi
         fi
         # Required for E2E testing.
-        run_with_go "${CSI_PROW_GO_VERSION_BUILD}" make container || die "'make container' failed"
+        #Cannot build windows container on Linux node
+        #run_with_go "${CSI_PROW_GO_VERSION_BUILD}" make container || die "'make container' failed"
     fi
 
     if tests_need_kind; then