fix: List should not crash when it's not a array value (#175)

Author: zombieJ

src/List.tsx

@@ -97,8 +97,17 @@ const List: React.FunctionComponent<ListProps> = ({ name, children }) => {
             },
           };
 
+          let listValue = value || [];
+          if (!Array.isArray(listValue)) {
+            listValue = [];
+
+            if (process.env.NODE_ENV !== 'production') {
+              warning(false, `Current value of '${prefixName.join(' > ')}' is not an array type.`);
+            }
+          }
+
           return children(
-            (value as StoreValue[]).map(
+            (listValue as StoreValue[]).map(
               (__, index): ListField => {
                 let key = keyManager.keys[index];
                 if (key === undefined) {

tests/list.test.js

@@ -1,6 +1,7 @@
 import React from 'react';
 import { act } from 'react-dom/test-utils';
 import { mount } from 'enzyme';
+import { resetWarned } from 'rc-util/lib/warning';
 import Form, { Field, List } from '../src';
 import { Input } from './common/InfoField';
 import { changeValue, getField } from './common';
@@ -67,6 +68,26 @@ describe('Form.List', () => {
     });
   });
 
+  it('not crash', () => {
+    // Empty only
+    mount(
+      <Form initialValues={{ list: null }}>
+        <Form.List name="list">{() => null}</Form.List>
+      </Form>,
+    );
+
+    // Not a array
+    const errorSpy = jest.spyOn(console, 'error').mockImplementation(() => {});
+    resetWarned();
+    mount(
+      <Form initialValues={{ list: {} }}>
+        <Form.List name="list">{() => null}</Form.List>
+      </Form>,
+    );
+    expect(errorSpy).toHaveBeenCalledWith("Warning: Current value of 'list' is not an array type.");
+    errorSpy.mockRestore();
+  });
+
   it('operation', async () => {
     let operation;
     const [wrapper, getList] = generateForm((fields, opt) => {