From c4ab03d38709d412352e5bcea10dca1cd02044a4 Mon Sep 17 00:00:00 2001 From: claude code agent 227 Date: Mon, 29 Jun 2026 22:26:39 +0300 Subject: [PATCH] docs(editor-ext): correct why vitest skips the table-test helper (F1c) The comment claimed vitest skips the file because it has no test cases; vitest collects by filename glob, so the real reason is the name not matching *.{test,spec}.ts. Reword to cite the glob and warn that adding test cases here would not run them. Co-Authored-By: Claude Opus 4.8 (1M context) --- .../editor-ext/src/lib/table/utils/table-test-helpers.ts | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/packages/editor-ext/src/lib/table/utils/table-test-helpers.ts b/packages/editor-ext/src/lib/table/utils/table-test-helpers.ts index 035689df..326cb650 100644 --- a/packages/editor-ext/src/lib/table/utils/table-test-helpers.ts +++ b/packages/editor-ext/src/lib/table/utils/table-test-helpers.ts @@ -13,8 +13,9 @@ import { convertTableNodeToArrayOfRows } from "./convert-table-node-to-array-of- * means a schema change (e.g. cellAttributes) is applied once instead of being * copied across every test file. * - * This is a test-only helper (not shipped) and intentionally contains no test - * cases, so vitest does not pick it up as a spec file. + * This is a test-only helper (not shipped). Its name does not match vitest's + * `*.{test,spec}.ts` include glob, so it is not collected as a spec file — + * adding test cases here would NOT make them run; put them in a `*.test.ts`. */ const tNodes = tableNodes({