From 305f79990fff188d88bb114d03f0c2dd9a16e27c Mon Sep 17 00:00:00 2001
From: Eelco Wiersma <contact@pagebakers.nl>
Date: Thu, 4 Aug 2022 18:40:09 +0200
Subject: [PATCH] fix: make sure snackbar tests dont fail

---
 .../tests/use-snackbar.test.tsx               | 46 +++++--------------
 1 file changed, 12 insertions(+), 34 deletions(-)

diff --git a/packages/saas-ui-snackbar/tests/use-snackbar.test.tsx b/packages/saas-ui-snackbar/tests/use-snackbar.test.tsx
index 878e0145c..c818e9e57 100644
--- a/packages/saas-ui-snackbar/tests/use-snackbar.test.tsx
+++ b/packages/saas-ui-snackbar/tests/use-snackbar.test.tsx
@@ -1,55 +1,33 @@
 import * as React from 'react'
-import { render, act, hooks, screen, waitFor } from '@saas-ui/test-utils'
+import { render, screen } from '@saas-ui/test-utils'
 
 import { useSnackbar } from '../src'
 
 test('can accept shorthand options', async () => {
   const title = 'Info'
 
-  const { result } = hooks.render(() => useSnackbar())
-
-  hooks.act(() => {
-    result.current.info(title)
-  })
-
-  const allByTitle = await screen.findAllByRole('alert', { name: title })
-
-  expect(allByTitle).toHaveLength(1)
-})
-
-test('can accept success handler', async () => {
-  const title = 'Success'
-
-  const { result } = hooks.render(() => useSnackbar())
-
-  hooks.act(() => {
-    result.current.success(title)
-  })
-
-  const allByTitle = await screen.findAllByRole('alert', { name: title })
+  const TestComponent = () => {
+    const snackbar = useSnackbar()
 
-  expect(allByTitle).toHaveLength(1)
-})
+    return <button onClick={() => snackbar.info(title)}>Snackbar</button>
+  }
 
-test('can accept error handler', async () => {
-  const title = 'Error'
+  const { user } = render(<TestComponent />)
 
-  const { result } = hooks.render(() => useSnackbar())
+  const button = await screen.findByText('Snackbar')
+  await user.click(button)
 
-  hooks.act(() => {
-    result.current.error(title)
+  const toast = await screen.findByRole('alert', {
+    name: title,
   })
-
-  const allByTitle = await screen.findAllByRole('alert', { name: title })
-
-  expect(allByTitle).toHaveLength(1)
+  expect(toast).toBeInTheDocument()
 })
 
 it('should allow promise toast chainings', async () => {
   const loadingTitle = 'Toast is loading'
   const successTitle = 'Promise resolved'
   const errorTitle = 'Error occurred'
-  const sleepTime = 200
+  const sleepTime = 500
   const dummyPromise = new Promise<{ payload: string }>((r) =>
     setTimeout(r, sleepTime, { payload: successTitle })
   )