fix: Dynamic change isLeaf from false to true should stop loading (#213)

* test: test driven

* fix: loading data logic

Author: zombieJ

src/OptionList/index.tsx

@@ -64,7 +64,7 @@ const RefOptionList = React.forwardRef<RefOptionListProps, OptionListProps>((pro
     if (loadingKeys.length) {
       loadingKeys.forEach(loadingKey => {
         const option = flattenOptions.find(opt => opt.value === loadingKey);
-        if (option.data.children) {
+        if (option.data.children || option.data.isLeaf === true) {
           setLoadingKeys(keys => keys.filter(key => key !== loadingKey));
         }
       });

tests/index.spec.tsx

@@ -1,6 +1,7 @@
 /* eslint-disable react/jsx-no-bind */
 
 import React from 'react';
+import { act } from 'react-dom/test-utils';
 import { resetWarned } from 'rc-util/lib/warning';
 import { spyElementPrototypes } from 'rc-util/lib/test/domHook';
 import { mount } from './enzyme';
@@ -603,6 +604,41 @@ describe('Cascader.Basic', () => {
       jest.runAllTimers();
       expect(loadData).toHaveBeenCalled();
     });
+
+    it('change isLeaf back to true should not loop loading', async () => {
+      const Demo = () => {
+        const [options, setOptions] = React.useState([
+          { value: 'zhejiang', label: 'Zhejiang', isLeaf: false },
+        ]);
+
+        const loadData = () => {
+          Promise.resolve().then(() => {
+            act(() => {
+              setOptions([
+                {
+                  value: 'zhejiang',
+                  label: 'Zhejiang',
+                  isLeaf: true,
+                },
+              ]);
+            });
+          });
+        };
+
+        return <Cascader options={options} loadData={loadData} open />;
+      };
+
+      const wrapper = mount(<Demo />);
+      wrapper.find('.rc-cascader-menu-item-content').first().simulate('click');
+      expect(wrapper.exists('.rc-cascader-menu-item-loading')).toBeTruthy();
+
+      for (let i = 0; i < 3; i += 1) {
+        await Promise.resolve();
+      }
+      wrapper.update();
+
+      expect(wrapper.exists('.rc-cascader-menu-item-loading')).toBeFalsy();
+    });
   });
 
   // https://github.com/ant-design/ant-design/issues/9793