chore: skip effect on shouldCellUpdate (#687)

Author: zombieJ

src/Cell/index.tsx

@@ -107,6 +107,9 @@ function Cell<RecordType extends DefaultRecordType>(
     startRow,
     endRow,
     onHover,
+
+    // MISC
+    shouldCellUpdate,
   }: InternalCellProps<RecordType>,
   ref: React.Ref<any>,
 ): React.ReactElement {
@@ -236,7 +239,7 @@ function Cell<RecordType extends DefaultRecordType>(
         [`${cellPrefixCls}-ellipsis`]: ellipsis,
         [`${cellPrefixCls}-with-append`]: appendNode,
         [`${cellPrefixCls}-fix-sticky`]: (isFixLeft || isFixRight) && isSticky && supportSticky,
-        [`${cellPrefixCls}-row-hover`]: hovering,
+        [`${cellPrefixCls}-row-hover`]: !shouldCellUpdate && hovering, // Not patch style if using shouldCellUpdate
       },
       additionalProps.className,
       cellClassName,

tests/Table.spec.js

@@ -932,9 +932,24 @@ describe('Table.Basic', () => {
     });
   });
 
-  it('hover', () => {
-    const wrapper = mount(createTable());
-    wrapper.find('tbody td').first().simulate('mouseEnter');
-    expect(wrapper.exists('.rc-table-cell-row-hover')).toBeTruthy();
+  describe('hover', () => {
+    it('basic', () => {
+      const wrapper = mount(createTable());
+      wrapper.find('tbody td').first().simulate('mouseEnter');
+      expect(wrapper.exists('.rc-table-cell-row-hover')).toBeTruthy();
+    });
+
+    it('skip when config should cell update', () => {
+      const wrapper = mount(
+        createTable({
+          columns: [
+            { title: 'Name', dataIndex: 'name', key: 'name', shouldCellUpdate: () => false },
+          ],
+        }),
+      );
+
+      wrapper.find('tbody td').first().simulate('mouseEnter');
+      expect(wrapper.exists('.rc-table-cell-row-hover')).toBeFalsy();
+    });
   });
 });