test: fix test

Author: zombieJ

src/BaseSelect/index.tsx

@@ -403,36 +403,6 @@ const BaseSelect = React.forwardRef<BaseSelectRef, BaseSelectProps>((props, ref)
     disabled || emptyListContent ? false : nextOpen,
   );
 
-  // // SSR not support Portal which means we need delay `open` for the first time render
-  // const [rendered, setRendered] = React.useState(false);
-  // useLayoutEffect(() => {
-  //   setRendered(true);
-  // }, []);
-
-  // const [innerOpen, setInnerOpen] = useControlledState<boolean>(defaultOpen || false, open);
-
-  // let mergedOpen = rendered ? innerOpen : false;
-
-  // if (disabled || (emptyListContent && mergedOpen && mode === 'combobox')) {
-  //   mergedOpen = false;
-  // }
-  // const triggerOpen = emptyListContent ? false : mergedOpen;
-
-  // const onToggleOpen = React.useCallback(
-  //   (newOpen?: boolean) => {
-  //     const nextOpen = newOpen !== undefined ? newOpen : !mergedOpen;
-
-  //     if (!disabled) {
-  //       setInnerOpen(nextOpen);
-
-  //       if (mergedOpen !== nextOpen) {
-  //         onPopupVisibleChange?.(nextOpen);
-  //       }
-  //     }
-  //   },
-  //   [disabled, mergedOpen, setInnerOpen, onPopupVisibleChange],
-  // );
-
   // ============================= Search =============================
   const tokenWithEnter = React.useMemo<boolean>(
     () => (tokenSeparators || []).some((tokenSeparator) => ['\n', '\r\n'].includes(tokenSeparator)),

src/SelectInput/Content/SingleContent.tsx

@@ -12,7 +12,7 @@ const SingleContent = React.forwardRef<HTMLInputElement, SharedContentProps>(
   ({ inputProps }, ref) => {
     const { prefixCls, searchValue, activeValue, displayValues, maxLength, mode } =
       useSelectInputContext();
-    const { triggerOpen, title: rootTitle } = useBaseProps();
+    const { triggerOpen, title: rootTitle, showSearch } = useBaseProps();
     const selectContext = React.useContext(SelectContext);
 
     const [inputChanged, setInputChanged] = React.useState(false);
@@ -26,8 +26,9 @@ const SingleContent = React.forwardRef<HTMLInputElement, SharedContentProps>(
         return activeValue;
       }
 
-      return searchValue || '';
-    }, [combobox, activeValue, inputChanged, triggerOpen, searchValue]);
+      return showSearch ? searchValue : '';
+    }, [combobox, activeValue, inputChanged, triggerOpen, searchValue, showSearch]);
+    console.log('>>>', showSearch, searchValue, mergedSearchValue);
 
     // Extract option props, excluding label and value, and handle className/style merging
     const optionProps = React.useMemo(() => {

src/hooks/useOpen.ts

@@ -1,5 +1,5 @@
 import { useControlledState, useEvent } from '@rc-component/util';
-import { useRef } from 'react';
+import { useRef, useState, useEffect } from 'react';
 
 const internalMacroTask = (fn: VoidFunction) => {
   const channel = new MessageChannel();
@@ -29,14 +29,27 @@ export type TriggerOpenType = (nextOpen?: boolean, ignoreNext?: boolean) => void
  * Otherwise use uncontrolled logic.
  * Setting `open` takes effect immediately,
  * but setting it to `false` is delayed via MessageChannel.
+ *
+ * SSR handling: During SSR, `open` is always false to avoid Portal issues.
+ * On client-side hydration, it syncs with the actual open state.
  */
 export default function useOpen(
   propOpen: boolean,
   onOpen: (nextOpen: boolean) => void,
   postOpen: (nextOpen: boolean) => boolean,
 ): [boolean, TriggerOpenType] {
+  // SSR not support Portal which means we need delay `open` for the first time render
+  const [rendered, setRendered] = useState(false);
+
+  useEffect(() => {
+    setRendered(true);
+  }, []);
+
   const [stateOpen, internalSetOpen] = useControlledState(false, propOpen);
-  const mergedOpen = postOpen(stateOpen);
+
+  // During SSR, always return false for open state
+  const ssrSafeOpen = rendered ? stateOpen : false;
+  const mergedOpen = postOpen(ssrSafeOpen);
 
   const taskIdRef = useRef(0);
   const taskLockRef = useRef(false);

tests/focus.test.tsx

@@ -66,23 +66,4 @@ describe('Select.Focus', () => {
 
     jest.useRealTimers();
   });
-
-  it('IE focus', () => {
-    jest.clearAllTimers();
-    jest.useFakeTimers();
-
-    jest.clearAllTimers();
-
-    (document.body.style as any).msTouchAction = true;
-    const { container } = render(<Select mode="tags" value="bamboo" />);
-
-    const focusFn = jest.spyOn(container.querySelector('input'), 'focus');
-
-    fireEvent.click(container.querySelector('.rc-select'));
-    jest.runAllTimers();
-
-    expect(focusFn).toHaveBeenCalled();
-
-    jest.useRealTimers();
-  });
 });

tests/placeholder.test.tsx

@@ -6,12 +6,12 @@ import { render } from '@testing-library/react';
 describe('Select placeholder', () => {
   it('when searchValue is controlled', () => {
     const { container } = render(<Select searchValue="light" placeholder="bamboo" />);
-    expect(container.querySelector('.rc-select-placeholder')).not.toHaveStyle({
-      visibility: 'hidden',
+    expect(container.querySelector('.rc-select-placeholder')).toHaveStyle({
+      visibility: 'visible',
     });
     toggleOpen(container);
     expect(container.querySelector('.rc-select-placeholder')).toHaveStyle({
-      visibility: 'hidden',
+      visibility: 'visible',
     });
   });
 
@@ -26,9 +26,9 @@ describe('Select placeholder', () => {
       <Select value={null} placeholder="bamboo" options={[{ value: null, label: 'light' }]} />,
     );
     expect(container.querySelector('.rc-select-placeholder')).toBeFalsy();
-    expect(container.querySelector('.rc-select-item').textContent).toBe('light');
+    expect(container.querySelector('.rc-select-content-value').textContent).toBe('light');
     toggleOpen(container);
-    expect(container.querySelector('.rc-select-item').textContent).toBe('light');
+    expect(container.querySelector('.rc-select-content-value').textContent).toBe('light');
   });
 
   it('should hide placeholder if force closed and showSearch with searchValue', () => {