fix: multiple 下清空时少触发一次

react-component · Nov 6, 2024 · 3f8c624 · 3f8c624
1 parent ca86130
commit 3f8c624
Show file tree

Hide file tree

Showing 2 changed files with 52 additions and 3 deletions.
diff --git a/src/BaseSelect/index.tsx b/src/BaseSelect/index.tsx
@@ -605,9 +605,15 @@ const BaseSelect = React.forwardRef<BaseSelectRef, BaseSelectProps>((props, ref)
         onSearch(mergedSearchValue, { source: 'submit' });
       } else if (mode === 'multiple') {
         // `multiple` mode only clean the search value but not trigger event
-        onSearch('', {
-          source: 'blur',
-        });
+        if (mergedShowSearch) {
+          onSearch('', {
+            source: 'effect',
+          });
+        } else {
+          onSearch('', {
+            source: 'blur',
+          });
+        }
       }
     }
 

diff --git a/tests/Select.test.tsx b/tests/Select.test.tsx
@@ -605,6 +605,49 @@ describe('Select.Basic', () => {
     jest.useRealTimers();
   });
 
+  describe('should call handleSearch twice on search and blur', () => {
+    injectRunAllTimers(jest);
+
+    beforeEach(() => {
+      jest.useFakeTimers();
+    });
+
+    afterEach(() => {
+      jest.useRealTimers();
+    });
+
+    it('multiple mode should call handleSearch twice on search and blur', async () => {
+      const handleSearch = jest.fn();
+      const { container } = render(
+        <Select showSearch onSearch={handleSearch} mode="multiple">
+          <Option value="1">1</Option>
+          <Option value="2">2</Option>
+        </Select>,
+      );
+      fireEvent.change(container.querySelector('input')!, { target: { value: '1' } });
+      // 模拟失去焦点（blur）事件
+      fireEvent.blur(container.querySelector('input'));
+      jest.runAllTimers();
+      expect(handleSearch).toHaveBeenCalledTimes(2);
+      jest.useRealTimers();
+    });
+
+    it('not multiple mode should call handleSearch twice on search and blur', async () => {
+      const handleSearch = jest.fn();
+      const { container } = render(
+        <Select showSearch onSearch={handleSearch}>
+          <Option value="1">1</Option>
+          <Option value="2">2</Option>
+        </Select>,
+      );
+      fireEvent.change(container.querySelector('input')!, { target: { value: '1' } });
+      fireEvent.blur(container.querySelector('input'));
+      jest.runAllTimers();
+      expect(handleSearch).toHaveBeenCalledTimes(2);
+      jest.useRealTimers();
+    });
+  });
+
   it('should render 0 as text properly', () => {
     const data = [
       { text: 0, value: '=0' },