Skip to content
This repository was archived by the owner on Jul 18, 2025. It is now read-only.

Commit d616bbb

Browse files
author
Ian Campbell
committed
ci: use gotestsum for e2e coverage tests too
The `Jenkinsfile*` already contain a `junit` rune which will pick these up. Signed-off-by: Ian Campbell <ijc@docker.com>
1 parent 273e433 commit d616bbb

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

Makefile

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,8 @@ lint: ## run linter(s)
7777

7878
test-e2e: bin/$(BIN_NAME) ## run end-to-end tests
7979
@echo "Running e2e tests..."
80-
$(GO_TEST) -v ./e2e/
80+
@$(call mkdir,$(TEST_RESULTS_DIR))
81+
$(call GO_TESTSUM,e2e.xml) -v ./e2e/
8182

8283
test-unit: ## run unit tests
8384
@echo "Running unit tests..."
@@ -96,7 +97,8 @@ coverage-test-unit:
9697
coverage-test-e2e: coverage-bin
9798
@echo "Running e2e tests (coverage)..."
9899
@$(call mkdir,_build/cov)
99-
DOCKERAPP_BINARY=../e2e/coverage-bin $(GO_TEST) -v ./e2e
100+
@$(call mkdir,$(TEST_RESULTS_DIR))
101+
DOCKERAPP_BINARY=../e2e/coverage-bin $(call GO_TESTSUM,e2e-coverage.xml) -v ./e2e
100102

101103
coverage: coverage-test-unit coverage-test-e2e ## run tests with coverage
102104
go install ./vendor/github.com/wadey/gocovmerge/

0 commit comments

Comments
 (0)