From 85fd0640c5199fdd13cdd6ade098c437c9b2c12d Mon Sep 17 00:00:00 2001
From: Zack Chang <73225408+jrr997@users.noreply.github.com>
Date: Wed, 4 May 2022 20:44:37 +0800
Subject: [PATCH 01/12] test: refactor test cases of Space (#35372)

---
 .../__snapshots__/index.test.js.snap          |  4 +-
 components/space/__tests__/gap.test.js        |  8 +--
 components/space/__tests__/index.test.js      | 61 +++++++++----------
 3 files changed, 34 insertions(+), 39 deletions(-)

diff --git a/components/space/__tests__/__snapshots__/index.test.js.snap b/components/space/__tests__/__snapshots__/index.test.js.snap
index 97918ea1eb..4a9b6aa9b1 100644
--- a/components/space/__tests__/__snapshots__/index.test.js.snap
+++ b/components/space/__tests__/__snapshots__/index.test.js.snap
@@ -29,7 +29,7 @@ exports[`Space should render correct with children 1`] = `
 `;
 
 exports[`Space should render width ConfigProvider 1`] = `
-Array [
+HTMLCollection [
   <div
     class="ant-space ant-space-horizontal ant-space-align-center"
   >
@@ -91,7 +91,7 @@ Array [
 `;
 
 exports[`Space should render width rtl 1`] = `
-Array [
+HTMLCollection [
   <div
     class="ant-space ant-space-horizontal ant-space-rtl ant-space-align-center"
   >
diff --git a/components/space/__tests__/gap.test.js b/components/space/__tests__/gap.test.js
index 6b88a9afc9..f41dca6099 100644
--- a/components/space/__tests__/gap.test.js
+++ b/components/space/__tests__/gap.test.js
@@ -1,5 +1,5 @@
 import React from 'react';
-import { mount } from 'enzyme';
+import { render } from '../../../tests/utils';
 import Space from '..';
 // eslint-disable-next-line no-unused-vars
 import * as styleChecker from '../../_util/styleChecker';
@@ -12,13 +12,13 @@ jest.mock('../../_util/styleChecker', () => ({
 
 describe('flex gap', () => {
   it('should render width empty children', () => {
-    const wrapper = mount(
+    const { container } = render(
       <Space>
         <span />
         <span />
       </Space>,
     );
-    expect(wrapper.getDOMNode().style['column-gap']).toBe('8px');
-    expect(wrapper.getDOMNode().style['row-gap']).toBe('8px');
+    expect(container.querySelector('div.ant-space').style['column-gap']).toBe('8px');
+    expect(container.querySelector('div.ant-space').style['row-gap']).toBe('8px');
   });
 });
diff --git a/components/space/__tests__/index.test.js b/components/space/__tests__/index.test.js
index b811ff34a5..0af061cd00 100644
--- a/components/space/__tests__/index.test.js
+++ b/components/space/__tests__/index.test.js
@@ -1,6 +1,5 @@
 import React, { useState } from 'react';
-import { mount } from 'enzyme';
-import { act } from 'react-dom/test-utils';
+import { render, fireEvent } from '../../../tests/utils';
 import Space from '..';
 import ConfigProvider from '../../config-provider';
 import mountTest from '../../../tests/shared/mountTest';
@@ -11,13 +10,13 @@ describe('Space', () => {
   rtlTest(Space);
 
   it('should render width empty children', () => {
-    const wrapper = mount(<Space />);
+    const { container } = render(<Space />);
 
-    expect(wrapper.instance()).toBe(null);
+    expect(container.children.length).toBe(0);
   });
 
   it('should render width ConfigProvider', () => {
-    const wrapper = mount(
+    const { container } = render(
       <ConfigProvider space={{ size: 'large' }}>
         <Space>
           <span>1</span>
@@ -34,11 +33,11 @@ describe('Space', () => {
       </ConfigProvider>,
     );
 
-    expect(wrapper.render()).toMatchSnapshot();
+    expect(container.children).toMatchSnapshot();
   });
 
   it('should render width rtl', () => {
-    const wrapper = mount(
+    const { container } = render(
       <ConfigProvider direction="rtl">
         <Space>
           <span>1</span>
@@ -55,46 +54,46 @@ describe('Space', () => {
       </ConfigProvider>,
     );
 
-    expect(wrapper.render()).toMatchSnapshot();
+    expect(container.children).toMatchSnapshot();
   });
 
   it('should render width customize size', () => {
-    const wrapper = mount(
+    const { container } = render(
       <Space size={10}>
         <span>1</span>
         <span>2</span>
       </Space>,
     );
 
-    expect(wrapper.find('div.ant-space-item').at(0).prop('style').marginRight).toBe(10);
-    expect(wrapper.find('div.ant-space-item').at(1).prop('style').marginRight).toBeUndefined();
+    expect(container.querySelector('div.ant-space-item').style.marginRight).toBe('10px');
+    expect(container.querySelectorAll('div.ant-space-item')[1].style.marginRight).toBe('');
   });
 
   it('should render width size 0', () => {
-    const wrapper = mount(
+    const { container } = render(
       <Space size={NaN}>
         <span>1</span>
         <span>2</span>
       </Space>,
     );
 
-    expect(wrapper.find('div.ant-space-item').at(0).prop('style').marginRight).toBe(0);
+    expect(container.querySelector('div.ant-space-item').style.marginRight).toBe('0px');
   });
 
   it('should render vertical space width customize size', () => {
-    const wrapper = mount(
+    const { container } = render(
       <Space size={10} direction="vertical">
         <span>1</span>
         <span>2</span>
       </Space>,
     );
 
-    expect(wrapper.find('div.ant-space-item').at(0).prop('style').marginBottom).toBe(10);
-    expect(wrapper.find('div.ant-space-item').at(1).prop('style').marginBottom).toBeUndefined();
+    expect(container.querySelector('div.ant-space-item').style.marginBottom).toBe('10px');
+    expect(container.querySelectorAll('div.ant-space-item')[1].style.marginBottom).toBe('');
   });
 
   it('should render correct with children', () => {
-    const wrapper = mount(
+    const { container } = render(
       <Space>
         text1<span>text1</span>
         {/* eslint-disable-next-line react/jsx-no-useless-fragment */}
@@ -102,18 +101,18 @@ describe('Space', () => {
       </Space>,
     );
 
-    expect(wrapper.render()).toMatchSnapshot();
+    expect(container.children[0]).toMatchSnapshot();
   });
 
   it('should render with invalidElement', () => {
-    const wrapper = mount(
+    const { container } = render(
       <Space>
         text1<span>text1</span>
         text1
       </Space>,
     );
 
-    expect(wrapper.find('div.ant-space-item').length).toBe(3);
+    expect(container.querySelectorAll('div.ant-space-item').length).toBe(3);
   });
 
   it('should be keep store', () => {
@@ -144,25 +143,21 @@ describe('Space', () => {
         </Space>
       );
     }
-    const wrapper = mount(<SpaceDemo />);
+    const { container } = render(<SpaceDemo />);
 
-    expect(wrapper.find('#demo').text()).toBe('1');
+    expect(container.querySelector('#demo')).toHaveTextContent('1');
 
-    act(() => {
-      wrapper.find('#demo').simulate('click');
-    });
+    fireEvent.click(container.querySelector('#demo'));
 
-    expect(wrapper.find('#demo').text()).toBe('2');
+    expect(container.querySelector('#demo')).toHaveTextContent('2');
 
-    act(() => {
-      wrapper.find('p').simulate('click');
-    });
+    fireEvent.click(container.querySelector('p'));
 
-    expect(wrapper.find('#demo').text()).toBe('2');
+    expect(container.querySelector('#demo')).toHaveTextContent('2');
   });
 
   it('split', () => {
-    const wrapper = mount(
+    const { container } = render(
       <Space split="-">
         text1<span>text1</span>
         {/* eslint-disable-next-line react/jsx-no-useless-fragment */}
@@ -170,13 +165,13 @@ describe('Space', () => {
       </Space>,
     );
 
-    expect(wrapper.render()).toMatchSnapshot();
+    expect(container.children[0]).toMatchSnapshot();
   });
 
   // https://github.com/ant-design/ant-design/issues/35305
   it('should not throw duplicated key warning', () => {
     jest.spyOn(console, 'error').mockImplementation(() => undefined);
-    mount(
+    render(
       <Space>
         <div key="1" />
         <div />

From 44b9ab7e40060b6697845157b5555b06814c9ad7 Mon Sep 17 00:00:00 2001
From: linqiqi077 <865530219@qq.com>
Date: Wed, 4 May 2022 23:14:07 +0800
Subject: [PATCH 02/12] test: migrate part of anchor tests (#35371)

---
 components/anchor/__tests__/Anchor.test.tsx   | 436 ++++++++++++------
 .../anchor/__tests__/cached-context.test.tsx  |  20 +-
 2 files changed, 305 insertions(+), 151 deletions(-)

diff --git a/components/anchor/__tests__/Anchor.test.tsx b/components/anchor/__tests__/Anchor.test.tsx
index 1a87f8dd57..996fcf01b5 100644
--- a/components/anchor/__tests__/Anchor.test.tsx
+++ b/components/anchor/__tests__/Anchor.test.tsx
@@ -1,8 +1,7 @@
 import React from 'react';
-import { mount } from 'enzyme';
 import Anchor from '..';
 import type { InternalAnchorClass } from '../Anchor';
-import { sleep, render } from '../../../tests/utils';
+import { sleep, render, fireEvent } from '../../../tests/utils';
 
 const { Link } = Anchor;
 
@@ -48,42 +47,54 @@ describe('Anchor Render', () => {
 
   it('Anchor render perfectly', () => {
     const hash = getHashUrl();
-    const wrapper = mount(
-      <Anchor>
+    let anchorInstance: InternalAnchorClass;
+    const { container } = render(
+      <Anchor
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href={`#${hash}`} title={hash} />
       </Anchor>,
     );
 
