Skip to content

Commit d69bde6

Browse files
committed
fix: restore snapshot-routing contract test, remove orphaned comment
snapshot-routing.test.ts validates that every command in the snapshot catalog has a handler — a structural check invisible to Istanbul. Same false-positive pattern as package-exports.test.ts.
1 parent 7d92ac6 commit d69bde6

2 files changed

Lines changed: 9 additions & 4 deletions

File tree

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
import { test, expect } from 'vitest';
2+
import { DAEMON_COMMAND_GROUPS } from '../../../command-catalog.ts';
3+
import { SNAPSHOT_COMMAND_HANDLERS } from '../snapshot.ts';
4+
5+
test('snapshot command catalog has handler coverage', () => {
6+
for (const command of DAEMON_COMMAND_GROUPS.snapshot) {
7+
expect(SNAPSHOT_COMMAND_HANDLERS).toHaveProperty(command);
8+
}
9+
});

src/daemon/handlers/__tests__/snapshot.test.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,6 @@ import { test } from 'vitest';
22
import assert from 'node:assert/strict';
33
import { parseWaitPositionals as parseWaitArgs } from '../../../command-codecs/wait.ts';
44

5-
// --- parseTimeout ---
6-
7-
// --- parseWaitArgs ---
8-
95
test('parseWaitArgs returns null for empty args', () => {
106
assert.equal(parseWaitArgs([]), null);
117
});

0 commit comments

Comments
 (0)