Make test args and chunking configurable - #67
Merged
Conversation
GO_TEST_ARGS lets a focused run override the default -race (e.g. GO_TEST_ARGS="-race -run TestX -v"), and test-chunk CHUNK=N/M runs one 1-indexed slice of the module list — usable locally today and ready for a CI matrix if the suite outgrows a single job.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
GO_TEST_ARGS ?= -race: focused runs can override the test flags per invocation, e.g.make test-obsx GO_TEST_ARGS="-race -run TestSetup -v". Default behavior unchanged.make test-chunk CHUNK=N/M: runs the full per-module gate for one 1-indexed slice of the module list (CHUNK=2/4= second quarter). Useful locally now; ready to back a CI matrix if the suite outgrows the current single ~2-minute job.Why
Ergonomics adopted from mature multi-module Go monorepos. CI is intentionally unchanged: the existing gate already runs tidy+fmt+vet+golangci-lint+race tests per module plus a dirty-tree check, so no extra lint/fmt jobs are needed.
Verification
make test-chunk CHUNK=1/3|2/3|3/3slices 13 modules as 5/5/3 and all gates pass;make test-flagx GO_TEST_ARGS="-race -run TestEnum -v"runs the focused subset; plainmake testbehavior is unchanged.🤖 Generated with Claude Code