Skip to content

Commit

Permalink
tests: enable both ginkgo and automated tests
Browse files Browse the repository at this point in the history
The `test` target in makefile doesn't run automated tests
if a ginkgo test suite is provided by a module. Fix the target
to always run all the tests provided by a module.
  • Loading branch information
dciabrin committed Oct 6, 2023
1 parent e942c68 commit 96be2cf
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -53,9 +53,8 @@ test: gowork generate fmt vet envtest ginkgo ## Run tests.
pushd ./$$mod ; \
if [ -f test/functional/suite_test.go ]; then \
KUBEBUILDER_ASSETS="$(shell $(ENVTEST) -v debug --bin-dir $(LOCALBIN) use $(ENVTEST_K8S_VERSION) -p path)" $(GINKGO) --trace --cover --coverprofile cover.out --covermode=atomic ${PROC_CMD} $(GINKGO_ARGS) ./test/... || exit 1; \
else \
KUBEBUILDER_ASSETS="$(shell $(ENVTEST) use $(ENVTEST_K8S_VERSION) -p path)" go test ./... -coverprofile cover.out || exit 1; \
fi; \
KUBEBUILDER_ASSETS="$(shell $(ENVTEST) use $(ENVTEST_K8S_VERSION) -p path)" go test ./... -coverprofile cover.out || exit 1; \
popd ; \
done
##@ Build
Expand Down

0 comments on commit 96be2cf

Please sign in to comment.