Merge pull request #462 from patternfly/some_version_bumps

fix(deps): bump various versions

Author: dlabaj

package-lock.json

@@ -47,14 +47,14 @@
     "@patternfly/patternfly": "^6.0.0",
     "@patternfly/react-code-editor": "^6.0.0",
     "@patternfly/patternfly-a11y": "^5.1.0",
-    "@types/react": "^18.3.21",
-    "@types/react-dom": "^18.3.5",
+    "@types/react": "^18.3.23",
+    "@types/react-dom": "^18.3.7",
     "@types/react-router-dom": "^5.3.3",
     "react": "^18.3.1",
     "react-dom": "^18.3.1",
-    "react-router": "^6.30.0",
-    "react-router-dom": "^6.30.0",
+    "react-router": "^6.30.1",
+    "react-router-dom": "^6.30.1",
     "rimraf": "^6.0.1",
-    "typescript": "^5.4.5"
+    "typescript": "^5.8.3"
   }
 }

packages/module/package.json

@@ -1,5 +1,5 @@
 import React from 'react';
-import { render } from '@testing-library/react';
+import { render, fireEvent } from '@testing-library/react';
 import DataViewFilters from './DataViewFilters';
 import DataViewToolbar from '../DataViewToolbar';
 import DataViewTextFilter from '../DataViewTextFilter';
@@ -18,4 +18,22 @@ describe('DataViewFilters component', () => {
     />);
     expect(container).toMatchSnapshot();
   });
+
+  it('should call onChange with correct key and value when filter changes', () => {
+    const mockOnChange = jest.fn();
+    const { getByLabelText } = render(
+      <DataViewToolbar
+        filters={
+          <DataViewFilters onChange={mockOnChange} values={{}}>
+            <DataViewTextFilter filterId="one" title="One" />
+            <DataViewTextFilter filterId="two" title="Two" />
+          </DataViewFilters>
+        }
+      />
+    );
+    const input = getByLabelText('One filter');
+    input.focus();
+    fireEvent.input(input, { target: { value: 'abc' } });
+    expect(mockOnChange).toHaveBeenCalledWith('one', { one: 'abc' });
+  });
 });

packages/module/src/DataViewFilters/DataViewFilters.test.tsx

@@ -130,7 +130,7 @@ export const DataViewFilters = <T extends object>({
               value: unknown;
             }>, {
               showToolbarItem: activeAttributeMenu === child.props.title,
-              onChange: (event, value) => onChange?.(event, { [child.props.filterId]: value } as Partial<T>),
+              onChange: (event, value) => onChange?.(child.props.filterId, { [child.props.filterId]: value } as Partial<T>),
               value: values?.[child.props.filterId],
               ...child.props
             })