Skip to content

Commit 30cb1cc

Browse files
alexluongclaude
andcommitted
refactor: simplify Makefile test targets to use scripts/test.sh
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 0910d66 commit 30cb1cc

File tree

1 file changed

+6
-11
lines changed

1 file changed

+6
-11
lines changed

Makefile

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -124,24 +124,19 @@ test/setup:
124124
@echo ""
125125

126126
test:
127-
@if [ "$(RUN)" != "" ]; then \
128-
gotestsum --rerun-fails=2 --hide-summary=skipped --format-hide-empty-pkg --packages="$(TEST)" -- $(TESTARGS) -run "$(RUN)"; \
129-
else \
130-
gotestsum --rerun-fails=2 --hide-summary=skipped --format-hide-empty-pkg --packages="$(TEST)" -- $(TESTARGS); \
131-
fi
127+
TEST="$(TEST)" RUN="$(RUN)" TESTARGS="$(TESTARGS)" ./scripts/test.sh test
132128

133129
test/unit:
134-
gotestsum --rerun-fails=2 --hide-summary=skipped --format-hide-empty-pkg --packages="$(TEST)" -- $(TESTARGS) -short
130+
TEST="$(TEST)" RUN="$(RUN)" TESTARGS="$(TESTARGS)" ./scripts/test.sh unit
135131

136132
test/integration:
137133
gotestsum --rerun-fails=2 --hide-summary=skipped --format-hide-empty-pkg --packages="$(TEST)" -- $(TESTARGS) -run "Integration"
138134

139135
test/e2e:
140-
@if [ "$(RUN)" != "" ]; then \
141-
$(if $(TESTINFRA),TESTINFRA=$(TESTINFRA)) go test ./cmd/e2e $(TESTARGS) -run "$(RUN)"; \
142-
else \
143-
$(if $(TESTINFRA),TESTINFRA=$(TESTINFRA)) go test ./cmd/e2e $(TESTARGS); \
144-
fi
136+
RUN="$(RUN)" TESTARGS="$(TESTARGS)" ./scripts/test.sh e2e
137+
138+
test/full:
139+
TEST="$(TEST)" RUN="$(RUN)" TESTARGS="$(TESTARGS)" ./scripts/test.sh full
145140

146141
test/e2e/rediscluster:
147142
@echo "Running Redis cluster e2e tests in Docker container..."

0 commit comments

Comments
 (0)