From 9de9b857225d36db7799e330755fade683a995f6 Mon Sep 17 00:00:00 2001
From: cssking <laji198888@163.com>
Date: Wed, 20 Dec 2023 22:17:01 +0800
Subject: [PATCH 01/16] =?UTF-8?q?feat:=20=E4=BC=98=E5=8C=96=E6=95=B0?=
 =?UTF-8?q?=E5=AD=97=E9=94=AE=E7=9B=98=E5=8F=8A=E8=99=9A=E6=8B=9F=E8=BE=93?=
 =?UTF-8?q?=E5=85=A5=E6=A1=86=E6=97=A0=E9=9A=9C=E7=A2=8D?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/components/mask/mask.tsx                       |  1 +
 src/components/number-keyboard/number-keyboard.tsx | 12 ++++++------
 src/components/virtual-input/virtual-input.tsx     |  2 +-
 3 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/src/components/mask/mask.tsx b/src/components/mask/mask.tsx
index 9cc8a248ad..99d0aadaf1 100644
--- a/src/components/mask/mask.tsx
+++ b/src/components/mask/mask.tsx
@@ -101,6 +101,7 @@ export const Mask: FC<MaskProps> = p => {
       <animated.div
         className={classPrefix}
         ref={ref}
+        aria-hidden
         style={{
           ...props.style,
           background,
diff --git a/src/components/number-keyboard/number-keyboard.tsx b/src/components/number-keyboard/number-keyboard.tsx
index 2907c8423d..6fe5279959 100644
--- a/src/components/number-keyboard/number-keyboard.tsx
+++ b/src/components/number-keyboard/number-keyboard.tsx
@@ -138,8 +138,8 @@ export const NumberKeyboard: FC<NumberKeyboardProps> = p => {
             onClick={() => {
               props.onClose?.()
             }}
-            role='grid'
-            title='CLOSE'
+            role='button'
+            title='收起'
             tabIndex={-1}
           >
             <DownOutline />
@@ -160,7 +160,7 @@ export const NumberKeyboard: FC<NumberKeyboardProps> = p => {
 
     const ariaProps = key
       ? {
-          role: 'grid',
+          role: 'button',
           title: key,
           tabIndex: -1,
         }
@@ -233,8 +233,8 @@ export const NumberKeyboard: FC<NumberKeyboardProps> = p => {
                     // Long press should not trigger native context menu
                     e.preventDefault()
                   }}
-                  title='BACKSPACE'
-                  role='grid'
+                  title='删除'
+                  role='button'
                   tabIndex={-1}
                 >
                   <TextDeletionOutline />
@@ -242,7 +242,7 @@ export const NumberKeyboard: FC<NumberKeyboardProps> = p => {
                 <div
                   className={`${classPrefix}-key ${classPrefix}-key-extra ${classPrefix}-key-ok`}
                   onTouchEnd={e => onKeyPress(e, 'OK')}
-                  role='grid'
+                  role='button'
                   tabIndex={-1}
                   aria-label={confirmText}
                 >
diff --git a/src/components/virtual-input/virtual-input.tsx b/src/components/virtual-input/virtual-input.tsx
index 26ba5b6203..955496f4b0 100644
--- a/src/components/virtual-input/virtual-input.tsx
+++ b/src/components/virtual-input/virtual-input.tsx
@@ -130,7 +130,7 @@ export const VirtualInput = forwardRef<VirtualInputRef, VirtualInputProps>(
           [`${classPrefix}-disabled`]: props.disabled,
         })}
         tabIndex={props.disabled ? undefined : 0}
-        role='option'
+        role='textbox'
         onFocus={onFocus}
         onBlur={onBlur}
         onClick={props.onClick}

From 59460e8524a0a2633667e48a26e0c2138ed8e806 Mon Sep 17 00:00:00 2001
From: cssking <laji198888@163.com>
Date: Thu, 21 Dec 2023 21:37:59 +0800
Subject: [PATCH 02/16] =?UTF-8?q?fix:=20=E4=BC=98=E5=8C=96test=20snapshots?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../tests/__snapshots__/action-sheet.test.tsx.snap          | 1 +
 .../tests/__snapshots__/image-viewer.test.tsx.snap          | 1 +
 .../number-keyboard/tests/number-keyboard.test.tsx          | 6 +++---
 src/components/virtual-input/tests/virtual-input.test.tsx   | 4 ++--
 4 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/components/action-sheet/tests/__snapshots__/action-sheet.test.tsx.snap b/src/components/action-sheet/tests/__snapshots__/action-sheet.test.tsx.snap
index 0f64c3b4be..14dcaea0e1 100644
--- a/src/components/action-sheet/tests/__snapshots__/action-sheet.test.tsx.snap
+++ b/src/components/action-sheet/tests/__snapshots__/action-sheet.test.tsx.snap
@@ -18,6 +18,7 @@ exports[`ActionSheet basic usage 1`] = `
     class="adm-popup adm-action-sheet-popup"
   >
     <div
+      aria-hidden="true"
       class="adm-mask"
       style="background: rgba(0, 0, 0, 0.55); opacity: 1;"
     >
diff --git a/src/components/image-viewer/tests/__snapshots__/image-viewer.test.tsx.snap b/src/components/image-viewer/tests/__snapshots__/image-viewer.test.tsx.snap
index 20c54a4c12..8713f9850b 100644
--- a/src/components/image-viewer/tests/__snapshots__/image-viewer.test.tsx.snap
+++ b/src/components/image-viewer/tests/__snapshots__/image-viewer.test.tsx.snap
@@ -24,6 +24,7 @@ exports[`ImageViewer.Multi calling ref.current.swipeTo before initialization 2`]
     </span>
   </button>
   <div
+    aria-hidden="true"
     class="adm-mask"
     style="background: rgba(0, 0, 0, 0.75); opacity: 1;"
   >
diff --git a/src/components/number-keyboard/tests/number-keyboard.test.tsx b/src/components/number-keyboard/tests/number-keyboard.test.tsx
index ca7e8c73c2..80b6f152a9 100644
--- a/src/components/number-keyboard/tests/number-keyboard.test.tsx
+++ b/src/components/number-keyboard/tests/number-keyboard.test.tsx
@@ -27,7 +27,7 @@ describe('NumberKeyboard', () => {
   test('onClose should be called by close button', () => {
     const onClose = jest.fn()
     render(<NumberKeyboard visible onClose={onClose} />)
-    fireEvent.click(screen.getByTitle('CLOSE'))
+    fireEvent.click(screen.getByTitle('收起'))
     expect(onClose).toBeCalledTimes(1)
   })
 
@@ -75,7 +75,7 @@ describe('NumberKeyboard', () => {
       />
     )
     const confirm = screen.getByText('confirm')
-    const del = screen.getByTitle('BACKSPACE')
+    const del = screen.getByTitle('删除')
     expect(confirm).toBeInTheDocument()
     expect(confirm).toHaveClass(
       `${classPrefix}-key-extra ${classPrefix}-key-ok`
@@ -118,7 +118,7 @@ describe('NumberKeyboard', () => {
     const popup = document.querySelector(`.${classPrefix}-popup`)
     const main = document.querySelector(`.${classPrefix}-main`)
     expect(main).not.toHaveTextContent('1234567890')
-    fireEvent.click(screen.getByTitle('CLOSE'))
+    fireEvent.click(screen.getByTitle('收起'))
     await waitFor(() => expect(popup).not.toBeVisible())
   })
 
diff --git a/src/components/virtual-input/tests/virtual-input.test.tsx b/src/components/virtual-input/tests/virtual-input.test.tsx
index 0cd11daab9..23ea53b615 100644
--- a/src/components/virtual-input/tests/virtual-input.test.tsx
+++ b/src/components/virtual-input/tests/virtual-input.test.tsx
@@ -99,8 +99,8 @@ describe('VirtualInput', () => {
     expect(document.querySelector(`.${classPrefix}-content`)).toHaveTextContent(
       '0'
     )
-    expect(screen.getByTitle('CLOSE')).toBeInTheDocument()
-    fireEvent.click(screen.getByTitle('CLOSE'))
+    expect(screen.getByTitle('收起')).toBeInTheDocument()
+    fireEvent.click(screen.getByTitle('收起'))
     fireEvent.blur(screen.getByTestId('virtualInput'))
     await waitFor(() => {
       expect(

From 7967e74d74547562c2bafa0a3204b5ad5dc039f7 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, 22 Dec 2023 10:45:19 +0800
Subject: [PATCH 03/16] chore: use locale

---
 src/components/number-keyboard/number-keyboard.tsx | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/components/number-keyboard/number-keyboard.tsx b/src/components/number-keyboard/number-keyboard.tsx
index 6fe5279959..b8ed822bb8 100644
--- a/src/components/number-keyboard/number-keyboard.tsx
+++ b/src/components/number-keyboard/number-keyboard.tsx
@@ -8,6 +8,7 @@ import Popup, { PopupProps } from '../popup'
 import { NativeProps, withNativeProps } from '../../utils/native-props'
 import SafeArea from '../safe-area'
 import { useMemoizedFn } from 'ahooks'
+import { useConfig } from '../config-provider'
 
 const classPrefix = 'adm-number-keyboard'
 
@@ -59,6 +60,8 @@ export const NumberKeyboard: FC<NumberKeyboardProps> = p => {
     onInput,
   } = props
 
+  const { locale } = useConfig()
+
   const keyboardRef = useRef<HTMLDivElement | null>(null)
 
   const keys = useMemo(() => {
@@ -139,7 +142,7 @@ export const NumberKeyboard: FC<NumberKeyboardProps> = p => {
               props.onClose?.()
             }}
             role='button'
-            title='收起'
+            title={locale.common.close}
             tabIndex={-1}
           >
             <DownOutline />

From 50dd211a40b56bdfec1657168a6441c4376d0088 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, 22 Dec 2023 14:22:07 +0800
Subject: [PATCH 04/16] chore: update desc

---
 src/components/number-keyboard/number-keyboard.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/number-keyboard/number-keyboard.tsx b/src/components/number-keyboard/number-keyboard.tsx
index b8ed822bb8..ab4ecac484 100644
--- a/src/components/number-keyboard/number-keyboard.tsx
+++ b/src/components/number-keyboard/number-keyboard.tsx
@@ -236,7 +236,7 @@ export const NumberKeyboard: FC<NumberKeyboardProps> = p => {
                     // Long press should not trigger native context menu
                     e.preventDefault()
                   }}
-                  title='删除'
+                  title={locale.Input.clear}
                   role='button'
                   tabIndex={-1}
                 >

From a56b92c3a8ac3cdb227b563fe5a244a7ac9fb503 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, 22 Dec 2023 14:40:39 +0800
Subject: [PATCH 05/16] test: fix img part

---
 .../image-viewer/tests/image-viewer.test.tsx  | 25 +++++++++++++++----
 src/tests/testing.tsx                         |  9 +++++++
 2 files changed, 29 insertions(+), 5 deletions(-)

diff --git a/src/components/image-viewer/tests/image-viewer.test.tsx b/src/components/image-viewer/tests/image-viewer.test.tsx
index 9e85d70f98..7e5133a6d6 100644
--- a/src/components/image-viewer/tests/image-viewer.test.tsx
+++ b/src/components/image-viewer/tests/image-viewer.test.tsx
@@ -1,3 +1,5 @@
+/* eslint-disable @typescript-eslint/no-non-null-assertion */
+
 import React, { useRef, useState } from 'react'
 import {
   render,
@@ -8,10 +10,12 @@ import {
   userEvent,
   mockDrag,
   act,
+  waitFakeTimers,
 } from 'testing'
 import ImageViewer, { MultiImageViewerRef } from '../index'
 import Button from '../../button'
 import image from './image.json'
+import { a } from '@react-spring/web'
 const classPrefix = `adm-image-viewer`
 
 const demoImages = [
@@ -244,7 +248,9 @@ describe('ImageViewer', () => {
   })
 
   test('`ImageViewer.show/ImageViewer.clear` should be work', async () => {
-    render(
+    jest.useFakeTimers()
+
+    const { container } = render(
       <button
         onClick={() => {
           ImageViewer.show({ image: demoImages[0] })
@@ -253,13 +259,22 @@ describe('ImageViewer', () => {
         show
       </button>
     )
-    fireEvent.click(screen.getByText('show'))
-    const img = await screen.findByRole('img')
-    expect(img).toBeVisible()
+
+    fireEvent.click(container.querySelector<HTMLButtonElement>('button')!)
+    await waitFakeTimers()
+
+    console.log()
+
+    expect(document.querySelector('img')).toBeTruthy()
 
     act(() => {
       ImageViewer.clear()
     })
-    await waitFor(() => expect(img).not.toBeVisible())
+    await waitFakeTimers()
+
+    expect(document.querySelector('img')).toBeFalsy()
+
+    jest.clearAllTimers()
+    jest.useRealTimers()
   })
 })
diff --git a/src/tests/testing.tsx b/src/tests/testing.tsx
index 06c5916e7c..b790d5d6e3 100644
--- a/src/tests/testing.tsx
+++ b/src/tests/testing.tsx
@@ -116,6 +116,15 @@ export const actSleep = (time: number) => {
   return act(() => sleep(time))
 }
 
+export const waitFakeTimers = async () => {
+  for (let i = 0; i < 10; i += 1) {
+    await act(async () => {
+      jest.advanceTimersByTime(1000)
+      await Promise.resolve()
+    })
+  }
+}
+
 export const mockDrag = async (el: Element, options: any[], time?: number) => {
   const [downOptions, ...moveOptions] = options
   fireEvent.mouseDown(el, {

From 886bdb0fd3b236b80ab383820ebf9ca30128cb49 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, 22 Dec 2023 15:28:24 +0800
Subject: [PATCH 06/16] test: fix img part

---
 jest.config.js                                |  2 +-
 .../image-viewer/tests/image-viewer.test.tsx  | 67 ++++++++++++-------
 2 files changed, 43 insertions(+), 26 deletions(-)

diff --git a/jest.config.js b/jest.config.js
index 9cf30a6087..9cf1772d76 100644
--- a/jest.config.js
+++ b/jest.config.js
@@ -1,7 +1,7 @@
 module.exports = {
   preset: 'ts-jest',
   testEnvironment: 'jsdom',
-  // maxConcurrency: 1,
+  maxConcurrency: 1,
   moduleFileExtensions: ['ts', 'tsx', 'js', 'jsx'],
   modulePathIgnorePatterns: ['<rootDir>/lib/', '<rootDir>/dist/'],
   moduleDirectories: ['node_modules', 'src/tests'],
diff --git a/src/components/image-viewer/tests/image-viewer.test.tsx b/src/components/image-viewer/tests/image-viewer.test.tsx
index 7e5133a6d6..9f92842f82 100644
--- a/src/components/image-viewer/tests/image-viewer.test.tsx
+++ b/src/components/image-viewer/tests/image-viewer.test.tsx
@@ -92,6 +92,16 @@ jest.mock('../../../utils/use-drag-and-pinch', () => {
 })
 
 describe('ImageViewer.Multi', () => {
+  beforeEach(() => {
+    document.body.innerHTML = ''
+    jest.useFakeTimers()
+  })
+
+  afterEach(() => {
+    jest.clearAllTimers()
+    jest.useRealTimers()
+  })
+
   test('calling ref.current.swipeTo before initialization', async () => {
     function App() {
       const [visible, setVisible] = useState(false)
@@ -118,15 +128,15 @@ describe('ImageViewer.Multi', () => {
     const renderer = render(<App />)
     expect(renderer.container).toMatchSnapshot()
     fireEvent.click(renderer.getByText('Show'))
-    await waitFor(() =>
-      // end of animation
-      expect(document.querySelectorAll('.adm-mask')[0]).toHaveStyle(
-        'opacity: 1;'
-      )
-    )
+
+    await waitFakeTimers()
+
+    // end of animation
+    expect(document.querySelectorAll('.adm-mask')[0]).toHaveStyle('opacity: 1;')
     expect(renderer.getByText('3 / 4')).not.toBeNull()
     expect(renderer.container).toMatchSnapshot()
   })
+
   test('rendering with footer', () => {
     function App() {
       return (
@@ -140,6 +150,7 @@ describe('ImageViewer.Multi', () => {
     render(<App />)
     expect(screen.getByText('查看原图')).toBeInTheDocument()
   })
+
   test('`ImageViewer.Multi.show` should be work', async () => {
     render(
       <>
@@ -152,42 +163,48 @@ describe('ImageViewer.Multi', () => {
         </button>
       </>
     )
+
+    console.log(document.body.innerHTML)
+
     fireEvent.click(screen.getByText('show'))
-    const imgs = await screen.findAllByRole('img')
-    expect(imgs[0]).toBeVisible()
-    await act(async () => {
-      await userEvent.click(imgs[0])
+    await waitFakeTimers()
+
+    const img = document.querySelector<HTMLImageElement>('img')!
+    expect(img).toBeTruthy()
+
+    console.log('~~~~~~~~~~~~~~~')
+    userEvent.click(img)
+    await waitFakeTimers()
+
+    await waitFor(() => {
+      expect(document.querySelector<HTMLImageElement>('img')).toBeFalsy()
     })
-    await waitFor(() => expect(imgs[0]).not.toBeVisible())
   })
 
+  return
+
   test('slide and slide with pinched should be work', async () => {
     Object.defineProperty(window, 'innerWidth', {
       value: 300,
     })
     const onIndexChange = jest.fn()
 
-    act(() => {
-      render(
-        <ImageViewer.Multi
-          visible
-          defaultIndex={3}
-          images={demoImages}
-          onIndexChange={onIndexChange}
-        ></ImageViewer.Multi>
-      )
-    })
+    render(
+      <ImageViewer.Multi
+        visible
+        defaultIndex={3}
+        images={demoImages}
+        onIndexChange={onIndexChange}
+      />
+    )
 
-    await screen.findAllByRole('img')
+    await waitFakeTimers()
 
     G?.onPinch({
       origin: [235, 202],
       offset: [1.94, 0],
     })
 
-    // need to wait image render.
-    await act(() => new Promise(resolve => setTimeout(resolve, 2500)))
-
     const slides = document.querySelectorAll(`.${classPrefix}-control`)[3]
 
     expect(screen.getByText('4 / 4')).toBeInTheDocument()

From 5ed2a3d6ef535f4cf18a56cb2d5f093145a0403f 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, 22 Dec 2023 15:35:54 +0800
Subject: [PATCH 07/16] test: fix img part

---
 .../image-viewer/tests/image-viewer.test.tsx  | 26 +++++++------------
 1 file changed, 10 insertions(+), 16 deletions(-)

diff --git a/src/components/image-viewer/tests/image-viewer.test.tsx b/src/components/image-viewer/tests/image-viewer.test.tsx
index 9f92842f82..00655de4a5 100644
--- a/src/components/image-viewer/tests/image-viewer.test.tsx
+++ b/src/components/image-viewer/tests/image-viewer.test.tsx
@@ -164,15 +164,12 @@ describe('ImageViewer.Multi', () => {
       </>
     )
 
-    console.log(document.body.innerHTML)
-
     fireEvent.click(screen.getByText('show'))
     await waitFakeTimers()
 
     const img = document.querySelector<HTMLImageElement>('img')!
     expect(img).toBeTruthy()
 
-    console.log('~~~~~~~~~~~~~~~')
     userEvent.click(img)
     await waitFakeTimers()
 
@@ -181,8 +178,6 @@ describe('ImageViewer.Multi', () => {
     })
   })
 
-  return
-
   test('slide and slide with pinched should be work', async () => {
     Object.defineProperty(window, 'innerWidth', {
       value: 300,
@@ -243,13 +238,21 @@ describe('ImageViewer.Multi', () => {
 })
 
 describe('ImageViewer', () => {
+  beforeEach(() => {
+    document.body.innerHTML = ''
+    jest.useFakeTimers()
+  })
+
+  afterEach(() => {
+    jest.clearAllTimers()
+    jest.useRealTimers()
+  })
+
   test('a11y', async () => {
     await testA11y(<ImageViewer image={demoImages[0]} visible={true} />)
   })
 
   test('maxZoom support auto', async () => {
-    jest.useFakeTimers()
-
     render(<ImageViewer image={demoImages[0]} visible maxZoom='auto' />)
 
     // Pinch to zoom bigger
@@ -259,14 +262,10 @@ describe('ImageViewer', () => {
 
     expect(G.nextZoom).toEqual(10)
 
-    jest.clearAllTimers()
-    jest.useRealTimers()
     jest.restoreAllMocks()
   })
 
   test('`ImageViewer.show/ImageViewer.clear` should be work', async () => {
-    jest.useFakeTimers()
-
     const { container } = render(
       <button
         onClick={() => {
@@ -280,8 +279,6 @@ describe('ImageViewer', () => {
     fireEvent.click(container.querySelector<HTMLButtonElement>('button')!)
     await waitFakeTimers()
 
-    console.log()
-
     expect(document.querySelector('img')).toBeTruthy()
 
     act(() => {
@@ -290,8 +287,5 @@ describe('ImageViewer', () => {
     await waitFakeTimers()
 
     expect(document.querySelector('img')).toBeFalsy()
-
-    jest.clearAllTimers()
-    jest.useRealTimers()
   })
 })

From 7221b079c779c31d84683e2b3278d5a70680f075 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, 22 Dec 2023 15:45:08 +0800
Subject: [PATCH 08/16] test: fix img test

---
 .../image-viewer/tests/image-viewer.test.tsx    | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/src/components/image-viewer/tests/image-viewer.test.tsx b/src/components/image-viewer/tests/image-viewer.test.tsx
index 00655de4a5..17c198df41 100644
--- a/src/components/image-viewer/tests/image-viewer.test.tsx
+++ b/src/components/image-viewer/tests/image-viewer.test.tsx
@@ -129,10 +129,13 @@ describe('ImageViewer.Multi', () => {
     expect(renderer.container).toMatchSnapshot()
     fireEvent.click(renderer.getByText('Show'))
 
-    await waitFakeTimers()
-
     // end of animation
-    expect(document.querySelectorAll('.adm-mask')[0]).toHaveStyle('opacity: 1;')
+    await waitFor(() => {
+      expect(document.querySelectorAll('.adm-mask')[0]).toHaveStyle(
+        'opacity: 1;'
+      )
+    })
+
     expect(renderer.getByText('3 / 4')).not.toBeNull()
     expect(renderer.container).toMatchSnapshot()
   })
@@ -249,6 +252,7 @@ describe('ImageViewer', () => {
   })
 
   test('a11y', async () => {
+    jest.useRealTimers()
     await testA11y(<ImageViewer image={demoImages[0]} visible={true} />)
   })
 
@@ -266,10 +270,12 @@ describe('ImageViewer', () => {
   })
 
   test('`ImageViewer.show/ImageViewer.clear` should be work', async () => {
+    const onClose = jest.fn()
+
     const { container } = render(
       <button
         onClick={() => {
-          ImageViewer.show({ image: demoImages[0] })
+          ImageViewer.show({ image: demoImages[0], onClose })
         }}
       >
         show
@@ -284,8 +290,7 @@ describe('ImageViewer', () => {
     act(() => {
       ImageViewer.clear()
     })
-    await waitFakeTimers()
 
-    expect(document.querySelector('img')).toBeFalsy()
+    expect(onClose).toHaveBeenCalled()
   })
 })

From c0072c0c06ad6c2284d857a80f97e5e6853940ef 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, 22 Dec 2023 16:02:28 +0800
Subject: [PATCH 09/16] test: rm useless snapshot

---
 .../tests/__snapshots__/image-viewer.test.tsx.snap  | 13 -------------
 .../image-viewer/tests/image-viewer.test.tsx        | 12 +++++-------
 2 files changed, 5 insertions(+), 20 deletions(-)

diff --git a/src/components/image-viewer/tests/__snapshots__/image-viewer.test.tsx.snap b/src/components/image-viewer/tests/__snapshots__/image-viewer.test.tsx.snap
index 8713f9850b..7ef91da737 100644
--- a/src/components/image-viewer/tests/__snapshots__/image-viewer.test.tsx.snap
+++ b/src/components/image-viewer/tests/__snapshots__/image-viewer.test.tsx.snap
@@ -1,19 +1,6 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`ImageViewer.Multi calling ref.current.swipeTo before initialization 1`] = `
-<div>
-  <button
-    class="adm-button adm-button-default adm-button-shape-default"
-    type="button"
-  >
-    <span>
-      Show
-    </span>
-  </button>
-</div>
-`;
-
-exports[`ImageViewer.Multi calling ref.current.swipeTo before initialization 2`] = `
 <div>
   <button
     class="adm-button adm-button-default adm-button-shape-default"
diff --git a/src/components/image-viewer/tests/image-viewer.test.tsx b/src/components/image-viewer/tests/image-viewer.test.tsx
index 17c198df41..efc98faac8 100644
--- a/src/components/image-viewer/tests/image-viewer.test.tsx
+++ b/src/components/image-viewer/tests/image-viewer.test.tsx
@@ -126,16 +126,14 @@ describe('ImageViewer.Multi', () => {
       )
     }
     const renderer = render(<App />)
-    expect(renderer.container).toMatchSnapshot()
+
+    expect(document.querySelector('.adm-mask')).toBeFalsy()
+
     fireEvent.click(renderer.getByText('Show'))
+    await waitFakeTimers()
 
     // end of animation
-    await waitFor(() => {
-      expect(document.querySelectorAll('.adm-mask')[0]).toHaveStyle(
-        'opacity: 1;'
-      )
-    })
-
+    expect(document.querySelector('.adm-mask')).toBeTruthy()
     expect(renderer.getByText('3 / 4')).not.toBeNull()
     expect(renderer.container).toMatchSnapshot()
   })

From 438ca86d9d864ca0e047482e89ed50e7c7f0f2b1 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, 22 Dec 2023 16:06:36 +0800
Subject: [PATCH 10/16] test: fix modal test

---
 src/components/modal/tests/modal.test.tsx | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/components/modal/tests/modal.test.tsx b/src/components/modal/tests/modal.test.tsx
index 0ca1acfe26..b96cf03409 100644
--- a/src/components/modal/tests/modal.test.tsx
+++ b/src/components/modal/tests/modal.test.tsx
@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-non-null-assertion */
 import React from 'react'
 import {
   render,
@@ -7,6 +8,7 @@ import {
   waitForElementToBeRemoved,
   screen,
   act,
+  waitFakeTimers,
 } from 'testing'
 import Modal, { ModalAlertProps } from '..'
 
@@ -59,6 +61,8 @@ describe('Modal', () => {
   })
 
   test('close on mask click', async () => {
+    jest.useFakeTimers()
+
     const onClose = jest.fn()
     const afterClose = jest.fn()
     render(
@@ -66,11 +70,16 @@ describe('Modal', () => {
     )
 
     fireEvent.click(screen.getByRole('button', { name: 'btn' }))
-    const mask = await screen.findByRole('button', { name: '背景蒙层' })
+    await waitFakeTimers()
+
+    const mask = document.querySelector('.adm-mask-aria-button')!
     fireEvent.click(mask)
     await waitForElementToBeRemoved(mask)
     expect(onClose).toBeCalled()
     expect(afterClose).toBeCalled()
+
+    jest.clearAllTimers()
+    jest.useRealTimers()
   })
 
   test('show close button', async () => {

From a3d21fbadda96cc0f5966d7f09a28759981131e7 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, 22 Dec 2023 16:16:07 +0800
Subject: [PATCH 11/16] test: more test case

---
 src/components/dialog/tests/dialog.test.tsx         | 13 ++++++++++++-
 .../number-keyboard/tests/number-keyboard.test.tsx  |  7 ++++---
 2 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/src/components/dialog/tests/dialog.test.tsx b/src/components/dialog/tests/dialog.test.tsx
index 4df06fe0f8..4da05a62fb 100644
--- a/src/components/dialog/tests/dialog.test.tsx
+++ b/src/components/dialog/tests/dialog.test.tsx
@@ -1,3 +1,5 @@
+/* eslint-disable @typescript-eslint/no-non-null-assertion */
+
 import React from 'react'
 import {
   render,
@@ -6,6 +8,7 @@ import {
   waitFor,
   waitForElementToBeRemoved,
   screen,
+  waitFakeTimers,
 } from 'testing'
 import Dialog, { DialogAlertProps } from '..'
 import { act } from '@testing-library/react'
@@ -58,6 +61,8 @@ describe('Dialog', () => {
   })
 
   test('close on mask click', async () => {
+    jest.useFakeTimers()
+
     const onClose = jest.fn()
     const afterClose = jest.fn()
     render(
@@ -65,11 +70,17 @@ describe('Dialog', () => {
     )
 
     fireEvent.click(screen.getByRole('button', { name: 'btn' }))
-    const mask = await screen.findByRole('button', { name: '背景蒙层' })
+    await waitFakeTimers()
+
+    const mask = document.querySelector('.adm-mask-aria-button')!
     fireEvent.click(mask)
+
     await waitForElementToBeRemoved(mask)
     expect(onClose).toBeCalled()
     expect(afterClose).toBeCalled()
+
+    jest.clearAllTimers()
+    jest.useRealTimers()
   })
 
   test('custom content', async () => {
diff --git a/src/components/number-keyboard/tests/number-keyboard.test.tsx b/src/components/number-keyboard/tests/number-keyboard.test.tsx
index 80b6f152a9..0f3e830991 100644
--- a/src/components/number-keyboard/tests/number-keyboard.test.tsx
+++ b/src/components/number-keyboard/tests/number-keyboard.test.tsx
@@ -27,7 +27,8 @@ describe('NumberKeyboard', () => {
   test('onClose should be called by close button', () => {
     const onClose = jest.fn()
     render(<NumberKeyboard visible onClose={onClose} />)
-    fireEvent.click(screen.getByTitle('收起'))
+
+    fireEvent.click(screen.getByTitle('关闭'))
     expect(onClose).toBeCalledTimes(1)
   })
 
@@ -75,7 +76,7 @@ describe('NumberKeyboard', () => {
       />
     )
     const confirm = screen.getByText('confirm')
-    const del = screen.getByTitle('删除')
+    const del = screen.getByTitle('清除')
     expect(confirm).toBeInTheDocument()
     expect(confirm).toHaveClass(
       `${classPrefix}-key-extra ${classPrefix}-key-ok`
@@ -118,7 +119,7 @@ describe('NumberKeyboard', () => {
     const popup = document.querySelector(`.${classPrefix}-popup`)
     const main = document.querySelector(`.${classPrefix}-main`)
     expect(main).not.toHaveTextContent('1234567890')
-    fireEvent.click(screen.getByTitle('收起'))
+    fireEvent.click(screen.getByTitle('Close'))
     await waitFor(() => expect(popup).not.toBeVisible())
   })
 

From a6aac2bf16ee35a5bc323e956ef769baf8a58350 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, 22 Dec 2023 16:19:38 +0800
Subject: [PATCH 12/16] test: more test case

---
 src/components/number-keyboard/tests/number-keyboard.test.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/number-keyboard/tests/number-keyboard.test.tsx b/src/components/number-keyboard/tests/number-keyboard.test.tsx
index 0f3e830991..d14db2c022 100644
--- a/src/components/number-keyboard/tests/number-keyboard.test.tsx
+++ b/src/components/number-keyboard/tests/number-keyboard.test.tsx
@@ -119,7 +119,7 @@ describe('NumberKeyboard', () => {
     const popup = document.querySelector(`.${classPrefix}-popup`)
     const main = document.querySelector(`.${classPrefix}-main`)
     expect(main).not.toHaveTextContent('1234567890')
-    fireEvent.click(screen.getByTitle('Close'))
+    fireEvent.click(screen.getByTitle('关闭'))
     await waitFor(() => expect(popup).not.toBeVisible())
   })
 

From bed82f8543b2433d40e93c55feaf26d65179b641 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, 22 Dec 2023 16:21:54 +0800
Subject: [PATCH 13/16] test: more test case

---
 src/components/virtual-input/tests/virtual-input.test.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/components/virtual-input/tests/virtual-input.test.tsx b/src/components/virtual-input/tests/virtual-input.test.tsx
index 23ea53b615..ef456a11ee 100644
--- a/src/components/virtual-input/tests/virtual-input.test.tsx
+++ b/src/components/virtual-input/tests/virtual-input.test.tsx
@@ -99,8 +99,8 @@ describe('VirtualInput', () => {
     expect(document.querySelector(`.${classPrefix}-content`)).toHaveTextContent(
       '0'
     )
-    expect(screen.getByTitle('收起')).toBeInTheDocument()
-    fireEvent.click(screen.getByTitle('收起'))
+    expect(screen.getByTitle('关闭')).toBeInTheDocument()
+    fireEvent.click(screen.getByTitle('关闭'))
     fireEvent.blur(screen.getByTestId('virtualInput'))
     await waitFor(() => {
       expect(

From 23b019fc1a448ad18852fee4a3285b6294b6d6a9 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, 22 Dec 2023 16:30:26 +0800
Subject: [PATCH 14/16] test: more test case

---
 src/components/modal/tests/modal.test.tsx | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/components/modal/tests/modal.test.tsx b/src/components/modal/tests/modal.test.tsx
index b96cf03409..e212458ec1 100644
--- a/src/components/modal/tests/modal.test.tsx
+++ b/src/components/modal/tests/modal.test.tsx
@@ -61,8 +61,6 @@ describe('Modal', () => {
   })
 
   test('close on mask click', async () => {
-    jest.useFakeTimers()
-
     const onClose = jest.fn()
     const afterClose = jest.fn()
     render(
@@ -74,12 +72,11 @@ describe('Modal', () => {
 
     const mask = document.querySelector('.adm-mask-aria-button')!
     fireEvent.click(mask)
+
     await waitForElementToBeRemoved(mask)
+
     expect(onClose).toBeCalled()
     expect(afterClose).toBeCalled()
-
-    jest.clearAllTimers()
-    jest.useRealTimers()
   })
 
   test('show close button', async () => {

From cfcebad61b6fc8f8c52b371014b73a0e7d59eb9f 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, 22 Dec 2023 16:43:50 +0800
Subject: [PATCH 15/16] test: role check

---
 .../__snapshots__/image-viewer.test.tsx.snap  |  13 +++
 .../image-viewer/tests/image-viewer.test.tsx  | 105 +++++++-----------
 2 files changed, 56 insertions(+), 62 deletions(-)

diff --git a/src/components/image-viewer/tests/__snapshots__/image-viewer.test.tsx.snap b/src/components/image-viewer/tests/__snapshots__/image-viewer.test.tsx.snap
index 7ef91da737..8713f9850b 100644
--- a/src/components/image-viewer/tests/__snapshots__/image-viewer.test.tsx.snap
+++ b/src/components/image-viewer/tests/__snapshots__/image-viewer.test.tsx.snap
@@ -1,6 +1,19 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`ImageViewer.Multi calling ref.current.swipeTo before initialization 1`] = `
+<div>
+  <button
+    class="adm-button adm-button-default adm-button-shape-default"
+    type="button"
+  >
+    <span>
+      Show
+    </span>
+  </button>
+</div>
+`;
+
+exports[`ImageViewer.Multi calling ref.current.swipeTo before initialization 2`] = `
 <div>
   <button
     class="adm-button adm-button-default adm-button-shape-default"
diff --git a/src/components/image-viewer/tests/image-viewer.test.tsx b/src/components/image-viewer/tests/image-viewer.test.tsx
index efc98faac8..3645a23224 100644
--- a/src/components/image-viewer/tests/image-viewer.test.tsx
+++ b/src/components/image-viewer/tests/image-viewer.test.tsx
@@ -1,5 +1,3 @@
-/* eslint-disable @typescript-eslint/no-non-null-assertion */
-
 import React, { useRef, useState } from 'react'
 import {
   render,
@@ -10,12 +8,10 @@ import {
   userEvent,
   mockDrag,
   act,
-  waitFakeTimers,
 } from 'testing'
 import ImageViewer, { MultiImageViewerRef } from '../index'
 import Button from '../../button'
 import image from './image.json'
-import { a } from '@react-spring/web'
 const classPrefix = `adm-image-viewer`
 
 const demoImages = [
@@ -91,17 +87,15 @@ jest.mock('../../../utils/use-drag-and-pinch', () => {
   }
 })
 
-describe('ImageViewer.Multi', () => {
-  beforeEach(() => {
-    document.body.innerHTML = ''
-    jest.useFakeTimers()
-  })
+async function getImages() {
+  const images = await screen.findAllByText(
+    (content, element) => element?.tagName.toLowerCase() === 'img'
+  )
 
-  afterEach(() => {
-    jest.clearAllTimers()
-    jest.useRealTimers()
-  })
+  return images[0]
+}
 
+describe('ImageViewer.Multi', () => {
   test('calling ref.current.swipeTo before initialization', async () => {
     function App() {
       const [visible, setVisible] = useState(false)
@@ -126,18 +120,17 @@ describe('ImageViewer.Multi', () => {
       )
     }
     const renderer = render(<App />)
-
-    expect(document.querySelector('.adm-mask')).toBeFalsy()
-
+    expect(renderer.container).toMatchSnapshot()
     fireEvent.click(renderer.getByText('Show'))
-    await waitFakeTimers()
-
-    // end of animation
-    expect(document.querySelector('.adm-mask')).toBeTruthy()
+    await waitFor(() =>
+      // end of animation
+      expect(document.querySelectorAll('.adm-mask')[0]).toHaveStyle(
+        'opacity: 1;'
+      )
+    )
     expect(renderer.getByText('3 / 4')).not.toBeNull()
     expect(renderer.container).toMatchSnapshot()
   })
-
   test('rendering with footer', () => {
     function App() {
       return (
@@ -151,7 +144,6 @@ describe('ImageViewer.Multi', () => {
     render(<App />)
     expect(screen.getByText('查看原图')).toBeInTheDocument()
   })
-
   test('`ImageViewer.Multi.show` should be work', async () => {
     render(
       <>
@@ -164,19 +156,14 @@ describe('ImageViewer.Multi', () => {
         </button>
       </>
     )
-
     fireEvent.click(screen.getByText('show'))
-    await waitFakeTimers()
-
-    const img = document.querySelector<HTMLImageElement>('img')!
-    expect(img).toBeTruthy()
-
-    userEvent.click(img)
-    await waitFakeTimers()
+    const img = await getImages()
 
-    await waitFor(() => {
-      expect(document.querySelector<HTMLImageElement>('img')).toBeFalsy()
+    expect(img).toBeVisible()
+    await act(async () => {
+      await userEvent.click(img)
     })
+    await waitFor(() => expect(img).not.toBeVisible())
   })
 
   test('slide and slide with pinched should be work', async () => {
@@ -185,22 +172,27 @@ describe('ImageViewer.Multi', () => {
     })
     const onIndexChange = jest.fn()
 
-    render(
-      <ImageViewer.Multi
-        visible
-        defaultIndex={3}
-        images={demoImages}
-        onIndexChange={onIndexChange}
-      />
-    )
+    act(() => {
+      render(
+        <ImageViewer.Multi
+          visible
+          defaultIndex={3}
+          images={demoImages}
+          onIndexChange={onIndexChange}
+        ></ImageViewer.Multi>
+      )
+    })
 
-    await waitFakeTimers()
+    await getImages()
 
     G?.onPinch({
       origin: [235, 202],
       offset: [1.94, 0],
     })
 
+    // need to wait image render.
+    await act(() => new Promise(resolve => setTimeout(resolve, 2500)))
+
     const slides = document.querySelectorAll(`.${classPrefix}-control`)[3]
 
     expect(screen.getByText('4 / 4')).toBeInTheDocument()
@@ -239,22 +231,13 @@ describe('ImageViewer.Multi', () => {
 })
 
 describe('ImageViewer', () => {
-  beforeEach(() => {
-    document.body.innerHTML = ''
-    jest.useFakeTimers()
-  })
-
-  afterEach(() => {
-    jest.clearAllTimers()
-    jest.useRealTimers()
-  })
-
   test('a11y', async () => {
-    jest.useRealTimers()
     await testA11y(<ImageViewer image={demoImages[0]} visible={true} />)
   })
 
   test('maxZoom support auto', async () => {
+    jest.useFakeTimers()
+
     render(<ImageViewer image={demoImages[0]} visible maxZoom='auto' />)
 
     // Pinch to zoom bigger
@@ -264,31 +247,29 @@ describe('ImageViewer', () => {
 
     expect(G.nextZoom).toEqual(10)
 
+    jest.clearAllTimers()
+    jest.useRealTimers()
     jest.restoreAllMocks()
   })
 
   test('`ImageViewer.show/ImageViewer.clear` should be work', async () => {
-    const onClose = jest.fn()
-
-    const { container } = render(
+    render(
       <button
         onClick={() => {
-          ImageViewer.show({ image: demoImages[0], onClose })
+          ImageViewer.show({ image: demoImages[0] })
         }}
       >
         show
       </button>
     )
+    fireEvent.click(screen.getByText('show'))
 
-    fireEvent.click(container.querySelector<HTMLButtonElement>('button')!)
-    await waitFakeTimers()
-
-    expect(document.querySelector('img')).toBeTruthy()
+    const img = await getImages()
+    expect(img).toBeVisible()
 
     act(() => {
       ImageViewer.clear()
     })
-
-    expect(onClose).toHaveBeenCalled()
+    await waitFor(() => expect(img).not.toBeVisible())
   })
 })

From a07952e394f329f03d585c678a50ef5fd58a59e2 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, 22 Dec 2023 16:47:45 +0800
Subject: [PATCH 16/16] test: rm fake timer

---
 src/components/dialog/tests/dialog.test.tsx | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/src/components/dialog/tests/dialog.test.tsx b/src/components/dialog/tests/dialog.test.tsx
index 4da05a62fb..5536f77179 100644
--- a/src/components/dialog/tests/dialog.test.tsx
+++ b/src/components/dialog/tests/dialog.test.tsx
@@ -61,8 +61,6 @@ describe('Dialog', () => {
   })
 
   test('close on mask click', async () => {
-    jest.useFakeTimers()
-
     const onClose = jest.fn()
     const afterClose = jest.fn()
     render(
@@ -78,9 +76,6 @@ describe('Dialog', () => {
     await waitForElementToBeRemoved(mask)
     expect(onClose).toBeCalled()
     expect(afterClose).toBeCalled()
-
-    jest.clearAllTimers()
-    jest.useRealTimers()
   })
 
   test('custom content', async () => {