Repository navigation
test: align with the synced testing conventions - #17
Conversation
One describe per export (TestList, componentTestSuite), build* helper naming, UPPER_CASE fixture literals, satisfies on the suite args, and @ts-expect-error in place of @ts-ignore and an as-cast. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GrSSi8k4q48PmHCXCMFVrN
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GrSSi8k4q48PmHCXCMFVrN
PR Diff Line Count
|
||||||||||||||||||||||||||||||||||||
|
Size Change: 0 B Total Size: 1.42 kB ℹ️ View Unchanged
|
|
@claude remove coverage from vitest config and add to .gitignore anyways |
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GrSSi8k4q48PmHCXCMFVrN
|
Done in d2e17e0:
Generated by Claude Code |
Requested by Blake · project thread
Before:
component-suite.test.tsxsplitcomponentTestSuiteacross two top-level describes and testedTestListin a baretest. It also named literal fixtures in camelCase, had amakeAnonymoushelper, and used@ts-ignoreand anas unknown ascast to feed values the types refuse.After: there is one
describeper export (ts-unit-testing.md), withexecutionandregistrationnested undercomponentTestSuite. Fixtures are UPPER_CASE, andMOCK_SUITE_ARGSis checked withsatisfies. The helper isbuildAnonymousComponent, and every deliberately refused value goes through// @ts-expect-errorwith a reason (ts-testing-all.md). Only tests change; there are still 26 and all pass.How: the edits are in
src/__tests__/component-suite.test.tsx, plus a blank line after the summary in__helpers__/spy-on-tests.ts's doc comment (all.md).pnpm checkandpnpm test:unitpass locally.🤖 Generated with Claude Code
https://claude.ai/code/session_01GrSSi8k4q48PmHCXCMFVrN
Generated by Claude Code