1 files changed, 19 insertions, 17 deletions

diff --git a/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts b/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts
index 41937c2..fdd1cc5 100644
--- a/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts
+++ b/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts
@@ -18,7 +18,7 @@
  * SPDX-License-Identifier: AGPL-3.0-only
  */
 
-import { describe, it, jest } from '@jest/globals';
+import { jest } from '@jest/globals';
 import { mocked } from 'jest-mock';
 import log from 'loglevel';
 import type { IJsonPatch } from 'mobx-state-tree';
@@ -67,7 +67,9 @@ const invalidAction = {
   action: 'not-a-valid-action',
 } as unknown as Action;
 
-log.disableAll();
+beforeAll(() => {
+  log.disableAll();
+});
 
 describe('createSophieRenderer', () => {
   it('registers a shared store patch listener', () => {
@@ -95,59 +97,59 @@ describe('SophieRendererImpl', () => {
   });
 
   describe('onSharedStoreChange', () => {
-    it('requests a snapshot from the main process', async () => {
+    it('should request a snapshot from the main process', async () => {
       mocked(ipcRenderer.invoke).mockResolvedValueOnce(snapshot);
       await sut.onSharedStoreChange(listener);
       expect(ipcRenderer.invoke).toBeCalledWith(RendererToMainIpcMessage.GetSharedStoreSnapshot);
       expect(listener.onSnapshot).toBeCalledWith(snapshot);
     });
 
-    it('catches IPC errors without exposing them', async () => {
+    it('should catch IPC errors without exposing them', async () => {
       mocked(ipcRenderer.invoke).mockRejectedValue(new Error('s3cr3t'));
       await expect(sut.onSharedStoreChange(listener)).rejects.not.toHaveProperty(
         'message',
         expect.stringMatching(/s3cr3t/),
       );
-      expect(listener.onSnapshot).toBeCalledTimes(0);
+      expect(listener.onSnapshot).not.toBeCalled();
     });
 
-    it('does not pass on invalid snapshots', async () => {
+    it('should not pass on invalid snapshots', async () => {
       mocked(ipcRenderer.invoke).mockResolvedValueOnce(invalidSnapshot);
       await expect(sut.onSharedStoreChange(listener)).rejects.toBeInstanceOf(Error);
-      expect(listener.onSnapshot).toBeCalledTimes(0);
+      expect(listener.onSnapshot).not.toBeCalled();
     });
   });
 
   describe('dispatchAction', () => {
-    it('dispatched valid actions', () => {
+    it('should dispatch valid actions', () => {
       sut.dispatchAction(action);
       expect(ipcRenderer.send).toBeCalledWith(RendererToMainIpcMessage.DispatchAction, action);
     });
 
-    it('does not dispatch invalid actions', () => {
+    it('should not dispatch invalid actions', () => {
       expect(() => sut.dispatchAction(invalidAction)).toThrowError();
-      expect(ipcRenderer.send).toBeCalledTimes(0);
+      expect(ipcRenderer.send).not.toBeCalled();
     });
   });
 
   describe('when no listener is registered', () => {
-    it('discards the received patch without any error', () => {
+    it('should discard the received patch without any error', () => {
       onSharedStorePatch(event, patch);
     });
   });
 
   function itRefusesToRegisterAnotherListener() {
-    it('refuses to register another listener', async () => {
+    it('should refuse to register another listener', async () => {
       await expect(sut.onSharedStoreChange(listener)).rejects.toBeInstanceOf(Error);
     });
   }
 
   function itDoesNotPassPatchesToTheListener(
-    name: string = 'does not pass patches to the listener',
+    name: string = 'should not pass patches to the listener',
   ) {
     it(name, () => {
       onSharedStorePatch(event, patch);
-      expect(listener.onPatch).toBeCalledTimes(0);
+      expect(listener.onPatch).not.toBeCalled();
     });
   }
 
@@ -157,12 +159,12 @@ describe('SophieRendererImpl', () => {
       await sut.onSharedStoreChange(listener);
     });
 
-    it('passes patches to the listener', () => {
+    it('should pass patches to the listener', () => {
       onSharedStorePatch(event, patch);
       expect(listener.onPatch).toBeCalledWith(patch);
     });
 
-    it('catches listener errors', () => {
+    it('should catch listener errors', () => {
       mocked(listener.onPatch).mockImplementation(() => { throw new Error(); });
       onSharedStorePatch(event, patch);
     });
@@ -176,7 +178,7 @@ describe('SophieRendererImpl', () => {
         listener.onPatch.mockRestore();
       });
 
-      itDoesNotPassPatchesToTheListener('does not pass on patches any more');
+      itDoesNotPassPatchesToTheListener('should not pass on patches any more');
     });
   });