-    wrapper.find(`a[href="#${hash}"]`).simulate('click');
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
-    anchorInstance.handleScroll();
-    expect(anchorInstance.state).not.toBe(null);
+    fireEvent.click(container.querySelector(`a[href="#${hash}"]`)!);
+    anchorInstance!.handleScroll();
+    expect(anchorInstance!.state).not.toBe(null);
   });
 
   it('Anchor render perfectly for complete href - click', () => {
     const hash = getHashUrl();
-    const wrapper = mount(
-      <Anchor>
+    let anchorInstance: InternalAnchorClass;
+    const { container } = render(
+      <Anchor
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href={`http://www.example.com/#${hash}`} title={hash} />
       </Anchor>,
     );
-    wrapper.find(`a[href="http://www.example.com/#${hash}"]`).simulate('click');
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
-    expect(anchorInstance.state.activeLink).toBe(`http://www.example.com/#${hash}`);
+    fireEvent.click(container.querySelector(`a[href="http://www.example.com/#${hash}"]`)!);
+    expect(anchorInstance!.state!.activeLink).toBe(`http://www.example.com/#${hash}`);
   });
 
   it('Anchor render perfectly for complete href - hash router', async () => {
     const root = createDiv();
     const scrollToSpy = jest.spyOn(window, 'scrollTo');
-    mount(<div id="/faq?locale=en#Q1">Q1</div>, { attachTo: root });
-    const wrapper = mount(
-      <Anchor>
+    render(<div id="/faq?locale=en#Q1">Q1</div>, { container: root });
+    let anchorInstance: InternalAnchorClass;
+    render(
+      <Anchor
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href="/#/faq?locale=en#Q1" title="Q1" />
       </Anchor>,
     );
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
-    anchorInstance.handleScrollTo('/#/faq?locale=en#Q1');
-    expect(anchorInstance.state.activeLink).toBe('/#/faq?locale=en#Q1');
+    anchorInstance!.handleScrollTo('/#/faq?locale=en#Q1');
+    expect(anchorInstance!.state.activeLink).toBe('/#/faq?locale=en#Q1');
     expect(scrollToSpy).not.toHaveBeenCalled();
     await sleep(1000);
     expect(scrollToSpy).toHaveBeenCalled();
@@ -92,32 +103,39 @@ describe('Anchor Render', () => {
   it('Anchor render perfectly for complete href - scroll', () => {
     const hash = getHashUrl();
     const root = createDiv();
-    mount(<div id={hash}>Hello</div>, { attachTo: root });
-    const wrapper = mount(
-      <Anchor>
+    render(<div id={hash}>Hello</div>, { container: root });
+    let anchorInstance: InternalAnchorClass;
+    render(
+      <Anchor
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href={`http://www.example.com/#${hash}`} title={hash} />
       </Anchor>,
     );
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
-
-    anchorInstance.handleScroll();
-    expect(anchorInstance.state.activeLink).toBe(`http://www.example.com/#${hash}`);
+    anchorInstance!.handleScroll();
+    expect(anchorInstance!.state!.activeLink).toBe(`http://www.example.com/#${hash}`);
   });
 
   it('Anchor render perfectly for complete href - scrollTo', async () => {
     const hash = getHashUrl();
     const scrollToSpy = jest.spyOn(window, 'scrollTo');
     const root = createDiv();
-    mount(<div id={`#${hash}`}>Hello</div>, { attachTo: root });
-    const wrapper = mount(
-      <Anchor>
+    render(<div id={`#${hash}`}>Hello</div>, { container: root });
+    let anchorInstance: InternalAnchorClass;
+    render(
+      <Anchor
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href={`##${hash}`} title={hash} />
       </Anchor>,
     );
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
 
-    anchorInstance.handleScrollTo(`##${hash}`);
-    expect(anchorInstance.state.activeLink).toBe(`##${hash}`);
+    anchorInstance!.handleScrollTo(`##${hash}`);
+    expect(anchorInstance!.state.activeLink).toBe(`##${hash}`);
     const calls = scrollToSpy.mock.calls.length;
     await sleep(1000);
     expect(scrollToSpy.mock.calls.length).toBeGreaterThan(calls);
@@ -125,15 +143,19 @@ describe('Anchor Render', () => {
 
   it('should remove listener when unmount', async () => {
     const hash = getHashUrl();
-    const wrapper = mount(
-      <Anchor>
+    let anchorInstance: InternalAnchorClass;
+    const { unmount } = render(
+      <Anchor
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href={`#${hash}`} title={hash} />
       </Anchor>,
     );
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
 
-    const removeListenerSpy = jest.spyOn((anchorInstance as any).scrollEvent, 'remove');
-    wrapper.unmount();
+    const removeListenerSpy = jest.spyOn((anchorInstance! as any).scrollEvent, 'remove');
+    unmount();
     expect(removeListenerSpy).toHaveBeenCalled();
   });
 
@@ -154,23 +176,27 @@ describe('Anchor Render', () => {
 
   it('should update links when link href update', async () => {
     const hash = getHashUrl();
+    let anchorInstance: InternalAnchorClass;
     function AnchorUpdate({ href }: { href: string }) {
       return (
-        <Anchor>
+        <Anchor
+          ref={node => {
+            anchorInstance = node as InternalAnchorClass;
+          }}
+        >
           <Link href={href} title={hash} />
         </Anchor>
       );
     }
-    const wrapper = mount(<AnchorUpdate href={`#${hash}`} />);
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
+    const { rerender } = render(<AnchorUpdate href={`#${hash}`} />);
 
-    if (anchorInstance == null) {
+    if (anchorInstance! == null) {
       throw new Error('anchorInstance should not be null');
     }
 
-    expect((anchorInstance as any).links).toEqual([`#${hash}`]);
-    wrapper.setProps({ href: `#${hash}_1` });
-    expect((anchorInstance as any).links).toEqual([`#${hash}_1`]);
+    expect((anchorInstance as any)!.links).toEqual([`#${hash}`]);
+    rerender(<AnchorUpdate href={`#${hash}_1`} />);
+    expect((anchorInstance as any)!.links).toEqual([`#${hash}_1`]);
   });
 
   it('Anchor onClick event', () => {
@@ -187,18 +213,20 @@ describe('Anchor Render', () => {
 
     const href = `#${hash}`;
     const title = hash;
-
-    const wrapper = mount(
-      <Anchor onClick={handleClick}>
+    let anchorInstance: InternalAnchorClass;
+    const { container } = render(
+      <Anchor
+        onClick={handleClick}
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href={href} title={title} />
       </Anchor>,
     );
 
-    wrapper.find(`a[href="${href}"]`).simulate('click');
-
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
-
-    anchorInstance.handleScroll();
+    fireEvent.click(container.querySelector(`a[href="${href}"]`)!);
+    anchorInstance!.handleScroll();
     expect(event).not.toBe(undefined);
     expect(link).toEqual({ href, title });
   });
@@ -206,20 +234,28 @@ describe('Anchor Render', () => {
   it('Different function returns the same DOM', async () => {
     const hash = getHashUrl();
     const root = createDiv();
-    mount(<div id={hash}>Hello</div>, { attachTo: root });
+    render(<div id={hash}>Hello</div>, { container: root });
     const getContainerA = createGetContainer(hash);
     const getContainerB = createGetContainer(hash);
-
-    const wrapper = mount(
-      <Anchor getContainer={getContainerA}>
+    let anchorInstance: InternalAnchorClass;
+    const { rerender } = render(
+      <Anchor
+        getContainer={getContainerA}
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href={`#${hash}`} title={hash} />
       </Anchor>,
     );
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
 
-    const removeListenerSpy = jest.spyOn((anchorInstance as any).scrollEvent, 'remove');
+    const removeListenerSpy = jest.spyOn((anchorInstance! as any).scrollEvent, 'remove');
     await sleep(1000);
-    wrapper.setProps({ getContainer: getContainerB });
+    rerender(
+      <Anchor getContainer={getContainerB}>
+        <Link href={`#${hash}`} title={hash} />
+      </Anchor>,
+    );
     expect(removeListenerSpy).not.toHaveBeenCalled();
   });
 
@@ -227,57 +263,73 @@ describe('Anchor Render', () => {
     const hash1 = getHashUrl();
     const hash2 = getHashUrl();
     const root = createDiv();
-    mount(
+    render(
       <div>
         <div id={hash1}>Hello</div>
         <div id={hash2}>World</div>
       </div>,
-      { attachTo: root },
+      { container: root },
     );
     const getContainerA = createGetContainer(hash1);
     const getContainerB = createGetContainer(hash2);
-    const wrapper = mount(
-      <Anchor getContainer={getContainerA}>
+    let anchorInstance: InternalAnchorClass;
+    const { rerender } = render(
+      <Anchor
+        getContainer={getContainerA}
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href={`#${hash1}`} title={hash1} />
         <Link href={`#${hash2}`} title={hash2} />
       </Anchor>,
     );
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
 
-    const removeListenerSpy = jest.spyOn((anchorInstance as any).scrollEvent, 'remove');
+    const removeListenerSpy = jest.spyOn((anchorInstance! as any).scrollEvent, 'remove');
     expect(removeListenerSpy).not.toHaveBeenCalled();
     await sleep(1000);
-    wrapper.setProps({ getContainer: getContainerB });
+    rerender(
+      <Anchor getContainer={getContainerB}>
+        <Link href={`#${hash1}`} title={hash1} />
+        <Link href={`#${hash2}`} title={hash2} />
+      </Anchor>,
+    );
     expect(removeListenerSpy).toHaveBeenCalled();
   });
 
   it('Same function returns the same DOM', () => {
     const hash = getHashUrl();
     const root = createDiv();
-    mount(<div id={hash}>Hello</div>, { attachTo: root });
+    render(<div id={hash}>Hello</div>, { container: root });
     const getContainer = createGetContainer(hash);
-    const wrapper = mount(
-      <Anchor getContainer={getContainer}>
+    let anchorInstance: InternalAnchorClass;
+    const { container } = render(
+      <Anchor
+        getContainer={getContainer}
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href={`#${hash}`} title={hash} />
       </Anchor>,
     );
-    wrapper.find(`a[href="#${hash}"]`).simulate('click');
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
 
-    (anchorInstance as any).handleScroll();
-    expect(anchorInstance.state).not.toBe(null);
+    fireEvent.click(container.querySelector(`a[href="#${hash}"]`)!);
+
+    anchorInstance!.handleScroll();
+    expect(anchorInstance!.state).not.toBe(null);
   });
 
   it('Same function returns different DOM', async () => {
     const hash1 = getHashUrl();
     const hash2 = getHashUrl();
     const root = createDiv();
-    mount(
+    render(
       <div>
         <div id={hash1}>Hello</div>
         <div id={hash2}>World</div>
       </div>,
-      { attachTo: root },
+      { container: root },
     );
     const holdContainer = {
       container: document.getElementById(hash1),
@@ -288,19 +340,28 @@ describe('Anchor Render', () => {
       }
       return holdContainer.container;
     };
-    const wrapper = mount(
+    let anchorInstance: InternalAnchorClass;
+    const { rerender } = render(
+      <Anchor
+        getContainer={getContainer}
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
+        <Link href={`#${hash1}`} title={hash1} />
+        <Link href={`#${hash2}`} title={hash2} />
+      </Anchor>,
+    );
+    const removeListenerSpy = jest.spyOn((anchorInstance! as any).scrollEvent, 'remove');
+    expect(removeListenerSpy).not.toHaveBeenCalled();
+    await sleep(1000);
+    holdContainer.container = document.getElementById(hash2);
+    rerender(
       <Anchor getContainer={getContainer}>
         <Link href={`#${hash1}`} title={hash1} />
         <Link href={`#${hash2}`} title={hash2} />
       </Anchor>,
     );
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
-
-    const removeListenerSpy = jest.spyOn((anchorInstance as any).scrollEvent, 'remove');
-    expect(removeListenerSpy).not.toHaveBeenCalled();
-    await sleep(1000);
-    holdContainer.container = document.getElementById(hash2);
-    wrapper.setProps({ 'data-only-trigger-re-render': true });
     expect(removeListenerSpy).toHaveBeenCalled();
   });
 
@@ -323,27 +384,45 @@ describe('Anchor Render', () => {
 
     const scrollToSpy = jest.spyOn(window, 'scrollTo');
     const root = createDiv();
-    mount(<h1 id={hash}>Hello</h1>, { attachTo: root });
-    const wrapper = mount(
-      <Anchor>
+    render(<h1 id={hash}>Hello</h1>, { container: root });
+    let anchorInstance: InternalAnchorClass;
+    const { rerender } = render(
+      <Anchor
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href={`#${hash}`} title={hash} />
       </Anchor>,
     );
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
 
-    anchorInstance.handleScrollTo(`#${hash}`);
+    const setProps = (props: Record<string, any>) =>
+      rerender(
+        <Anchor
+          ref={node => {
+            anchorInstance = node as InternalAnchorClass;
+          }}
+          {...props}
+        >
+          <Link href={`#${hash}`} title={hash} />
+        </Anchor>,
+      );
+
+    anchorInstance!.handleScrollTo(`#${hash}`);
     await sleep(30);
     expect(scrollToSpy).toHaveBeenLastCalledWith(0, 1000);
     dateNowMock = dataNowMockFn();
 
-    wrapper.setProps({ offsetTop: 100 });
-    anchorInstance.handleScrollTo(`#${hash}`);
+    setProps({ offsetTop: 100 });
+
+    anchorInstance!.handleScrollTo(`#${hash}`);
     await sleep(30);
     expect(scrollToSpy).toHaveBeenLastCalledWith(0, 900);
     dateNowMock = dataNowMockFn();
 
-    wrapper.setProps({ targetOffset: 200 });
-    anchorInstance.handleScrollTo(`#${hash}`);
+    setProps({ targetOffset: 200 });
+
+    anchorInstance!.handleScrollTo(`#${hash}`);
     await sleep(30);
     expect(scrollToSpy).toHaveBeenLastCalledWith(0, 800);
 
@@ -370,27 +449,43 @@ describe('Anchor Render', () => {
 
     const scrollToSpy = jest.spyOn(window, 'scrollTo');
     const root = createDiv();
-    mount(<h1 id={hash}>Hello</h1>, { attachTo: root });
-    const wrapper = mount(
-      <Anchor>
+    render(<h1 id={hash}>Hello</h1>, { container: root });
+    let anchorInstance: InternalAnchorClass;
+    const { rerender } = render(
+      <Anchor
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href={`#${hash}`} title={hash} />
       </Anchor>,
     );
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
 
-    anchorInstance.handleScrollTo(`#${hash}`);
+    const setProps = (props: Record<string, any>) =>
+      rerender(
+        <Anchor
+          ref={node => {
+            anchorInstance = node as InternalAnchorClass;
+          }}
+          {...props}
+        >
+          <Link href={`#${hash}`} title={hash} />
+        </Anchor>,
+      );
+
+    anchorInstance!.handleScrollTo(`#${hash}`);
     await sleep(30);
     expect(scrollToSpy).toHaveBeenLastCalledWith(0, 1000);
     dateNowMock = dataNowMockFn();
 
-    wrapper.setProps({ offsetTop: 100 });
-    anchorInstance.handleScrollTo(`#${hash}`);
+    setProps({ offsetTop: 100 });
+    anchorInstance!.handleScrollTo(`#${hash}`);
     await sleep(30);
     expect(scrollToSpy).toHaveBeenLastCalledWith(0, 900);
     dateNowMock = dataNowMockFn();
 
-    wrapper.setProps({ targetOffset: 200 });
-    anchorInstance.handleScrollTo(`#${hash}`);
+    setProps({ targetOffset: 200 });
+    anchorInstance!.handleScrollTo(`#${hash}`);
     await sleep(30);
     expect(scrollToSpy).toHaveBeenLastCalledWith(0, 800);
 
@@ -401,33 +496,44 @@ describe('Anchor Render', () => {
     const hash1 = getHashUrl();
     const hash2 = getHashUrl();
     const onChange = jest.fn();
-    const wrapper = mount(
-      <Anchor onChange={onChange}>
+    let anchorInstance: InternalAnchorClass;
+    render(
+      <Anchor
+        onChange={onChange}
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href={`#${hash1}`} title={hash1} />
         <Link href={`#${hash2}`} title={hash2} />
       </Anchor>,
+      // https://github.com/testing-library/react-testing-library/releases/tag/v13.0.0
+      // @ts-ignore
+      { legacyRoot: true },
     );
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
 
     expect(onChange).toHaveBeenCalledTimes(1);
-    anchorInstance.handleScrollTo(hash2);
+    anchorInstance!.handleScrollTo(hash2);
     expect(onChange).toHaveBeenCalledTimes(2);
     expect(onChange).toHaveBeenCalledWith(hash2);
   });
 
   it('invalid hash', async () => {
-    const wrapper = mount(
-      <Anchor>
+    let anchorInstance: InternalAnchorClass;
+    const { container } = render(
+      <Anchor
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href="notexsited" title="title" />
       </Anchor>,
     );
 
-    wrapper.find(`a[href="notexsited"]`).simulate('click');
+    fireEvent.click(container.querySelector(`a[href="notexsited"]`)!);
 
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
-
-    anchorInstance.handleScrollTo('notexsited');
-    expect(anchorInstance.state).not.toBe(null);
+    anchorInstance!.handleScrollTo('notexsited');
+    expect(anchorInstance!.state).not.toBe(null);
   });
 
   it('test edge case when getBoundingClientRect return zero size', async () => {
@@ -454,27 +560,42 @@ describe('Anchor Render', () => {
 
     const scrollToSpy = jest.spyOn(window, 'scrollTo');
     const root = createDiv();
-    mount(<h1 id={hash}>Hello</h1>, { attachTo: root });
-    const wrapper = mount(
-      <Anchor>
+    render(<h1 id={hash}>Hello</h1>, { container: root });
+    let anchorInstance: InternalAnchorClass;
+    const { rerender } = render(
+      <Anchor
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href={`#${hash}`} title={hash} />
       </Anchor>,
     );
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
 
-    anchorInstance.handleScrollTo(`#${hash}`);
+    const setProps = (props: Record<string, any>) =>
+      rerender(
+        <Anchor
+          ref={node => {
+            anchorInstance = node as InternalAnchorClass;
+          }}
+          {...props}
+        >
+          <Link href={`#${hash}`} title={hash} />
+        </Anchor>,
+      );
+    anchorInstance!.handleScrollTo(`#${hash}`);
     await sleep(30);
     expect(scrollToSpy).toHaveBeenLastCalledWith(0, 1000);
     dateNowMock = dataNowMockFn();
 
-    wrapper.setProps({ offsetTop: 100 });
-    anchorInstance.handleScrollTo(`#${hash}`);
+    setProps({ offsetTop: 100 });
+    anchorInstance!.handleScrollTo(`#${hash}`);
     await sleep(30);
     expect(scrollToSpy).toHaveBeenLastCalledWith(0, 900);
     dateNowMock = dataNowMockFn();
 
-    wrapper.setProps({ targetOffset: 200 });
-    anchorInstance.handleScrollTo(`#${hash}`);
+    setProps({ targetOffset: 200 });
+    anchorInstance!.handleScrollTo(`#${hash}`);
     await sleep(30);
     expect(scrollToSpy).toHaveBeenLastCalledWith(0, 800);
 
@@ -505,27 +626,44 @@ describe('Anchor Render', () => {
 
     const scrollToSpy = jest.spyOn(window, 'scrollTo');
     const root = createDiv();
-    mount(<h1 id={hash}>Hello</h1>, { attachTo: root });
-    const wrapper = mount(
-      <Anchor getContainer={() => document.body}>
+    render(<h1 id={hash}>Hello</h1>, { container: root });
+
+    let anchorInstance: InternalAnchorClass;
+    const { rerender } = render(
+      <Anchor
+        getContainer={() => document.body}
+        ref={node => {
+          anchorInstance = node as InternalAnchorClass;
+        }}
+      >
         <Link href={`#${hash}`} title={hash} />
       </Anchor>,
     );
-    const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
 
-    anchorInstance.handleScrollTo(`#${hash}`);
+    const setProps = (props: Record<string, any>) =>
+      rerender(
+        <Anchor
+          getContainer={() => document.body}
+          ref={node => {
+            anchorInstance = node as InternalAnchorClass;
+          }}
+          {...props}
+        >
+          <Link href={`#${hash}`} title={hash} />
+        </Anchor>,
+      );
+    anchorInstance!.handleScrollTo(`#${hash}`);
     await sleep(30);
     expect(scrollToSpy).toHaveBeenLastCalledWith(0, 800);
     dateNowMock = dataNowMockFn();
 
-    wrapper.setProps({ offsetTop: 100 });
-    anchorInstance.handleScrollTo(`#${hash}`);
+    setProps({ offsetTop: 100 });
+    anchorInstance!.handleScrollTo(`#${hash}`);
     await sleep(30);
     expect(scrollToSpy).toHaveBeenLastCalledWith(0, 800);
     dateNowMock = dataNowMockFn();
-
-    wrapper.setProps({ targetOffset: 200 });
-    anchorInstance.handleScrollTo(`#${hash}`);
+    setProps({ targetOffset: 200 });
+    anchorInstance!.handleScrollTo(`#${hash}`);
     await sleep(30);
     expect(scrollToSpy).toHaveBeenLastCalledWith(0, 800);
 
@@ -537,15 +675,20 @@ describe('Anchor Render', () => {
       const hash1 = getHashUrl();
       const hash2 = getHashUrl();
       const getCurrentAnchor = () => `#${hash2}`;
-      const wrapper = mount(
-        <Anchor getCurrentAnchor={getCurrentAnchor}>
+      let anchorInstance: InternalAnchorClass;
+      render(
+        <Anchor
+          getCurrentAnchor={getCurrentAnchor}
+          ref={node => {
+            anchorInstance = node as InternalAnchorClass;
+          }}
+        >
           <Link href={`#${hash1}`} title={hash1} />
           <Link href={`#${hash2}`} title={hash2} />
         </Anchor>,
       );
-      const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
 
-      expect(anchorInstance.state.activeLink).toBe(`#${hash2}`);
+      expect(anchorInstance!.state.activeLink).toBe(`#${hash2}`);
     });
 
     // https://github.com/ant-design/ant-design/issues/30584
@@ -553,16 +696,25 @@ describe('Anchor Render', () => {
       const hash1 = getHashUrl();
       const hash2 = getHashUrl();
       const onChange = jest.fn();
-      const wrapper = mount(
-        <Anchor onChange={onChange} getCurrentAnchor={() => hash1}>
+      let anchorInstance: InternalAnchorClass;
+      render(
+        <Anchor
+          onChange={onChange}
+          getCurrentAnchor={() => hash1}
+          ref={node => {
+            anchorInstance = node as InternalAnchorClass;
+          }}
+        >
           <Link href={`#${hash1}`} title={hash1} />
           <Link href={`#${hash2}`} title={hash2} />
         </Anchor>,
+        // https://github.com/testing-library/react-testing-library/releases/tag/v13.0.0
+        // @ts-ignore
+        { legacyRoot: true },
       );
-      const anchorInstance = wrapper.find('Anchor').last().instance() as any as InternalAnchorClass;
 
       expect(onChange).toHaveBeenCalledTimes(1);
-      anchorInstance.handleScrollTo(hash2);
+      anchorInstance!.handleScrollTo(hash2);
       expect(onChange).toHaveBeenCalledTimes(2);
       expect(onChange).toHaveBeenCalledWith(hash2);
     });
@@ -572,16 +724,16 @@ describe('Anchor Render', () => {
       const hash1 = getHashUrl();
       const hash2 = getHashUrl();
       const getCurrentAnchor = jest.fn();
-      const wrapper = mount(
+      const { container } = render(
         <Anchor getCurrentAnchor={getCurrentAnchor}>
           <Link href={`#${hash1}`} title={hash1} />
           <Link href={`#${hash2}`} title={hash2} />
         </Anchor>,
       );
 
-      wrapper.find(`a[href="#${hash1}"]`).simulate('click');
+      fireEvent.click(container.querySelector(`a[href="#${hash1}"]`)!);
       expect(getCurrentAnchor).toHaveBeenCalledWith(`#${hash1}`);
-      wrapper.find(`a[href="#${hash2}"]`).simulate('click');
+      fireEvent.click(container.querySelector(`a[href="#${hash2}"]`)!);
       expect(getCurrentAnchor).toHaveBeenCalledWith(`#${hash2}`);
     });
   });
diff --git a/components/anchor/__tests__/cached-context.test.tsx b/components/anchor/__tests__/cached-context.test.tsx
index 0fa0db6f93..27c3f683df 100644
--- a/components/anchor/__tests__/cached-context.test.tsx
+++ b/components/anchor/__tests__/cached-context.test.tsx
@@ -1,7 +1,7 @@
 import React, { memo, useState, useRef, useContext } from 'react';
-import { mount } from 'enzyme';
 import Anchor from '../Anchor';
 import AnchorContext from '../context';
+import { getNodeText, render, fireEvent } from '../../../tests/utils';
 
 // we use'memo' here in order to only render inner component while context changed.
 const CacheInner = memo(() => {
@@ -38,14 +38,16 @@ const CacheOuter = () => {
 };
 
 it("Rendering on Anchor without changed AnchorContext won't trigger rendering on child component.", () => {
-  const wrapper = mount(<CacheOuter />);
-  const childCount = wrapper.find('#child_count').text();
-  wrapper.find('#parent_btn').at(0).simulate('click');
-  expect(wrapper.find('#parent_count').text()).toBe('2');
+  const { container } = render(<CacheOuter />);
+  const childCount = getNodeText(container.querySelector('#child_count')!);
+
+  fireEvent.click(container.querySelector('#parent_btn')!);
+
+  expect(getNodeText(container.querySelector('#parent_count')!)).toBe('2');
   // child component won't rerender
-  expect(wrapper.find('#child_count').text()).toBe(childCount);
-  wrapper.find('#parent_btn').at(0).simulate('click');
-  expect(wrapper.find('#parent_count').text()).toBe('3');
+  expect(getNodeText(container.querySelector('#child_count')!)).toBe(childCount);
+  fireEvent.click(container.querySelector('#parent_btn')!);
+  expect(getNodeText(container.querySelector('#parent_count')!)).toBe('3');
   // child component won't rerender
-  expect(wrapper.find('#child_count').text()).toBe(childCount);
+  expect(getNodeText(container.querySelector('#child_count')!)).toBe(childCount);
 });

From 50d3d99a78bedd8e84bc1466f7a0c354344178ed Mon Sep 17 00:00:00 2001
From: TrickyPi <33021497+TrickyPi@users.noreply.github.com>
Date: Thu, 5 May 2022 10:25:22 +0800
Subject: [PATCH 03/12] test: migrate part of checkbox tests (#35354)

---
 .../checkbox/__tests__/checkbox.test.js       |  12 +-
 components/checkbox/__tests__/group.test.js   | 109 +++++++++---------
 2 files changed, 62 insertions(+), 59 deletions(-)

diff --git a/components/checkbox/__tests__/checkbox.test.js b/components/checkbox/__tests__/checkbox.test.js
index 274ef9c6c0..f7a4bf89f6 100644
--- a/components/checkbox/__tests__/checkbox.test.js
+++ b/components/checkbox/__tests__/checkbox.test.js
@@ -1,5 +1,5 @@
 import React from 'react';
-import { mount } from 'enzyme';
+import { render, fireEvent } from '../../../tests/utils';
 import Checkbox from '..';
 import focusTest from '../../../tests/shared/focusTest';
 import { resetWarned } from '../../_util/devWarning';
@@ -15,12 +15,14 @@ describe('Checkbox', () => {
     const onMouseEnter = jest.fn();
     const onMouseLeave = jest.fn();
 
-    const wrapper = mount(<Checkbox onMouseEnter={onMouseEnter} onMouseLeave={onMouseLeave} />);
+    const { container } = render(
+      <Checkbox onMouseEnter={onMouseEnter} onMouseLeave={onMouseLeave} />,
+    );
 
-    wrapper.find('label').simulate('mouseenter');
+    fireEvent.mouseEnter(container.querySelector('label'));
     expect(onMouseEnter).toHaveBeenCalled();
 
-    wrapper.find('label').simulate('mouseleave');
+    fireEvent.mouseLeave(container.querySelector('label'));
     expect(onMouseLeave).toHaveBeenCalled();
   });
 
@@ -28,7 +30,7 @@ describe('Checkbox', () => {
     resetWarned();
 
     const errorSpy = jest.spyOn(console, 'error').mockImplementation(() => {});
-    mount(<Checkbox value />);
+    render(<Checkbox value />);
     expect(errorSpy).toHaveBeenCalledWith(
       'Warning: [antd: Checkbox] `value` is not a valid prop, do you mean `checked`?',
     );
diff --git a/components/checkbox/__tests__/group.test.js b/components/checkbox/__tests__/group.test.js
index b16a84d665..ff10263105 100644
--- a/components/checkbox/__tests__/group.test.js
+++ b/components/checkbox/__tests__/group.test.js
@@ -1,5 +1,4 @@
 import React, { useState } from 'react';
-import { mount } from 'enzyme';
 import Collapse from '../../collapse';
 import Table from '../../table';
 import Checkbox from '../index';
@@ -14,16 +13,16 @@ describe('CheckboxGroup', () => {
 
   it('should work basically', () => {
     const onChange = jest.fn();
-    const wrapper = mount(
+    const { container } = render(
       <Checkbox.Group options={['Apple', 'Pear', 'Orange']} onChange={onChange} />,
     );
-    wrapper.find('.ant-checkbox-input').at(0).simulate('change');
+    fireEvent.click(container.querySelectorAll('.ant-checkbox-input')[0]);
     expect(onChange).toHaveBeenCalledWith(['Apple']);
-    wrapper.find('.ant-checkbox-input').at(1).simulate('change');
+    fireEvent.click(container.querySelectorAll('.ant-checkbox-input')[1]);
     expect(onChange).toHaveBeenCalledWith(['Apple', 'Pear']);
-    wrapper.find('.ant-checkbox-input').at(2).simulate('change');
+    fireEvent.click(container.querySelectorAll('.ant-checkbox-input')[2]);
     expect(onChange).toHaveBeenCalledWith(['Apple', 'Pear', 'Orange']);
-    wrapper.find('.ant-checkbox-input').at(1).simulate('change');
+    fireEvent.click(container.querySelectorAll('.ant-checkbox-input')[1]);
     expect(onChange).toHaveBeenCalledWith(['Apple', 'Orange']);
   });
 
@@ -35,12 +34,12 @@ describe('CheckboxGroup', () => {
       { label: 'Pear', value: 'Pear' },
     ];
 
-    const groupWrapper = mount(
+    const { container } = render(
       <Checkbox.Group options={options} onChange={onChangeGroup} disabled />,
     );
-    groupWrapper.find('.ant-checkbox-input').at(0).simulate('change');
+    fireEvent.click(container.querySelectorAll('.ant-checkbox-input')[0]);
     expect(onChangeGroup).not.toHaveBeenCalled();
-    groupWrapper.find('.ant-checkbox-input').at(1).simulate('change');
+    fireEvent.click(container.querySelectorAll('.ant-checkbox-input')[1]);
     expect(onChangeGroup).not.toHaveBeenCalled();
   });
 
@@ -52,17 +51,17 @@ describe('CheckboxGroup', () => {
       { label: 'Orange', value: 'Orange', disabled: true },
     ];
 
-    const groupWrapper = mount(<Checkbox.Group options={options} onChange={onChangeGroup} />);
-    groupWrapper.find('.ant-checkbox-input').at(0).simulate('change');
+    const { container } = render(<Checkbox.Group options={options} onChange={onChangeGroup} />);
+    fireEvent.click(container.querySelectorAll('.ant-checkbox-input')[0]);
     expect(onChangeGroup).toHaveBeenCalledWith(['Apple']);
-    groupWrapper.find('.ant-checkbox-input').at(1).simulate('change');
+    fireEvent.click(container.querySelectorAll('.ant-checkbox-input')[1]);
     expect(onChangeGroup).toHaveBeenCalledWith(['Apple']);
   });
 
   it('all children should have a name property', () => {
-    const wrapper = mount(<Checkbox.Group name="checkboxgroup" options={['Yes', 'No']} />);
-    wrapper.find('input[type="checkbox"]').forEach(el => {
-      expect(el.props().name).toEqual('checkboxgroup');
+    const { container } = render(<Checkbox.Group name="checkboxgroup" options={['Yes', 'No']} />);
+    [...container.querySelectorAll('input[type="checkbox"]')].forEach(el => {
+      expect(el.getAttribute('name')).toEqual('checkboxgroup');
     });
   });
 
@@ -72,9 +71,9 @@ describe('CheckboxGroup', () => {
       { label: 'Orange', value: 'Orange', style: { fontSize: 12 } },
     ];
 
-    const wrapper = mount(<Checkbox.Group prefixCls="my-checkbox" options={options} />);
+    const { container } = render(<Checkbox.Group prefixCls="my-checkbox" options={options} />);
 
-    expect(wrapper.render()).toMatchSnapshot();
+    expect(container.firstChild).toMatchSnapshot();
   });
 
   it('should be controlled by value', () => {
@@ -82,23 +81,22 @@ describe('CheckboxGroup', () => {
       { label: 'Apple', value: 'Apple' },
       { label: 'Orange', value: 'Orange' },
     ];
-
-    const wrapper = mount(<Checkbox.Group options={options} />);
-    expect(wrapper.find('.ant-checkbox-checked').length).toBe(0);
-    wrapper.setProps({ value: ['Apple'] });
-    wrapper.update();
-    expect(wrapper.find('.ant-checkbox-checked').length).toBe(1);
+    const renderCheckbox = props => <Checkbox.Group {...props} />;
+    const { container, rerender } = render(renderCheckbox({ options }));
+    expect(container.querySelectorAll('.ant-checkbox-checked').length).toBe(0);
+    rerender(renderCheckbox({ options, value: 'Apple' }));
+    expect(container.querySelectorAll('.ant-checkbox-checked').length).toBe(1);
   });
 
   // https://github.com/ant-design/ant-design/issues/12642
   it('should trigger onChange in sub Checkbox', () => {
     const onChange = jest.fn();
-    const wrapper = mount(
+    const { container } = render(
       <Checkbox.Group>
         <Checkbox value="my" onChange={onChange} />
       </Checkbox.Group>,
     );
-    wrapper.find('.ant-checkbox-input').at(0).simulate('change');
+    fireEvent.click(container.querySelectorAll('.ant-checkbox-input')[0]);
     expect(onChange).toHaveBeenCalled();
     expect(onChange.mock.calls[0][0].target.value).toEqual('my');
   });
@@ -143,7 +141,7 @@ describe('CheckboxGroup', () => {
   // https://github.com/ant-design/ant-design/issues/17297
   it('onChange should keep the order of the original values', () => {
     const onChange = jest.fn();
-    const wrapper = mount(
+    const { container } = render(
       <Checkbox.Group onChange={onChange}>
         <Checkbox key={1} value={1} />
         <Checkbox key={2} value={2} />
@@ -151,20 +149,19 @@ describe('CheckboxGroup', () => {
         <Checkbox key={4} value={4} />
       </Checkbox.Group>,
     );
-
-    wrapper.find('.ant-checkbox-input').at(0).simulate('change');
+    fireEvent.click(container.querySelectorAll('.ant-checkbox-input')[0]);
     expect(onChange).toHaveBeenCalledWith([1]);
-    wrapper.find('.ant-checkbox-input').at(1).simulate('change');
+    fireEvent.click(container.querySelectorAll('.ant-checkbox-input')[1]);
     expect(onChange).toHaveBeenCalledWith([1, 2]);
-    wrapper.find('.ant-checkbox-input').at(0).simulate('change');
+    fireEvent.click(container.querySelectorAll('.ant-checkbox-input')[0]);
     expect(onChange).toHaveBeenCalledWith([2]);
-    wrapper.find('.ant-checkbox-input').at(0).simulate('change');
+    fireEvent.click(container.querySelectorAll('.ant-checkbox-input')[0]);
     expect(onChange).toHaveBeenCalledWith([1, 2]);
   });
 
   // https://github.com/ant-design/ant-design/issues/21134
   it('should work when checkbox is wrapped by other components', () => {
-    const wrapper = mount(
+    const { container } = render(
       <Checkbox.Group>
         <Collapse bordered={false}>
           <Collapse.Panel header="test panel">
@@ -175,28 +172,31 @@ describe('CheckboxGroup', () => {
         </Collapse>
       </Checkbox.Group>,
     );
-    wrapper.find('.ant-collapse-item').at(0).find('.ant-collapse-header').simulate('click');
-    wrapper.find('.ant-checkbox-input').at(0).simulate('change');
-    expect(wrapper.find('.ant-checkbox-checked').length).toBe(1);
-    wrapper.find('.ant-checkbox-input').at(0).simulate('change');
-    expect(wrapper.find('.ant-checkbox-checked').length).toBe(0);
+
+    fireEvent.click(
+      container.querySelector('.ant-collapse-item').querySelector('.ant-collapse-header'),
+    );
+    fireEvent.click(container.querySelector('.ant-checkbox-input'));
+    expect(container.querySelectorAll('.ant-checkbox-checked').length).toBe(1);
+    fireEvent.click(container.querySelector('.ant-checkbox-input'));
+    expect(container.querySelectorAll('.ant-checkbox-checked').length).toBe(0);
   });
 
   it('skipGroup', () => {
     const onChange = jest.fn();
-    const wrapper = mount(
+    const { container } = render(
       <Checkbox.Group onChange={onChange}>
         <Checkbox value={1} />
         <Checkbox value={2} skipGroup />
       </Checkbox.Group>,
     );
-    wrapper.find('.ant-checkbox-input').at(1).simulate('change');
+    fireEvent.click(container.querySelectorAll('.ant-checkbox-input')[1]);
     expect(onChange).not.toHaveBeenCalled();
   });
 
   it('Table rowSelection', () => {
     const onChange = jest.fn();
-    const wrapper = mount(
+    const { container } = render(
       <Checkbox.Group onChange={onChange}>
         <Table
           dataSource={[{ key: 1, value: '1' }]}
@@ -205,27 +205,32 @@ describe('CheckboxGroup', () => {
         />
       </Checkbox.Group>,
     );
-    wrapper.find('.ant-checkbox-input').at(1).simulate('change');
+    fireEvent.click(container.querySelectorAll('.ant-checkbox-input')[1]);
     expect(onChange).not.toHaveBeenCalled();
   });
 
   it('should get div ref', () => {
-    mount(
+    const refCalls = [];
+    render(
       <Checkbox.Group
         options={['Apple', 'Pear', 'Orange']}
         ref={node => {
-          expect(node.nodeName).toBe('DIV');
+          refCalls.push(node);
         }}
       />,
     );
+    const [mountCall] = refCalls;
+    expect(mountCall.nodeName).toBe('DIV');
   });
 
   it('should support number option', () => {
     const onChange = jest.fn();
 
-    const wrapper = mount(<Checkbox.Group options={[1, 'Pear', 'Orange']} onChange={onChange} />);
+    const { container } = render(
+      <Checkbox.Group options={[1, 'Pear', 'Orange']} onChange={onChange} />,
+    );
 
-    wrapper.find('.ant-checkbox-input').at(0).simulate('change');
+    fireEvent.click(container.querySelector('.ant-checkbox-input'));
     expect(onChange).toHaveBeenCalledWith([1]);
   });
 
@@ -251,17 +256,13 @@ describe('CheckboxGroup', () => {
       );
     };
 
-    const wrapper = mount(<Demo />);
-
-    wrapper.find('.ant-checkbox-input').first().simulate('change');
+    const { container } = render(<Demo />);
+    fireEvent.click(container.querySelector('.ant-checkbox-input'));
     expect(onChange).toHaveBeenCalledWith([]);
-    wrapper.find('.ant-checkbox-input').first().simulate('change');
+    fireEvent.click(container.querySelector('.ant-checkbox-input'));
     expect(onChange).toHaveBeenCalledWith(['length1']);
-    wrapper
-      .find('.ant-input')
-      .first()
-      .simulate('change', { target: { value: '' } });
-    wrapper.find('.ant-checkbox-input').first().simulate('change');
+    fireEvent.change(container.querySelector('.ant-input'), { target: { value: '' } });
+    fireEvent.click(container.querySelector('.ant-checkbox-input'));
     expect(onChange).toHaveBeenCalledWith(['A']);
   });
 });

From a217301cb723c9bc1068f1221f9e92eaacc86e17 Mon Sep 17 00:00:00 2001
From: haipeng <firemmet@gmail.com>
Date: Thu, 5 May 2022 10:52:41 +0800
Subject: [PATCH 04/12] fix: Carousel compatible with vertical property
 (#35349)

---
 components/carousel/index.tsx | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/components/carousel/index.tsx b/components/carousel/index.tsx
index d8975a38d0..7953ee9706 100644
--- a/components/carousel/index.tsx
+++ b/components/carousel/index.tsx
@@ -30,7 +30,17 @@ export interface CarouselRef {
 }
 
 const Carousel = React.forwardRef<CarouselRef, CarouselProps>(
-  ({ dots = true, arrows = false, draggable = false, dotPosition = 'bottom', ...props }, ref) => {
+  (
+    {
+      dots = true,
+      arrows = false,
+      draggable = false,
+      dotPosition = 'bottom',
+      vertical = dotPosition === 'left' || dotPosition === 'right',
+      ...props
+    },
+    ref,
+  ) => {
     const { getPrefixCls, direction } = React.useContext(ConfigContext);
     const slickRef = React.useRef<any>();
 
@@ -60,6 +70,7 @@ const Carousel = React.forwardRef<CarouselRef, CarouselProps>(
     }, [props.children]);
 
     const newProps = {
+      vertical,
       ...props,
     };
 
@@ -69,7 +80,6 @@ const Carousel = React.forwardRef<CarouselRef, CarouselProps>(
 
     const prefixCls = getPrefixCls('carousel', newProps.prefixCls);
     const dotsClass = 'slick-dots';
-    newProps.vertical = dotPosition === 'left' || dotPosition === 'right';
 
     const enableDots = !!dots;
     const dsClass = classNames(
@@ -80,7 +90,7 @@ const Carousel = React.forwardRef<CarouselRef, CarouselProps>(
 
     const className = classNames(prefixCls, {
       [`${prefixCls}-rtl`]: direction === 'rtl',
-      [`${prefixCls}-vertical`]: newProps.vertical,
+      [`${prefixCls}-vertical`]: dotPosition === 'left' || dotPosition === 'right',
     });
 
     return (

From 978226fb13a7ec2c4ee07bb8eeba6bb18188a622 Mon Sep 17 00:00:00 2001
From: afc163 <afc163@gmail.com>
Date: Thu, 5 May 2022 10:58:35 +0800
Subject: [PATCH 05/12] chore: fix npm start error in Gitpod

close #33444
---
 package.json | 1 +
 1 file changed, 1 insertion(+)

diff --git a/package.json b/package.json
index 2486dea145..0320a005c9 100644
--- a/package.json
+++ b/package.json
@@ -287,6 +287,7 @@
     "stylelint-order": "^5.0.0",
     "theme-switcher": "^1.0.2",
     "typescript": "~4.6.2",
+    "webpack": "^4.46.0",
     "webpack-bundle-analyzer": "^4.1.0",
     "xhr-mock": "^2.4.1",
     "yaml-front-matter": "^4.0.0"

From e039b10b2f72f01df81829f478960d2ccc50d5d5 Mon Sep 17 00:00:00 2001
From: afc163 <afc163@gmail.com>
Date: Thu, 5 May 2022 11:27:49 +0800
Subject: [PATCH 06/12] Revert "chore: fix npm start error in Gitpod"

This reverts commit 978226fb13a7ec2c4ee07bb8eeba6bb18188a622.
---
 package.json | 1 -
 1 file changed, 1 deletion(-)

diff --git a/package.json b/package.json
index 0320a005c9..2486dea145 100644
--- a/package.json
+++ b/package.json
@@ -287,7 +287,6 @@
     "stylelint-order": "^5.0.0",
     "theme-switcher": "^1.0.2",
     "typescript": "~4.6.2",
-    "webpack": "^4.46.0",
     "webpack-bundle-analyzer": "^4.1.0",
     "xhr-mock": "^2.4.1",
     "yaml-front-matter": "^4.0.0"

From 6c0f67e4074e7e33108f5def556225971ce4fb26 Mon Sep 17 00:00:00 2001
From: lalalazero <zzzero520@hotmail.com>
Date: Thu, 5 May 2022 12:17:15 +0800
Subject: [PATCH 07/12] fix: table column filter reset is not working (#35226)
 (#35386)

---
 .../table/__tests__/Table.filter.test.js      | 49 +++++++++++++++++++
 .../table/hooks/useFilter/FilterDropdown.tsx  | 18 ++++---
 2 files changed, 61 insertions(+), 6 deletions(-)

diff --git a/components/table/__tests__/Table.filter.test.js b/components/table/__tests__/Table.filter.test.js
index cf74e8d1cd..569c6badfa 100644
--- a/components/table/__tests__/Table.filter.test.js
+++ b/components/table/__tests__/Table.filter.test.js
@@ -2305,4 +2305,53 @@ describe('Table.filter', () => {
     );
     expect(errorSpy).not.toBeCalled();
   });
+
+  it('can reset if filterResetToDefaultFilteredValue and filter is changing', () => {
+    const wrapper = mount(
+      createTable({
+        columns: [
+          {
+            ...column,
+            filters: [
+              { text: 'Jack', value: 'Jack' },
+              { text: 'Lucy', value: 'Lucy' },
+            ],
+            defaultFilteredValue: ['Jack'],
+            filterResetToDefaultFilteredValue: true,
+          },
+        ],
+      }),
+    );
+    expect(wrapper.find('tbody tr').length).toBe(1);
+    expect(wrapper.find('tbody tr').text()).toBe('Jack');
+
+    // open filter
+    wrapper.find('span.ant-dropdown-trigger').first().simulate('click');
+    expect(
+      wrapper.find('.ant-table-filter-dropdown-btns .ant-btn-link').props().disabled,
+    ).toBeTruthy();
+    expect(wrapper.find('li.ant-dropdown-menu-item').at(0).text()).toBe('Jack');
+    expect(wrapper.find('li.ant-dropdown-menu-item').at(1).text()).toBe('Lucy');
+
+    // deselect default
+    wrapper.find('li.ant-dropdown-menu-item').at(0).simulate('click');
+    expect(
+      wrapper.find('.ant-table-filter-dropdown-btns .ant-btn-link').props().disabled,
+    ).toBeFalsy();
+    // select other one
+    wrapper.find('li.ant-dropdown-menu-item').at(1).simulate('click');
+    expect(
+      wrapper.find('.ant-table-filter-dropdown-btns .ant-btn-link').props().disabled,
+    ).toBeFalsy();
+    // deselect other one
+    wrapper.find('li.ant-dropdown-menu-item').at(1).simulate('click');
+    expect(
+      wrapper.find('.ant-table-filter-dropdown-btns .ant-btn-link').props().disabled,
+    ).toBeFalsy();
+    // select default
+    wrapper.find('li.ant-dropdown-menu-item').at(0).simulate('click');
+    expect(
+      wrapper.find('.ant-table-filter-dropdown-btns .ant-btn-link').props().disabled,
+    ).toBeTruthy();
+  });
 });
diff --git a/components/table/hooks/useFilter/FilterDropdown.tsx b/components/table/hooks/useFilter/FilterDropdown.tsx
index 4d5fa69f1e..8408fa1673 100644
--- a/components/table/hooks/useFilter/FilterDropdown.tsx
+++ b/components/table/hooks/useFilter/FilterDropdown.tsx
@@ -407,16 +407,22 @@ function FilterDropdown<RecordType>(props: FilterDropdownProps<RecordType>) {
       );
     };
 
+    const getResetDisabled = () => {
+      if (filterResetToDefaultFilteredValue) {
+        return isEqual(
+          (defaultFilteredValue || []).map(key => String(key)),
+          selectedKeys,
+        );
+      }
+
+      return selectedKeys.length === 0;
+    };
+
     dropdownContent = (
       <>
         {getFilterComponent()}
         <div className={`${prefixCls}-dropdown-btns`}>
-          <Button
-            type="link"
-            size="small"
-            disabled={selectedKeys.length === 0}
-            onClick={() => onReset()}
-          >
+          <Button type="link" size="small" disabled={getResetDisabled()} onClick={() => onReset()}>
             {locale.filterReset}
           </Button>
           <Button type="primary" size="small" onClick={onConfirm}>

From 10596d96045a761167e96327175a88030df44882 Mon Sep 17 00:00:00 2001
From: Yanlin Jiang <cncolder@gmail.com>
Date: Thu, 5 May 2022 13:46:52 +0800
Subject: [PATCH 08/12] fix: Respect user provided space item key (#35344)

---
 components/space/index.tsx | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/components/space/index.tsx b/components/space/index.tsx
index 30535b2218..5d1e7745b8 100644
--- a/components/space/index.tsx
+++ b/components/space/index.tsx
@@ -88,15 +88,12 @@ const Space: React.FC<SpaceProps> = props => {
       latestIndex = i;
     }
 
-    const keyOfChild = child && child.key;
-    // Add `-space-item` as suffix in case simple key string trigger duplicated key warning
-    // https://github.com/ant-design/ant-design/issues/35305
-    const defaultKey = `${i}-space-item`;
+    const key = child && child.key || `${itemClassName}-${i}`;
 
     return (
       <Item
         className={itemClassName}
-        key={`${itemClassName}-${keyOfChild || defaultKey}`}
+        key={key}
         direction={direction}
         index={i}
         marginDirection={marginDirection}

From d8ab5a577113098731b5bf6c370285c59e83ea3e Mon Sep 17 00:00:00 2001
From: Cooper Veysey <cwveysey@gmail.com>
Date: Thu, 5 May 2022 02:06:57 -0400
Subject: [PATCH 09/12] Resolve Ant Design Notification component TypeScript
 error via adding maxCount as an ArgsProps property (#35369)

---
 components/notification/index.tsx | 1 +
 1 file changed, 1 insertion(+)

diff --git a/components/notification/index.tsx b/components/notification/index.tsx
index fe743147f7..40a1783baf 100755
--- a/components/notification/index.tsx
+++ b/components/notification/index.tsx
@@ -206,6 +206,7 @@ export interface ArgsProps {
   duration?: number | null;
   icon?: React.ReactNode;
   placement?: NotificationPlacement;
+  maxCount?: number;
   style?: React.CSSProperties;
   prefixCls?: string;
   className?: string;

From ce1fc29bd75c5e1c1157a2ded2adee105aec6c61 Mon Sep 17 00:00:00 2001
From: vagusX <vagusX@users.noreply.github.com>
Date: Thu, 5 May 2022 14:09:27 +0800
Subject: [PATCH 10/12] fix: use less variables for Segmented (#35387)

---
 components/segmented/style/index.less | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/components/segmented/style/index.less b/components/segmented/style/index.less
index ec15358a0e..fd44ee1189 100644
--- a/components/segmented/style/index.less
+++ b/components/segmented/style/index.less
@@ -12,7 +12,7 @@
   padding: @segmented-container-padding;
   color: @segmented-label-color;
   background-color: @segmented-bg;
-  border-radius: 2px;
+  border-radius: @border-radius-base;
   transition: all 0.3s @ease-in-out;
 
   &-group {
@@ -67,7 +67,7 @@
 
     // syntactic sugar to add `icon` for Segmented Item
     &-icon + * {
-      margin-left: 6px;
+      margin-left: @margin-sm / 2;
     }
 
     &-input {

From 206e6dd138db30afe3172c171953e661d067ed5e Mon Sep 17 00:00:00 2001
From: afc163 <afc163@gmail.com>
Date: Thu, 5 May 2022 16:40:49 +0800
Subject: [PATCH 11/12] docs: reverse React 18 demo

---
 site/theme/template/Content/ComponentDoc.jsx | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/site/theme/template/Content/ComponentDoc.jsx b/site/theme/template/Content/ComponentDoc.jsx
index d5acf408c3..ab7045de69 100644
--- a/site/theme/template/Content/ComponentDoc.jsx
+++ b/site/theme/template/Content/ComponentDoc.jsx
@@ -25,7 +25,7 @@ class ComponentDoc extends React.Component {
     expandAll: false,
     visibleAll: process.env.NODE_ENV !== 'production',
     showRiddleButton: false,
-    react18Demo: true,
+    react17Demo: false,
   };
 
   componentDidMount() {
@@ -49,12 +49,12 @@ class ComponentDoc extends React.Component {
   shouldComponentUpdate(nextProps, nextState) {
     const { location, theme } = this.props;
     const { location: nextLocation, theme: nextTheme } = nextProps;
-    const { expandAll, visibleAll, showRiddleButton, react18Demo } = this.state;
+    const { expandAll, visibleAll, showRiddleButton, react17Demo } = this.state;
     const {
       expandAll: nextExpandAll,
       visibleAll: nextVisibleAll,
       showRiddleButton: nextShowRiddleButton,
-      react18Demo: nextReact18Demo,
+      react17Demo: nextReact17Demo,
     } = nextState;
 
     if (
@@ -64,7 +64,7 @@ class ComponentDoc extends React.Component {
       theme === nextTheme &&
       visibleAll === nextVisibleAll &&
       showRiddleButton === nextShowRiddleButton &&
-      react18Demo === nextReact18Demo
+      react17Demo === nextReact17Demo
     ) {
       return false;
     }
@@ -90,9 +90,9 @@ class ComponentDoc extends React.Component {
   };
 
   handleDemoVersionToggle = () => {
-    const { react18Demo } = this.state;
+    const { react17Demo } = this.state;
     this.setState({
-      react18Demo: !react18Demo,
+      react17Demo: !react17Demo,
     });
   };
 
@@ -108,7 +108,7 @@ class ComponentDoc extends React.Component {
     } = this.props;
     const { content, meta } = doc;
     const demoValues = Object.keys(demos).map(key => demos[key]);
-    const { expandAll, visibleAll, showRiddleButton, react18Demo } = this.state;
+    const { expandAll, visibleAll, showRiddleButton, react17Demo } = this.state;
     const isSingleCol = meta.cols === 1;
     const leftChildren = [];
     const rightChildren = [];
@@ -131,7 +131,7 @@ class ComponentDoc extends React.Component {
             location={location}
             theme={theme}
             setIframeTheme={setIframeTheme}
-            react18={react18Demo}
+            react18={react17Demo}
           />
         );
         if (index % 2 === 0 || isSingleCol) {
@@ -224,12 +224,12 @@ class ComponentDoc extends React.Component {
                 title={
                   <FormattedMessage
                     id={`app.component.examples.${
-                      react18Demo ? 'openDemoNotReact18' : 'openDemoWithReact18'
+                      react17Demo ? 'openDemoWithReact18' : 'openDemoNotReact18'
                     }`}
                   />
                 }
               >
-                {react18Demo ? (
+                {react17Demo ? (
                   <ExperimentFilled
                     className={expandTriggerClass}
                     onClick={this.handleDemoVersionToggle}

From 7511b9e8ac4ad17442b41e8b0b7025cee25c27f2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E4=BA=8C=E8=B4=A7=E6=9C=BA=E5=99=A8=E4=BA=BA?=
 <smith3816@gmail.com>
Date: Fri, 6 May 2022 11:48:18 +0800
Subject: [PATCH 12/12] test: replace textarea some test case with testing lib
 (#35398)

* test: replace textarea some test case with testing lib

* test: full process

* test: mv input test to input scope

* test: fix lint
---
 components/input/__tests__/index.test.tsx   | 35 +++++++++++++++++++++
 components/input/__tests__/textarea.test.js | 21 -------------
 2 files changed, 35 insertions(+), 21 deletions(-)

diff --git a/components/input/__tests__/index.test.tsx b/components/input/__tests__/index.test.tsx
index 602f7fac0d..2b02ce39f9 100644
--- a/components/input/__tests__/index.test.tsx
+++ b/components/input/__tests__/index.test.tsx
@@ -1,5 +1,6 @@
 import React, { useState } from 'react';
 import { mount } from 'enzyme';
+import { createPortal } from 'react-dom';
 import { render, fireEvent } from '../../../tests/utils';
 // eslint-disable-next-line import/no-unresolved
 import Form from '../../form';
@@ -75,6 +76,40 @@ describe('Input', () => {
     });
   });
 
+  describe('click focus', () => {
+    it('click outside should also get focus', () => {
+      const { container } = render(<Input suffix={<span className="test-suffix" />} />);
+      const onFocus = jest.spyOn(container.querySelector('input')!, 'focus');
+      fireEvent.mouseDown(container.querySelector('.test-suffix')!);
+      fireEvent.mouseUp(container.querySelector('.test-suffix')!);
+      expect(onFocus).toHaveBeenCalled();
+    });
+
+    it('not get focus if out of component', () => {
+      const holder = document.createElement('span');
+      document.body.appendChild(holder);
+
+      const Popup = () => createPortal(<span className="popup" />, holder);
+
+      const { container } = render(
+        <Input
+          suffix={
+            <span className="test-suffix">
+              <Popup />
+            </span>
+          }
+        />,
+      );
+
+      const onFocus = jest.spyOn(container.querySelector('input')!, 'focus');
+      fireEvent.mouseDown(document.querySelector('.popup')!);
+      fireEvent.mouseUp(document.querySelector('.popup')!);
+
+      expect(onFocus).not.toHaveBeenCalled();
+      document.body.removeChild(holder);
+    });
+  });
+
   it('set mouse cursor position', () => {
     const defaultValue = '11111';
     const valLength = defaultValue.length;
diff --git a/components/input/__tests__/textarea.test.js b/components/input/__tests__/textarea.test.js
index 5d314c7eb9..1f6788c9bc 100644
--- a/components/input/__tests__/textarea.test.js
+++ b/components/input/__tests__/textarea.test.js
@@ -404,27 +404,6 @@ describe('TextArea allowClear', () => {
     expect(wrapper.find('input').props().value).toEqual('Light');
   });
 
-  describe('click focus', () => {
-    it('click outside should also get focus', () => {
-      const wrapper = mount(<Input suffix={<span className="test-suffix" />} />);
-      const onFocus = jest.spyOn(wrapper.find('input').instance(), 'focus');
-      wrapper.find('.test-suffix').simulate('mouseUp');
-      expect(onFocus).toHaveBeenCalled();
-    });
-
-    it('not get focus if out of component', () => {
-      const wrapper = mount(<Input suffix={<span className="test-suffix" />} />);
-      const onFocus = jest.spyOn(wrapper.find('input').instance(), 'focus');
-      const ele = document.createElement('span');
-      document.body.appendChild(ele);
-      wrapper.find('.test-suffix').simulate('mouseUp', {
-        target: ele,
-      });
-      expect(onFocus).not.toHaveBeenCalled();
-      document.body.removeChild(ele);
-    });
-  });
-
   it('scroll to bottom when autoSize', async () => {
     const wrapper = mount(<Input.TextArea autoSize />, { attachTo: document.body });
     wrapper.find('textarea').simulate('focus');