From d85f09cbed5f3d2501f791e689011ae127df1cbb Mon Sep 17 00:00:00 2001 From: Kristóf Marussy Date: Sun, 9 Jan 2022 20:33:53 +0100 Subject: build: Add prettier MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit eslint will also enforce prettier rules, so there is no need to call prettier separately in CI. Signed-off-by: Kristóf Marussy --- .../__tests__/createSophieRenderer.spec.ts | 56 +++++++++++++++------- 1 file changed, 39 insertions(+), 17 deletions(-) (limited to 'packages/preload/src/contextBridge/__tests__') diff --git a/packages/preload/src/contextBridge/__tests__/createSophieRenderer.spec.ts b/packages/preload/src/contextBridge/__tests__/createSophieRenderer.spec.ts index a38dbac..f63c3f6 100644 --- a/packages/preload/src/contextBridge/__tests__/createSophieRenderer.spec.ts +++ b/packages/preload/src/contextBridge/__tests__/createSophieRenderer.spec.ts @@ -40,9 +40,12 @@ jest.unstable_mockModule('electron', () => ({ const { ipcRenderer } = await import('electron'); -const { default: createSophieRenderer } = await import('../createSophieRenderer'); +const { default: createSophieRenderer } = await import( + '../createSophieRenderer' +); -const event: Electron.IpcRendererEvent = null as unknown as Electron.IpcRendererEvent; +const event: Electron.IpcRendererEvent = + null as unknown as Electron.IpcRendererEvent; const snapshot: SharedStoreSnapshotIn = { shouldUseDarkColors: true, @@ -83,7 +86,10 @@ describe('createSophieRenderer', () => { describe('SharedStoreConnector', () => { let sut: SophieRenderer; - let onSharedStorePatch: (eventArg: Electron.IpcRendererEvent, patchArg: unknown) => void; + let onSharedStorePatch: ( + eventArg: Electron.IpcRendererEvent, + patchArg: unknown, + ) => void; const listener = { // eslint-disable-next-line @typescript-eslint/no-unused-vars, no-unused-vars onSnapshot: jest.fn((_snapshot: SharedStoreSnapshotIn) => {}), @@ -102,22 +108,25 @@ describe('SharedStoreConnector', () => { 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(ipcRenderer.invoke).toBeCalledWith( + RendererToMainIpcMessage.GetSharedStoreSnapshot, + ); expect(listener.onSnapshot).toBeCalledWith(snapshot); }); 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/), - ); + await expect( + sut.onSharedStoreChange(listener), + ).rejects.not.toHaveProperty('message', expect.stringMatching(/s3cr3t/)); expect(listener.onSnapshot).not.toBeCalled(); }); it('should not pass on invalid snapshots', async () => { mocked(ipcRenderer.invoke).mockResolvedValueOnce(invalidSnapshot); - await expect(sut.onSharedStoreChange(listener)).rejects.toBeInstanceOf(Error); + await expect(sut.onSharedStoreChange(listener)).rejects.toBeInstanceOf( + Error, + ); expect(listener.onSnapshot).not.toBeCalled(); }); }); @@ -125,7 +134,10 @@ describe('SharedStoreConnector', () => { describe('dispatchAction', () => { it('should dispatch valid actions', () => { sut.dispatchAction(action); - expect(ipcRenderer.send).toBeCalledWith(RendererToMainIpcMessage.DispatchAction, action); + expect(ipcRenderer.send).toBeCalledWith( + RendererToMainIpcMessage.DispatchAction, + action, + ); }); it('should not dispatch invalid actions', () => { @@ -142,7 +154,9 @@ describe('SharedStoreConnector', () => { function itRefusesToRegisterAnotherListener(): void { it('should refuse to register another listener', async () => { - await expect(sut.onSharedStoreChange(listener)).rejects.toBeInstanceOf(Error); + await expect(sut.onSharedStoreChange(listener)).rejects.toBeInstanceOf( + Error, + ); }); } @@ -167,7 +181,9 @@ describe('SharedStoreConnector', () => { }); it('should catch listener errors', () => { - mocked(listener.onPatch).mockImplementation(() => { throw new Error(); }); + mocked(listener.onPatch).mockImplementation(() => { + throw new Error(); + }); onSharedStorePatch(event, patch); }); @@ -175,7 +191,9 @@ describe('SharedStoreConnector', () => { describe('after the listener threw in onPatch', () => { beforeEach(() => { - mocked(listener.onPatch).mockImplementation(() => { throw new Error(); }); + mocked(listener.onPatch).mockImplementation(() => { + throw new Error(); + }); onSharedStorePatch(event, patch); listener.onPatch.mockRestore(); }); @@ -217,7 +235,9 @@ describe('SharedStoreConnector', () => { describe('when a listener failed to register due to listener error', () => { beforeEach(async () => { mocked(ipcRenderer.invoke).mockResolvedValueOnce(snapshot); - mocked(listener.onSnapshot).mockImplementation(() => { throw new Error(); }); + mocked(listener.onSnapshot).mockImplementation(() => { + throw new Error(); + }); try { await sut.onSharedStoreChange(listener); } catch { @@ -236,15 +256,17 @@ describe('SharedStoreConnector', () => { }; const listener2 = { // eslint-disable-next-line @typescript-eslint/no-unused-vars, no-unused-vars - onSnapshot: jest.fn((_snapshot: SharedStoreSnapshotIn) => { }), + onSnapshot: jest.fn((_snapshot: SharedStoreSnapshotIn) => {}), // eslint-disable-next-line @typescript-eslint/no-unused-vars, no-unused-vars - onPatch: jest.fn((_patch: IJsonPatch) => { }), + onPatch: jest.fn((_patch: IJsonPatch) => {}), }; it('should fetch a second snapshot', async () => { mocked(ipcRenderer.invoke).mockResolvedValueOnce(snapshot2); await sut.onSharedStoreChange(listener2); - expect(ipcRenderer.invoke).toBeCalledWith(RendererToMainIpcMessage.GetSharedStoreSnapshot); + expect(ipcRenderer.invoke).toBeCalledWith( + RendererToMainIpcMessage.GetSharedStoreSnapshot, + ); expect(listener2.onSnapshot).toBeCalledWith(snapshot2); }); -- cgit v1.2.3-54-g00ecf