From ed16cdd61d87cbdb13c52a7c2b9e4711b0a8b08c Mon Sep 17 00:00:00 2001 From: Eric NGUYEN Date: Tue, 16 Aug 2022 13:53:47 +0200 Subject: [PATCH] Fix regressions + fix tests because some inputs does not have onChange event anymore (id, parentId) --- src/Components/Properties/DynamicForm.tsx | 2 +- src/Components/Properties/Properties.test.tsx | 10 +++++----- src/Components/Properties/StaticForm.tsx | 2 +- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/Components/Properties/DynamicForm.tsx b/src/Components/Properties/DynamicForm.tsx index 142763f..c604178 100644 --- a/src/Components/Properties/DynamicForm.tsx +++ b/src/Components/Properties/DynamicForm.tsx @@ -43,7 +43,7 @@ const DynamicForm: React.FunctionComponent = (props) => { /> { fireEvent.change(propertyParentId as Element, { target: { value: 'parentedId' } }); fireEvent.change(propertyX as Element, { target: { value: '2' } }); fireEvent.change(propertyY as Element, { target: { value: '2' } }); - expect(handleChange).toBeCalledTimes(4); + expect(handleChange).toBeCalledTimes(2); - expect(prop.id).toBe('stuffed'); - expect(prop.parentId).toBe('parentedId'); + expect(prop.id).toBe('stuff'); + expect(prop.parentId).toBe('parentId'); expect(prop.x).toBe('2'); expect(prop.y).toBe('2'); rerender( { propertyX = container.querySelector('#x'); propertyY = container.querySelector('#y'); expect(propertyId).toBeDefined(); - expect((propertyId as HTMLInputElement).value).toBe('stuffed'); + expect((propertyId as HTMLInputElement).value).toBe('stuff'); expect(propertyParentId).toBeDefined(); - expect((propertyParentId as HTMLInputElement).value).toBe('parentedId'); + expect((propertyParentId as HTMLInputElement).value).toBe('parentId'); expect(propertyX).toBeDefined(); expect((propertyX as HTMLInputElement).value).toBe('2'); expect(propertyY).toBeDefined(); diff --git a/src/Components/Properties/StaticForm.tsx b/src/Components/Properties/StaticForm.tsx index e7cb6d8..174fd46 100644 --- a/src/Components/Properties/StaticForm.tsx +++ b/src/Components/Properties/StaticForm.tsx @@ -43,7 +43,7 @@ const StaticForm: React.FunctionComponent = (props) => { />