Skip to content

Commit c8d31a3

Browse files
committed
test(cli): make the schema-valid pins actually parse the fixture
The four rows in "the localized fixtures are schema-VALID" called `normalizeStackInput(stack).stack`, but `normalizeStackInput` returns the normalized stack itself, not a `{ stack }` wrapper — so `.stack` was `undefined` and every row was parsing `undefined`, not the fixture. All four were red. Worse, the CONTROL row was passing for the wrong reason: it asserts a number label does NOT parse, and `undefined` does not parse either, so it went green without ever discriminating on the label. The block that exists to prove the localized fixtures are supported authoring input was measuring nothing. Drop the `.stack`. The four positives now parse clean and the control still rejects, so the control discriminates on the label for the first time. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01D47qPfEWVPmhguWgBZCi5N
1 parent 6cd2dc2 commit c8d31a3

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

packages/cli/test/lint-label-case-localized.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -208,13 +208,13 @@ describe('the localized fixtures are schema-VALID — this is not bad input', ()
208208

209209
for (const [carrier, stack] of stacks) {
210210
it(`${carrier} parses clean`, () => {
211-
const parsed = ObjectStackDefinitionSchema.safeParse(normalizeStackInput(stack as any).stack);
211+
const parsed = ObjectStackDefinitionSchema.safeParse(normalizeStackInput(stack as any));
212212
expect(parsed.success).toBe(true);
213213
});
214214
}
215215

216216
it('CONTROL: a number label does NOT parse, so the parse check discriminates', () => {
217-
const parsed = ObjectStackDefinitionSchema.safeParse(normalizeStackInput(stackWithApp(42) as any).stack);
217+
const parsed = ObjectStackDefinitionSchema.safeParse(normalizeStackInput(stackWithApp(42) as any));
218218
expect(parsed.success).toBe(false);
219219
});
220220
});

0 commit comments

Comments
 (0)