From ef6c7b805de0ffcffae5c58d0b1d65567433d1bd Mon Sep 17 00:00:00 2001 From: Roland Date: Wed, 5 Aug 2026 17:05:23 -0600 Subject: [PATCH] test: the label test read half the alignment contract It asserted the label is block but not the 4px gap, so dropping the margin -- the half that actually lines an InputForm up with the DestinationForm beside it -- would still have passed. Co-Authored-By: Claude Opus 5 (1M context) --- app/src/hooks/useForm.test.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/app/src/hooks/useForm.test.js b/app/src/hooks/useForm.test.js index 6ea3b68e..0b55d395 100644 --- a/app/src/hooks/useForm.test.js +++ b/app/src/hooks/useForm.test.js @@ -968,7 +968,7 @@ describe('Form Components', () => { ); }); - it('renders the label as a block above the input', () => { + it('renders the label as a block above the input, with the gap FieldLabel uses', () => { const form = createMockForm({name: ''}); renderWithProviders( @@ -980,7 +980,9 @@ describe('Form Components', () => { ); const label = screen.getByText(/Channel Name/).closest('label'); - expect(label).toHaveStyle({display: 'block'}); + // Both halves of the alignment: without the margin, an InputForm beside a + // DestinationForm starts its input 4px higher. + expect(label).toHaveStyle({display: 'block', marginBottom: '4px'}); }); });