test: format test

Signed-off-by: Adam Setch <adam.setch@outlook.com>

Author: setchy

src/main/updater.test.ts

@@ -110,7 +110,6 @@ describe('main/updater.ts', () => {
         }),
       );
       expect(autoUpdater.quitAndInstall).not.toHaveBeenCalled();
-      // Menu state updates invoked
       expect(menuBuilder.setUpdateAvailableMenuVisibility).toHaveBeenCalledWith(
         false,
       );
@@ -126,7 +125,9 @@ describe('main/updater.ts', () => {
       });
 
       await updater.start();
+
       emit('update-downloaded', { releaseName: 'v9.9.9' });
+
       // Allow then() of showMessageBox promise to resolve
       await Promise.resolve();
 
@@ -137,7 +138,9 @@ describe('main/updater.ts', () => {
   describe('update event handlers & scheduling', () => {
     it('skips when app is not packaged', async () => {
       Object.defineProperty(menubar.app, 'isPackaged', { value: false });
+
       await updater.start();
+
       expect(logInfo).toHaveBeenCalledWith(
         'app updater',
         'Skipping updater since app is in development mode',
@@ -147,7 +150,9 @@ describe('main/updater.ts', () => {
 
     it('handles checking-for-update', async () => {
       await updater.start();
+
       emit('checking-for-update');
+
       expect(menuBuilder.setCheckForUpdatesMenuEnabled).toHaveBeenCalledWith(
         false,
       );
@@ -158,7 +163,9 @@ describe('main/updater.ts', () => {
 
     it('handles update-available', async () => {
       await updater.start();
+
       emit('update-available');
+
       expect(menuBuilder.setUpdateAvailableMenuVisibility).toHaveBeenCalledWith(
         true,
       );
@@ -169,15 +176,19 @@ describe('main/updater.ts', () => {
 
     it('handles download-progress', async () => {
       await updater.start();
+
       emit('download-progress', { percent: 12.3456 });
+
       expect(menubar.tray.setToolTip).toHaveBeenCalledWith(
         expect.stringContaining('12.35%'),
       );
     });
 
     it('handles update-not-available', async () => {
       await updater.start();
+
       emit('update-not-available');
+
       expect(menuBuilder.setCheckForUpdatesMenuEnabled).toHaveBeenCalledWith(
         true,
       );
@@ -197,9 +208,8 @@ describe('main/updater.ts', () => {
       try {
         await updater.start();
 
-        menuBuilder.setNoUpdateAvailableMenuVisibility.mockClear();
-
         emit('update-not-available');
+
         // Immediately shows the message
         expect(
           menuBuilder.setNoUpdateAvailableMenuVisibility,
@@ -219,16 +229,17 @@ describe('main/updater.ts', () => {
       vi.useFakeTimers();
       try {
         await updater.start();
-        menuBuilder.setNoUpdateAvailableMenuVisibility.mockClear();
 
         emit('update-not-available');
+
         // Message shown
         expect(
           menuBuilder.setNoUpdateAvailableMenuVisibility,
         ).toHaveBeenCalledWith(true);
 
         // New check should hide immediately and clear pending timeout
         emit('checking-for-update');
+
         expect(
           menuBuilder.setNoUpdateAvailableMenuVisibility,
         ).toHaveBeenLastCalledWith(false);
@@ -247,7 +258,9 @@ describe('main/updater.ts', () => {
 
     it('handles update-cancelled (reset state)', async () => {
       await updater.start();
+
       emit('update-cancelled');
+
       expect(menubar.tray.setToolTip).toHaveBeenCalledWith(APPLICATION.NAME);
       expect(menuBuilder.setCheckForUpdatesMenuEnabled).toHaveBeenCalledWith(
         true,
@@ -256,8 +269,10 @@ describe('main/updater.ts', () => {
 
     it('handles error (reset + logError)', async () => {
       await updater.start();
+
       const err = new Error('failure');
       emit('error', err);
+
       expect(logError).toHaveBeenCalledWith(
         'auto updater',
         'Error checking for update',
@@ -276,6 +291,7 @@ describe('main/updater.ts', () => {
         }) as unknown as typeof setInterval);
       try {
         await updater.start();
+
         // At minimum the initial check should have occurred
         const callCount = vi.mocked(autoUpdater.checkForUpdatesAndNotify).mock
           .calls.length;

src/preload/utils.test.ts

@@ -29,12 +29,6 @@ vi.mock('electron', () => {
 import { ipcRenderer } from 'electron';
 
 describe('preload/utils', () => {
-  afterEach(() => {
-    vi.mocked(ipcRenderer.send).mockClear();
-    vi.mocked(ipcRenderer.invoke).mockClear();
-    vi.mocked(ipcRenderer.on).mockClear();
-  });
-
   it('sendMainEvent forwards to ipcRenderer.send', () => {
     sendMainEvent(EVENTS.WINDOW_SHOW);
     expect(ipcRenderer.send).toHaveBeenCalledWith(