tests: selection actions tests

Author: afshinm

plugins/selection/src/rowSelection/actions.ts

@@ -13,12 +13,13 @@ export const CheckRow = (rowId: string) => (state) => {
 };
 
 export const UncheckRow = (rowId: string) => (state) => {
-  const index = state.rowSelection.rowIds.indexOf(rowId);
+  const rowIds = state.rowSelection?.rowIds || [];
+  const index = rowIds.indexOf(rowId);
 
   // rowId doesn't exist
-  if (index === -1) state;
+  if (index === -1) return state;
 
-  const cloned = [...state.rowSelection.rowIds];
+  const cloned = [...rowIds];
   cloned.splice(index, 1);
 
   return {

plugins/selection/tests/rowSelection/actions.test.ts

@@ -39,6 +39,26 @@ describe('Actions', () => {
     });
   });
 
+  it('should UNCHECK the correct item', () => {
+    const state = Actions.UncheckRow('42')({
+      rowSelection: {
+        rowIds: ['22', '11'],
+      },
+    });
+
+    expect(state).toStrictEqual({
+      rowSelection: {
+        rowIds: ['22', '11'],
+      },
+    });
+  });
+
+  it('should UNCHECK when rowIds is null', () => {
+    const state = Actions.UncheckRow('42')({});
+
+    expect(state).toStrictEqual({});
+  });
+
   it('should trigger UNCHECK when rowIds is empty', () => {
     const state = Actions.UncheckRow('42')({
       rowSelection: {