From 33e3f2bdc622c7e8d9a3e8fc99150f11762aa8a7 Mon Sep 17 00:00:00 2001 From: nmemmert <67961292+nmemmert@users.noreply.github.com> Date: Tue, 22 Sep 2026 15:35:32 -0400 Subject: [PATCH] Fix Greek lookup test expecting a single 'faith, belief' field With no gloss-map entry, the English gloss falls back to the KJV definition, so both English Gloss and Short Definition show it. Assert both instead of failing on the duplicate. Co-Authored-By: Claude Opus 5.5 --- src/App.test.jsx | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/App.test.jsx b/src/App.test.jsx index 4c1eeab..c18da1c 100644 --- a/src/App.test.jsx +++ b/src/App.test.jsx @@ -322,7 +322,9 @@ describe('Greek word lookup', () => { fireEvent.click(screen.getByRole('button', { name: /look up greek/i })); await screen.findByDisplayValue('πίστις'); expect(screen.getByDisplayValue('pistis')).toBeInTheDocument(); - expect(screen.getByDisplayValue('faith, belief')).toBeInTheDocument(); + // With no gloss-map entry, the English gloss falls back to the KJV + // definition, so it matches the Short Definition field. + expect(screen.getAllByDisplayValue('faith, belief')).toHaveLength(2); }); test('shows "No definition found." when the API returns an empty array', async () => {