From 7c6573a563bd7f870cf9eeba30ec83275fc16038 Mon Sep 17 00:00:00 2001 From: Sergei Maertens Date: Fri, 17 Nov 2023 10:40:08 +0100 Subject: [PATCH] :green_heart: [#52] Update tests after behaviour change Setting the data source to 'manual' now populates the values with a single empty option. --- src/components/builder/values/values-config.stories.tsx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/components/builder/values/values-config.stories.tsx b/src/components/builder/values/values-config.stories.tsx index ff3bbd57..aac059bb 100644 --- a/src/components/builder/values/values-config.stories.tsx +++ b/src/components/builder/values/values-config.stories.tsx @@ -135,7 +135,6 @@ export const SelectBoxesResetState: StoryObj<{ await userEvent.click(await canvas.findByText('Manually fill in')); const addBtn = await canvas.findByRole('button', {name: 'Add another'}); await expect(addBtn).toBeVisible(); - await userEvent.click(addBtn); await doSubmit(); await expect(args.onSubmit).toHaveBeenCalledWith({ @@ -183,7 +182,8 @@ export const SelectBoxesResetState: StoryObj<{ await doSubmit(); await expect(args.onSubmit).toHaveBeenCalledWith({ openForms: {dataSrc: 'manual'}, - values: [], // all pre-existing items have been cleared + // all pre-existing items have been cleared and we ensure there's at least 1 item + values: [{value: '', label: '', openForms: {translations: {}}}], }); // @ts-expect-error jest mocks + TS doesn't play nice together args.onSubmit.mockClear();