Skip to content

Commit b08c41a

Browse files
committed
docs(testing): prefer dynamic smoke test over many stored smoke-all fixtures
1 parent 14ef83b commit b08c41a

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

.claude/rules/testing/smoke-all-tests.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -147,6 +147,10 @@ Keep the `_quarto: tests:` block assertion-focused, but inline `#` comments labe
147147
**Issue-based:** `tests/docs/smoke-all/YYYY/MM/DD/<issue>.qmd`
148148
**Feature-based:** `tests/docs/smoke-all/<feature>/`
149149

150+
### Many fixtures: prefer a dynamic smoke test
151+
152+
Every non-`_`-prefixed `.qmd` under `smoke-all/` is globbed and rendered as its own test target. When a test needs many fixture files (e.g. dozens of pages to reproduce a project-level bug), each stored page becomes a redundant per-file render — plus repo bloat — on top of the project render. Generate the project in a temp directory from a TypeScript `tests/smoke/<feature>/` test instead (`testQuartoCmd("render", [tempDir], ...)` with `setup`/`teardown`). See the "Project Rendering Tests" and "Performance Budget" patterns in `llm-docs/testing-patterns.md`.
153+
150154
## Creating New Tests
151155

152156
```bash

0 commit comments

Comments
 (0)