Skip to content

Commit 2e1aa00

Browse files
committed
docs(driver-tests): drop vitest name filter
1 parent 5ffa0d0 commit 2e1aa00

1 file changed

Lines changed: 6 additions & 7 deletions

File tree

  • .claude/skills/driver-test-runner

.claude/skills/driver-test-runner/SKILL.md

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -173,14 +173,16 @@ For each unchecked row in order, run the runtimes selected by the `runtime` arg
173173
| native | local |
174174
| wasm | remote |
175175

176-
**b) Build the filter command:**
176+
**b) Build the scoped file command:**
177177

178178
Each suite lives in its own file under `rivetkit-typescript/packages/rivetkit/tests/driver/<file>.test.ts`. With env overrides set, the describe block nesting is:
179179

180180
```
181181
<Outer Suite> > static registry > runtime (<runtime>) / sqlite (<backend>) / encoding (<encoding>) > <Suite Description>
182182
```
183183

184+
Do not use Vitest `-t` for driver matrix cells. The env vars already constrain the file to one runtime/backend/encoding cell, and the `-t` path can break `describe.sequential` ordering in this suite.
185+
184186
Base command (native):
185187

186188
```bash
@@ -189,7 +191,6 @@ cd rivetkit-typescript/packages/rivetkit && \
189191
RIVETKIT_DRIVER_TEST_SQLITE=local \
190192
RIVETKIT_DRIVER_TEST_ENCODING=bare \
191193
pnpm test tests/driver/<FILE>.test.ts \
192-
-t "static registry.*runtime \\(native\\) / sqlite \\(local\\) / encoding \\(bare\\).*<SUITE_DESCRIPTION>" \
193194
> /tmp/driver-test-current.log 2>&1
194195
echo "EXIT: $?"
195196
```
@@ -202,14 +203,11 @@ cd rivetkit-typescript/packages/rivetkit && \
202203
RIVETKIT_DRIVER_TEST_SQLITE=remote \
203204
RIVETKIT_DRIVER_TEST_ENCODING=bare \
204205
pnpm test tests/driver/<FILE>.test.ts \
205-
-t "static registry.*runtime \\(wasm\\) / sqlite \\(remote\\) / encoding \\(bare\\).*<SUITE_DESCRIPTION>" \
206206
> /tmp/driver-test-current.log 2>&1
207207
echo "EXIT: $?"
208208
```
209209

210-
Replace `<FILE>` with the file name stem (part before the `|` in the progress file) and `<SUITE_DESCRIPTION>` with the suite description (part after the `|`). Escape parentheses in the description if present. Forward slashes inside the describe path do not need to be escaped.
211-
212-
**Important:** The suite description in the `-t` filter must match the inner `describe(...)` text in the test file exactly. Some mappings:
210+
Replace `<FILE>` with the file name stem (part before the `|` in the progress file). Keep the suite description in the progress file for human-readable tracking only. Some mappings:
213211

214212
| File | Suite Description Text |
215213
|------|----------------------|
@@ -300,10 +298,11 @@ cd rivetkit-typescript/packages/rivetkit && \
300298
RIVETKIT_DRIVER_TEST_SQLITE=<backend> \
301299
RIVETKIT_DRIVER_TEST_ENCODING=bare \
302300
pnpm test tests/driver/<FILE>.test.ts \
303-
-t "static registry.*runtime \\(<runtime>\\) / sqlite \\(<backend>\\) / encoding \\(bare\\).*<SUITE>.*<PARTIAL_TEST_NAME>" \
304301
> /tmp/driver-test-narrow.log 2>&1
305302
```
306303

304+
For true single-test isolation, prefer a temporary `.only` while debugging and remove it before committing. Do not use `-t` as a flake workaround.
305+
307306
If the bug only appears on one runtime, that's a strong signal — focus the diff hunt on the corresponding runtime adapter (`napi-runtime.ts` / `wasm-runtime.ts`) and any wasm-feature-gated code in `rivetkit-core` and `rivetkit-typescript/packages/rivetkit-wasm`.
308307

309308
### 6. Completion

0 commit comments

Comments
 (0)