@coveord/plasma-mantine 52.7.1 → 52.8.1

package/src/components/code-editor/search/__tests__/Search.unit.spec.tsx

@@ -0,0 +1,16 @@
+import {render, screen, userEvent} from '@test-utils';
+import {Search} from '../Search';
+
+describe('Search', () => {
+    describe('handleSearch', () => {
+        it('should call handleSearch when the search button is clicked', async () => {
+            const handleSearchSpy = vi.fn();
+            const user = userEvent.setup();
+            render(<Search handleSearch={handleSearchSpy} />);
+
+            await user.click(screen.getByRole('button', {name: /search/i}));
+
+            expect(handleSearchSpy).toHaveBeenCalled();
+        });
+    });
+});

package/src/components/code-editor/search/index.ts

@@ -0,0 +1 @@
+export * from './Search';

package/src/components/table/TablePredicate.tsx

@@ -39,11 +39,10 @@ export const TablePredicate: FunctionComponent<TablePredicateProps> = ({
     ...others
 }) => {
     const {classes} = useStyles(null, {name: 'TablePredicate', classNames, styles, unstyled});
-    const {onChange, form} = useTable();
+    const {form} = useTable();
 
     const onUpdate = (newValue: string) => {
         form.setFieldValue('predicates', {...form.values.predicates, [id]: newValue});
-        onChange?.();
     };
 
     return (

package/src/components/table/__tests__/TablePredicate.spec.tsx

@@ -1,5 +1,5 @@
 import {ColumnDef, createColumnHelper} from '@tanstack/table-core';
-import {render, screen, userEvent, waitFor} from '@test-utils';
+import {act, render, screen, userEvent, waitFor} from '@test-utils';
 
 import {Table} from '../Table';
 
@@ -75,7 +75,7 @@ describe('Table.Predicate', () => {
     });
 
     it('calls onChange when changing the predicate', async () => {
-        const user = userEvent.setup({delay: null});
+        const user = userEvent.setup({advanceTimers: vi.advanceTimersByTime});
         const onChange = vi.fn();
         render(
             <Table
@@ -96,33 +96,18 @@ describe('Table.Predicate', () => {
             </Table>
         );
 
-        await waitFor(() => {
-            expect(
-                screen.getByRole('searchbox', {
-                    name: 'rank',
-                })
-            ).toHaveValue('Second');
-        });
+        expect(screen.getByRole('searchbox', {name: 'rank'})).toHaveValue('Second');
 
-        await user.click(
-            screen.getByRole('searchbox', {
-                name: 'rank',
-            })
-        );
+        await user.click(screen.getByRole('searchbox', {name: 'rank'}));
 
-        await user.click(
-            screen.getByRole('option', {
-                name: 'First',
-            })
-        );
+        await user.click(screen.getByRole('option', {name: 'First'}));
 
-        expect(
-            screen.getByRole('searchbox', {
-                name: 'rank',
-            })
-        ).toHaveValue('First');
-        vi.advanceTimersByTime(500);
+        expect(screen.getByRole('searchbox', {name: 'rank'})).toHaveValue('First');
+        act(() => {
+            vi.advanceTimersByTime(500);
+        });
 
+        expect(onChange).toHaveBeenCalledTimes(1);
         expect(onChange).toHaveBeenCalledWith(expect.objectContaining({predicates: {rank: 'first'}}));
     });
 });