Skip to content

Commit c1eb031

Browse files
committed
CLAUDE: use verbose output in filtered tests
I observed a `./dev test pkg -f=x` scenario where no test matched, and false success was declared. Release note: None
1 parent 899c462 commit c1eb031

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

CLAUDE.md

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,15 +23,19 @@ CockroachDB is a distributed SQL database written in Go, built with Bazel and ma
2323

2424
**Testing:**
2525
```bash
26-
./dev test pkg/sql # Run unit tests for SQL package
27-
./dev test pkg/sql -f=TestParse # Run specific test pattern
26+
./dev test pkg/sql # Run unit tests for SQL package
27+
./dev test pkg/sql -f=TestParse -v # Run specific test pattern.
2828
./dev test pkg/sql --race # Run with race detection
2929
./dev test pkg/sql --stress # Run repeatedly until failure
3030
./dev testlogic # Run all SQL logic tests
3131
./dev testlogic ccl # Run enterprise logic tests
3232
./dev testlogic --files='prepare|fk' # Run specific test files
3333
```
3434

35+
Note that when filtering tests via `-f` to include the `-v` flag which
36+
will warn you in the output if your filter didn't match anything. Look
37+
for `testing: warning: no tests to run` in the output.
38+
3539
**Code Generation and Linting:**
3640
```bash
3741
./dev generate # Generate all code (protobuf, parsers, etc.)
@@ -129,4 +133,4 @@ Always run `./dev generate` after modifying `.proto` files, SQL grammar, or opti
129133
- Require the user to specify an epic number (or None) which should be included at the bottom of the commit record following "Epic:".
130134
- Prefix the subject line with the package in which the bulk of the changes occur.
131135
- For multi-commit PRs, summarize each commit in the PR record.
132-
- Do not include a test plan unless explicitly asked by the user.
136+
- Do not include a test plan unless explicitly asked by the user.

0 commit comments

Comments
 (0)