Merge pull request #551 from Gekko0114/Makefile_args

k8s-ci-robot · web-flow · commit e5a5dbd145e8 · 2023-03-23T00:26:23.000-07:00
Add integration test args to Makefile
diff --git a/Makefile b/Makefile
@@ -120,7 +120,7 @@ install-envtest:
 
 .PHONY: integration-test
 integration-test: install-envtest
-	$(INTEGTESTENVVAR) hack/integration-test.sh
+	$(INTEGTESTENVVAR) hack/integration-test.sh $(ARGS)
 
 .PHONY: verify
 verify:
diff --git a/hack/integration-test.sh b/hack/integration-test.sh
@@ -32,8 +32,8 @@ runTests() {
   source "${TEMP_DIR}/setup-envtest"
 
   kube::log::status "Running integration test cases"
-  # TODO: make args customizable.
-  go test -timeout=40m -mod=vendor sigs.k8s.io/scheduler-plugins/test/integration/...
+
+  go test -timeout=40m -mod=vendor sigs.k8s.io/scheduler-plugins/test/integration/... ${ARGS:-}
 }
 
 trap cleanup EXIT

Original file line number	Diff line number	Diff line change
`@@ -32,8 +32,8 @@ runTests() {`
`32`	`32`	`source "${TEMP_DIR}/setup-envtest"`
`33`	`33`
`34`	`34`	`kube::log::status "Running integration test cases"`
`35`		`- # TODO: make args customizable.`
`36`		`- go test -timeout=40m -mod=vendor sigs.k8s.io/scheduler-plugins/test/integration/...`
	`35`	`+`
	`36`	`+ go test -timeout=40m -mod=vendor sigs.k8s.io/scheduler-plugins/test/integration/... ${ARGS:-}`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`trap cleanup EXIT`