fix: Should skip invalidate jsx

Author: zombieJ

src/hooks/useColumns.tsx

@@ -13,19 +13,21 @@ import {
 import { INTERNAL_COL_DEFINE } from '../utils/legacyUtil';
 
 function convertChildrenToColumns<RecordType>(children: React.ReactNode): ColumnsType<RecordType> {
-  return toArray(children).map(({ key, props }: React.ReactElement) => {
-    const { children: nodeChildren, ...restProps } = props;
-    const column = {
-      key,
-      ...restProps,
-    };
-
-    if (nodeChildren) {
-      column.children = convertChildrenToColumns(nodeChildren);
-    }
+  return toArray(children)
+    .filter(node => React.isValidElement(node))
+    .map(({ key, props }: React.ReactElement) => {
+      const { children: nodeChildren, ...restProps } = props;
+      const column = {
+        key,
+        ...restProps,
+      };
+
+      if (nodeChildren) {
+        column.children = convertChildrenToColumns(nodeChildren);
+      }
 
-    return column;
-  });
+      return column;
+    });
 }
 
 function flatColumns<RecordType>(columns: ColumnsType<RecordType>): ColumnType<RecordType>[] {

tests/FixedColumn.spec.js

@@ -148,4 +148,12 @@ describe('Table.FixedColumn', () => {
       );
     });
   });
+
+  it('ellipsis will wrap additional dom', () => {
+    const myColumns = [{ ...columns[0], ellipsis: true }];
+    const wrapper = mount(<Table columns={myColumns} data={data} />);
+
+    expect(wrapper.find('tr th').find('.rc-table-cell-content')).toHaveLength(1);
+    expect(wrapper.find('tr td').find('.rc-table-cell-content')).toHaveLength(data.length);
+  });
 });

tests/Table.spec.js

@@ -638,6 +638,7 @@ describe('Table.Basic', () => {
           <ColumnGroup title="total">
             <Column title="Name" dataIndex="name" />
           </ColumnGroup>
+          {'Invalidate Column'}
         </Table>,
       ).render(),
     ).toMatchSnapshot();