@patternfly/react-data-view 6.3.0-prerelease.3 → 6.4.0-prerelease.2

package/src/Hooks/selection.test.tsx

@@ -9,6 +9,7 @@ describe('useDataViewSelection', () => {
       selected: [],
       onSelect: expect.any(Function),
       isSelected: expect.any(Function),
+      setSelected: expect.any(Function),
     })
   });
 
@@ -19,6 +20,7 @@ describe('useDataViewSelection', () => {
       selected: initialSelected,
       onSelect: expect.any(Function),
       isSelected: expect.any(Function),
+      setSelected: expect.any(Function),
     })
   });
 
@@ -49,4 +51,66 @@ describe('useDataViewSelection', () => {
     expect(result.current.isSelected({ id: 1, name: 'test1' })).toBe(true);
     expect(result.current.isSelected({ id: 3, name: 'test2' })).toBe(false);
   });
-});
+
+  it('should have setSelected function in return object', () => {
+    const { result } = renderHook(() => useDataViewSelection({ matchOption: (a, b) => a.id === b.id }))
+    expect(result.current).toEqual({ 
+      selected: [],
+      onSelect: expect.any(Function),
+      isSelected: expect.any(Function),
+      setSelected: expect.any(Function),
+    })
+  });
+
+  it('should set selected items directly using setSelected - objects', async () => {
+    const initialSelected = [ { id: 1, name: 'test1' } ];
+    const { result } = renderHook(() => useDataViewSelection({ initialSelected, matchOption: (a, b) => a.id === b.id }))
+    
+    const newSelected = [ { id: 2, name: 'test2' }, { id: 3, name: 'test3' } ];
+    
+    await act(async () => {
+      result.current.setSelected(newSelected);
+    });
+    
+    expect(result.current.selected).toEqual(newSelected);
+  });
+
+  it('should set selected items directly using setSelected - strings', async () => {
+    const initialSelected = [ 'test1', 'test2' ];
+    const { result } = renderHook(() => useDataViewSelection({ initialSelected, matchOption: (a, b) => a === b }))
+    
+    const newSelected = [ 'test3', 'test4', 'test5' ];
+    
+    await act(async () => {
+      result.current.setSelected(newSelected);
+    });
+    
+    expect(result.current.selected).toEqual(newSelected);
+  });
+
+  it('should clear all selections using setSelected with empty array', async () => {
+    const initialSelected = [ { id: 1, name: 'test1' }, { id: 2, name: 'test2' } ];
+    const { result } = renderHook(() => useDataViewSelection({ initialSelected, matchOption: (a, b) => a.id === b.id }))
+    
+    await act(async () => {
+      result.current.setSelected([]);
+    });
+    
+    expect(result.current.selected).toEqual([]);
+  });
+
+  it('should update isSelected correctly after using setSelected', async () => {
+    const initialSelected = [ { id: 1, name: 'test1' } ];
+    const { result } = renderHook(() => useDataViewSelection({ initialSelected, matchOption: (a, b) => a.id === b.id }))
+    
+    const newSelected = [ { id: 2, name: 'test2' }, { id: 3, name: 'test3' } ];
+    
+    await act(async () => {
+      result.current.setSelected(newSelected);
+    });
+    
+    expect(result.current.isSelected({ id: 1, name: 'test1' })).toBe(false);
+    expect(result.current.isSelected({ id: 2, name: 'test2' })).toBe(true);
+    expect(result.current.isSelected({ id: 3, name: 'test3' })).toBe(true);
+  });
+});

package/src/Hooks/selection.ts

@@ -24,9 +24,14 @@ export const useDataViewSelection = (props?: UseDataViewSelectionProps) => {
   
   const isSelected = (item: any): boolean => Boolean(selected.find(selected => matchOption(selected, item)));
 
+  const setSelectedItems = (items: any[]) => {
+    setSelected(items);
+  };
+
   return {
     selected,
     onSelect,
-    isSelected
+    isSelected,
+    setSelected: setSelectedItems
   };
 };

package/src/InternalContext/InternalContext.tsx

@@ -6,6 +6,8 @@ export interface DataViewSelection {
   onSelect: (isSelecting: boolean, items?: any[] | any) => void;  // eslint-disable-line @typescript-eslint/no-explicit-any
   /** Checks if a specific item is currently selected */
   isSelected: (item: any) => boolean;  // eslint-disable-line @typescript-eslint/no-explicit-any
+  /** Directly sets the selected items */
+  setSelected?: (items: any[]) => void;  // eslint-disable-line @typescript-eslint/no-explicit-any
   /** Determines if selection is disabled for a given item */
   isSelectDisabled?: (item: any) => boolean;  // eslint-disable-line @typescript-eslint/no-explicit-any
 }