test: use test instead of it

Signed-off-by: Kristóf Marussy <kristof@marussy.com>

1 files changed, 21 insertions, 21 deletions

diff --git a/packages/preload/src/contextBridge/__tests__/createSophieRenderer.spec.ts b/packages/preload/src/contextBridge/__tests__/createSophieRenderer.test.ts
index 4411789..21620ed 100644
--- a/packages/preload/src/contextBridge/__tests__/createSophieRenderer.spec.ts
+++ b/packages/preload/src/contextBridge/__tests__/createSophieRenderer.test.ts
@@ -73,7 +73,7 @@ beforeAll(() => {
 });
 
 describe('createSophieRenderer', () => {
-  it('registers a shared store patch listener', () => {
+  test('registers a shared store patch listener', () => {
     createSophieRenderer(false);
     expect(ipcRenderer.on).toHaveBeenCalledWith(
       MainToRendererIpcMessage.SharedStorePatch,
@@ -103,7 +103,7 @@ describe('SharedStoreConnector', () => {
   });
 
   describe('onSharedStoreChange', () => {
-    it('should request a snapshot from the main process', async () => {
+    test('requests a snapshot from the main process', async () => {
       mocked(ipcRenderer.invoke).mockResolvedValueOnce(snapshot);
       await sut.onSharedStoreChange(listener);
       expect(ipcRenderer.invoke).toHaveBeenCalledWith(
@@ -112,7 +112,7 @@ describe('SharedStoreConnector', () => {
       expect(listener.onSnapshot).toHaveBeenCalledWith(snapshot);
     });
 
-    it('should catch IPC errors without exposing them', async () => {
+    test('catches IPC errors without exposing them', async () => {
       mocked(ipcRenderer.invoke).mockRejectedValue(new Error('s3cr3t'));
       await expect(
         sut.onSharedStoreChange(listener),
@@ -122,7 +122,7 @@ describe('SharedStoreConnector', () => {
   });
 
   describe('dispatchAction', () => {
-    it('should dispatch valid actions', () => {
+    test('dispatches valid actions', () => {
       sut.dispatchAction(action);
       expect(ipcRenderer.send).toHaveBeenCalledWith(
         RendererToMainIpcMessage.DispatchAction,
@@ -130,31 +130,31 @@ describe('SharedStoreConnector', () => {
       );
     });
 
-    it('should not dispatch invalid actions', () => {
+    test('does not dispatch invalid actions', () => {
       expect(() => sut.dispatchAction(invalidAction)).toThrow();
       expect(ipcRenderer.send).not.toHaveBeenCalled();
     });
   });
 
   describe('when no listener is registered', () => {
-    it('should discard the received patch without any error', () => {
+    test('discards the received patch without any error', () => {
       expect(() => onSharedStorePatch(event, patch)).not.toThrow();
     });
   });
 
-  function itRefusesToRegisterAnotherListener(): void {
-    it('should refuse to register another listener', async () => {
+  function testRefusesToRegisterAnotherListener(): void {
+    test('refuses to register another listener', async () => {
       await expect(sut.onSharedStoreChange(listener)).rejects.toBeInstanceOf(
         Error,
       );
     });
   }
 
-  function itDoesNotPassPatchesToTheListener(
-    name = 'should not pass patches to the listener',
+  function testDoesNotPassPatchesToTheListener(
+    name = 'does not pass patches to the listener',
   ): void {
     // eslint-disable-next-line jest/valid-title -- Title is a string parameter.
-    it(name, () => {
+    test(name, () => {
       onSharedStorePatch(event, patch);
       expect(listener.onPatch).not.toHaveBeenCalled();
     });
@@ -166,19 +166,19 @@ describe('SharedStoreConnector', () => {
       await sut.onSharedStoreChange(listener);
     });
 
-    it('should pass patches to the listener', () => {
+    test('passes patches to the listener', () => {
       onSharedStorePatch(event, patch);
       expect(listener.onPatch).toHaveBeenCalledWith(patch);
     });
 
-    it('should catch listener errors', () => {
+    test('catches listener errors', () => {
       mocked(listener.onPatch).mockImplementation(() => {
         throw new Error('listener error');
       });
       expect(() => onSharedStorePatch(event, patch)).not.toThrow();
     });
 
-    itRefusesToRegisterAnotherListener();
+    testRefusesToRegisterAnotherListener();
 
     describe('after the listener threw in onPatch', () => {
       beforeEach(() => {
@@ -189,7 +189,7 @@ describe('SharedStoreConnector', () => {
         listener.onPatch.mockRestore();
       });
 
-      itDoesNotPassPatchesToTheListener('should not pass on patches any more');
+      testDoesNotPassPatchesToTheListener('does not pass on patches any more');
     });
   });
 
@@ -205,9 +205,9 @@ describe('SharedStoreConnector', () => {
       }
     });
 
-    itRefusesToRegisterAnotherListener();
+    testRefusesToRegisterAnotherListener();
 
-    itDoesNotPassPatchesToTheListener();
+    testDoesNotPassPatchesToTheListener();
   });
 
   describe('when a listener failed to register due to listener error', () => {
@@ -223,9 +223,9 @@ describe('SharedStoreConnector', () => {
       }
     });
 
-    itRefusesToRegisterAnotherListener();
+    testRefusesToRegisterAnotherListener();
 
-    itDoesNotPassPatchesToTheListener();
+    testDoesNotPassPatchesToTheListener();
   });
 
   describe('when it is allowed to replace listeners', () => {
@@ -239,7 +239,7 @@ describe('SharedStoreConnector', () => {
       onPatch: jest.fn((_patch: IJsonPatch[]) => {}),
     };
 
-    it('should fetch a second snapshot', async () => {
+    test('fetches a second snapshot', async () => {
       mocked(ipcRenderer.invoke).mockResolvedValueOnce(snapshot2);
       await sut.onSharedStoreChange(listener2);
       expect(ipcRenderer.invoke).toHaveBeenCalledWith(
@@ -248,7 +248,7 @@ describe('SharedStoreConnector', () => {
       expect(listener2.onSnapshot).toHaveBeenCalledWith(snapshot2);
     });
 
-    it('should pass the second snapshot to the new listener', async () => {
+    test('passes the second snapshot to the new listener', async () => {
       mocked(ipcRenderer.invoke).mockResolvedValueOnce(snapshot2);
       await sut.onSharedStoreChange(listener2);
       onSharedStorePatch(event, patch);