test: fix test error logs

will-stone · Dec 16, 2023 · 3d2cbb7 · 3d2cbb7
1 parent a8f4b74
commit 3d2cbb7
Show file tree

Hide file tree

Showing 4 changed files with 30 additions and 30 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -98,7 +98,7 @@
     "style-loader": "^3.3.3",
     "tailwindcss": "^3.3.6",
     "tidy-url": "^1.10.1",
-    "tings": "^8.1.4",
+    "tings": "^9.1.0",
     "ts-jest": "^29.1.1",
     "ts-loader": "^9.5.1",
     "ts-node": "^10.9.2",

diff --git a/src/renderers/picker/components/organisms/apps.test.tsx b/src/renderers/picker/components/organisms/apps.test.tsx
@@ -29,10 +29,10 @@ beforeAll(() => {
   })
 })
 
-test('kitchen sink', () => {
+test('kitchen sink', async () => {
   render(<Wrapper />)
   const win = new electron.BrowserWindow()
-  win.webContents.send(
+  await win.webContents.send(
     Channel.MAIN,
     retrievedInstalledApps(['Firefox', 'Safari', 'Brave Browser']),
   )
@@ -48,7 +48,7 @@ test('kitchen sink', () => {
 
   expect(screen.getAllByRole('button', { name: /[A-z]+ App/u })).toHaveLength(3)
 
-  win.webContents.send(
+  await win.webContents.send(
     Channel.MAIN,
     receivedRendererStartupSignal({
       data: {
@@ -104,7 +104,7 @@ test('kitchen sink', () => {
 
   // Correct info sent to main when app clicked
   const url = 'http://example.com'
-  win.webContents.send(Channel.MAIN, openedUrl(url))
+  await win.webContents.send(Channel.MAIN, openedUrl(url))
   fireEvent.click(screen.getByRole('button', { name: 'Brave Browser App' }), {
     altKey: true,
   })
@@ -121,10 +121,10 @@ test('kitchen sink', () => {
   )
 })
 
-test('should show spinner when no installed apps are found', () => {
+test('should show spinner when no installed apps are found', async () => {
   render(<Wrapper />)
   const win = new electron.BrowserWindow()
-  win.webContents.send(
+  await win.webContents.send(
     Channel.MAIN,
     receivedRendererStartupSignal({
       data: defaultData,
@@ -145,11 +145,11 @@ test('should show spinner when no installed apps are found', () => {
   expect(screen.getByRole('alert', { name: 'Loading browsers' })).toBeVisible()
 })
 
-test('should use hotkey', () => {
+test('should use hotkey', async () => {
   render(<Wrapper />)
   const win = new electron.BrowserWindow()
-  win.webContents.send(Channel.MAIN, retrievedInstalledApps(['Safari']))
-  win.webContents.send(
+  await win.webContents.send(Channel.MAIN, retrievedInstalledApps(['Safari']))
+  await win.webContents.send(
     Channel.MAIN,
     receivedRendererStartupSignal({
       data: defaultData,
@@ -169,7 +169,7 @@ test('should use hotkey', () => {
   )
 
   const url = 'http://example.com'
-  win.webContents.send(Channel.MAIN, openedUrl(url))
+  await win.webContents.send(Channel.MAIN, openedUrl(url))
   fireEvent.keyDown(document, { code: 'KeyS', key: 'S', keyCode: 83 })
   expect(electron.ipcRenderer.send).toHaveBeenCalledWith(
     Channel.PICKER,
@@ -186,12 +186,12 @@ test('should use hotkey', () => {
   )
 })
 
-test('should use hotkey with alt', () => {
+test('should use hotkey with alt', async () => {
   render(<Wrapper />)
   const win = new electron.BrowserWindow()
-  win.webContents.send(Channel.MAIN, retrievedInstalledApps(['Safari']))
+  await win.webContents.send(Channel.MAIN, retrievedInstalledApps(['Safari']))
 
-  win.webContents.send(
+  await win.webContents.send(
     Channel.MAIN,
     receivedRendererStartupSignal({
       data: defaultData,
@@ -211,7 +211,7 @@ test('should use hotkey with alt', () => {
   )
 
   const url = 'http://example.com'
-  win.webContents.send(Channel.MAIN, openedUrl(url))
+  await win.webContents.send(Channel.MAIN, openedUrl(url))
   fireEvent.keyDown(document, {
     altKey: true,
     code: 'KeyS',
@@ -233,11 +233,11 @@ test('should use hotkey with alt', () => {
   )
 })
 
-test('should hold shift', () => {
+test('should hold shift', async () => {
   render(<Wrapper />)
   const win = new electron.BrowserWindow()
-  win.webContents.send(Channel.MAIN, retrievedInstalledApps(['Firefox']))
-  win.webContents.send(Channel.MAIN, openedUrl('http://example.com'))
+  await win.webContents.send(Channel.MAIN, retrievedInstalledApps(['Firefox']))
+  await win.webContents.send(Channel.MAIN, openedUrl('http://example.com'))
   fireEvent.click(screen.getByRole('button', { name: 'Firefox App' }), {
     shiftKey: true,
   })
@@ -254,12 +254,12 @@ test('should hold shift', () => {
   )
 })
 
-test('should order tiles', () => {
+test('should order tiles', async () => {
   render(<Wrapper />)
 
   const win = new electron.BrowserWindow()
 
-  win.webContents.send(
+  await win.webContents.send(
     Channel.MAIN,
     receivedRendererStartupSignal({
       data: defaultData,
@@ -272,7 +272,7 @@ test('should order tiles', () => {
     }),
   )
 
-  win.webContents.send(
+  await win.webContents.send(
     Channel.MAIN,
     retrievedInstalledApps([
       'Firefox',
@@ -287,21 +287,21 @@ test('should order tiles', () => {
 
   expect(apps).toHaveLength(5)
 
-  win.webContents.send(
+  await win.webContents.send(
     Channel.MAIN,
     reorderedApp({
       destinationName: 'Firefox',
       sourceName: 'Safari',
     }),
   )
-  win.webContents.send(
+  await win.webContents.send(
     Channel.MAIN,
     reorderedApp({
       destinationName: 'Firefox',
       sourceName: 'Opera',
     }),
   )
-  win.webContents.send(
+  await win.webContents.send(
     Channel.MAIN,
     reorderedApp({
       destinationName: 'Firefox',

diff --git a/src/renderers/picker/components/organisms/url-bar.test.tsx b/src/renderers/picker/components/organisms/url-bar.test.tsx
@@ -21,15 +21,15 @@ beforeAll(() => {
   })
 })
 
-test('url bar', () => {
+test('url bar', async () => {
   render(<Wrapper />)
   const win = new electron.BrowserWindow()
   const protocol = 'http://'
   const host = 'example.com'
   const port = ':8000'
   const rest = '/foo?bar=moo'
   const url = `${protocol}${host}${port}${rest}`
-  win.webContents.send(Channel.MAIN, openedUrl(url))
+  await win.webContents.send(Channel.MAIN, openedUrl(url))
   expect(screen.queryByText(protocol)).not.toBeInTheDocument()
   expect(screen.getByText(host + port)).toBeVisible()
   expect(screen.queryByText(rest)).not.toBeInTheDocument()