Skip to content

Commit dbeba09

Browse files
committed
style(forks): use TSDoc for the new test declaration comments
CLAUDE.md requires TSDoc for documentation and no non-TSDoc comments. The vi.mock boundary note, the resolve helper, and the beforeEach mock-reset rationale all document declarations, so doc tooling could not associate them. In-body step comments are left as-is - they explain a flow, not a declaration.
1 parent 68c1b10 commit dbeba09

1 file changed

Lines changed: 13 additions & 7 deletions

File tree

apps/sim/ee/workspace-forking/lib/mapping/dependent-reconfigs.test.ts

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,10 @@ const { mockGetToolInputParamConfigs } = vi.hoisted(() => ({
77
mockGetToolInputParamConfigs: vi.fn(() => [] as unknown[]),
88
}))
99

10-
// Mocked at the module boundary so these tests stay about the collector's own logic rather
11-
// than the tool/block registries the real resolver reaches into.
10+
/**
11+
* Mocked at the module boundary so these tests stay about the collector's own logic rather
12+
* than the tool/block registries the real resolver reaches into.
13+
*/
1214
vi.mock('@/lib/workflows/search-replace/indexer', () => ({
1315
getToolInputParamConfigs: mockGetToolInputParamConfigs,
1416
}))
@@ -56,13 +58,17 @@ const replaceItem = {
5658
mode: 'replace' as const,
5759
}
5860

59-
// No persisted block map in these unit tests, so the resolver derives - matching the
60-
// `deriveForkBlockId(...)` ids the expectations assert.
61+
/**
62+
* No persisted block map in these unit tests, so the resolver derives - matching the
63+
* `deriveForkBlockId(...)` ids the expectations assert.
64+
*/
6165
const resolve = buildForkBlockIdResolver(true, EMPTY_FORK_BLOCK_MAP)
6266

63-
// The indexer mock is module-scoped, so a `mockReturnValue` from one test would otherwise
64-
// leak into the next and make these order-dependent. Reset to the empty (no authoritative
65-
// visibility) default before each.
67+
/**
68+
* The indexer mock is module-scoped, so a `mockReturnValue` from one test would otherwise
69+
* leak into the next and make these order-dependent. Reset to the empty (no authoritative
70+
* visibility) default before each.
71+
*/
6672
beforeEach(() => {
6773
mockGetToolInputParamConfigs.mockReset()
6874
mockGetToolInputParamConfigs.mockReturnValue([])

0 commit comments

Comments
 (0)