test: fix failed test

ant-design · Apr 13, 2022 · fd75c6b · fd75c6b
1 parent c5bbdb9
commit fd75c6b
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 13 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
@@ -2,7 +2,7 @@
 
 exports[`ActionSheet basic usage 1`] = `
 <body
-  class="adm-overflow-hidden"
+  class=""
 >
   <div>
     <button

diff --git a/src/components/action-sheet/tests/action-sheet.test.tsx b/src/components/action-sheet/tests/action-sheet.test.tsx
@@ -154,11 +154,12 @@ describe('ActionSheet', () => {
     )
     fireEvent.click(getByText('button'))
 
-    await waitFor(() =>
+    await waitFor(() => {
       fireEvent.click(baseElement.querySelectorAll(`.adm-mask-aria-button`)[0])
-    )
-    await sleep(1000)
-    expect(baseElement.querySelectorAll(`.${classPrefix}`)[0]).toBeVisible()
+    })
+    await waitFor(() => {
+      expect(baseElement.querySelectorAll(`.${classPrefix}`)[0]).toBeVisible()
+    })
   })
 
   test('action click shound be called', async () => {

diff --git a/src/components/form/tests/form.test.tsx b/src/components/form/tests/form.test.tsx
@@ -46,23 +46,23 @@ describe('Form', () => {
 
     console.error = jest.fn()
 
+    fireEvent.click(getByText('submit'))
     await waitFor(() => {
-      fireEvent.click(getByText('submit'))
+      expect($$(`.${classPrefix}-item-feedback-error`).length).toBeTruthy()
     })
 
-    expect($$(`.${classPrefix}-item-feedback-error`).length).toBeTruthy()
     expect(console.error).toBeCalledTimes(0)
 
     fireEvent.change(getByLabelText(/name/i), { target: { value: 'name' } })
     fireEvent.change(getByLabelText(/address/i), {
       target: { value: 'address' },
     })
 
+    fireEvent.click(getByText('submit'))
+    expect(console.error).toBeCalledTimes(0)
     await waitFor(() => {
-      fireEvent.click(getByText('submit'))
+      expect(fn.mock.calls[0][0]).toEqual({ name: 'name', address: 'address' })
     })
-    expect(console.error).toBeCalledTimes(0)
-    expect(fn.mock.calls[0][0]).toEqual({ name: 'name', address: 'address' })
   })
 
   test('renders with horizontal layout', async () => {
@@ -151,11 +151,11 @@ describe('Form', () => {
       </Form>
     )
 
+    fireEvent.submit(getByTestId('form'))
+
     await waitFor(() => {
-      fireEvent.submit(getByTestId('form'))
+      expect(getByTestId('form')).toHaveTextContent(`'test' is required`)
     })
-
-    expect(getByTestId('form')).toHaveTextContent(`'test' is required`)
   })
 
   test("`shouldUpdate` shouldn't work with render porps", async () => {