test: vitest setup and cleanup (#2726)

* test(vitest): cleanup

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

* test: format test

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

---------

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

Author: setchy

src/main/events.test.ts

@@ -19,10 +19,6 @@ type MockMenubar = {
 };
 
 describe('main/events', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
   it('onMainEvent registers ipcMain.on listener', () => {
     const listenerMock = vi.fn();
 

src/main/handlers/app.test.ts

@@ -28,8 +28,6 @@ describe('main/handlers/app.ts', () => {
   let menubar: Menubar;
 
   beforeEach(() => {
-    vi.clearAllMocks();
-
     menubar = {
       showWindow: vi.fn(),
       hideWindow: vi.fn(),

src/main/handlers/storage.test.ts

@@ -20,10 +20,6 @@ vi.mock('../../shared/logger', () => ({
 }));
 
 describe('main/handlers/storage.ts', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
   describe('registerStorageHandlers', () => {
     it('registers handlers without throwing', () => {
       expect(() => registerStorageHandlers()).not.toThrow();

src/main/handlers/system.test.ts

@@ -26,8 +26,6 @@ describe('main/handlers/system.ts', () => {
   let menubar: Menubar;
 
   beforeEach(() => {
-    vi.clearAllMocks();
-
     menubar = {
       showWindow: vi.fn(),
       hideWindow: vi.fn(),

src/main/handlers/tray.test.ts

@@ -20,8 +20,6 @@ describe('main/handlers/tray.ts', () => {
   let menubar: Menubar;
 
   beforeEach(() => {
-    vi.clearAllMocks();
-
     menubar = {
       tray: {
         isDestroyed: vi.fn().mockReturnValue(false),

src/main/lifecycle/first-run.test.ts

@@ -52,7 +52,6 @@ function configPath() {
 
 describe('main/lifecycle/first-run', () => {
   beforeEach(() => {
-    vi.clearAllMocks();
     mac = true;
   });
 

src/main/lifecycle/reset.test.ts

@@ -23,10 +23,6 @@ function createMb() {
 }
 
 describe('main/lifecycle/reset.ts', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
   it('sends reset event and quits when user confirms', () => {
     vi.mocked(dialog.showMessageBoxSync).mockReturnValue(1);
     const mb = createMb();

src/main/lifecycle/startup.test.ts

@@ -43,10 +43,6 @@ function createMb() {
 }
 
 describe('main/lifecycle/startup.ts', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
   describe('initializeAppLifecycle', () => {
     it('registers menubar ready handler', () => {
       const mb = createMb();

src/main/lifecycle/window.test.ts

@@ -13,8 +13,6 @@ describe('main/lifecycle/window.ts', () => {
   let menubar: Menubar;
 
   beforeEach(() => {
-    vi.clearAllMocks();
-
     menubar = {
       hideWindow: vi.fn(),
       tray: {

src/main/menu.test.ts

@@ -88,7 +88,6 @@ describe('main/menu.ts', () => {
   };
 
   beforeEach(() => {
-    vi.clearAllMocks();
     vi.mocked(isMacOS).mockReturnValue(false);
     menuItemInstances.length = 0; // Clear tracked instances
     menubar = { app: { quit: vi.fn() } } as unknown as Menubar;