From 7aa8a5600d0bb44949749893e0bbc630c06e62be Mon Sep 17 00:00:00 2001 From: Kristóf Marussy Date: Thu, 30 Dec 2021 23:11:08 +0100 Subject: test: Silence logging in preload tests --- packages/preload/src/contextBridge/SophieRendererImpl.ts | 9 +++++---- .../src/contextBridge/__tests__/SophieRendererImpl.spec.ts | 3 +++ 2 files changed, 8 insertions(+), 4 deletions(-) (limited to 'packages/preload') diff --git a/packages/preload/src/contextBridge/SophieRendererImpl.ts b/packages/preload/src/contextBridge/SophieRendererImpl.ts index 4c24b74..f3c07c5 100644 --- a/packages/preload/src/contextBridge/SophieRendererImpl.ts +++ b/packages/preload/src/contextBridge/SophieRendererImpl.ts @@ -19,6 +19,7 @@ */ import { ipcRenderer } from 'electron'; +import log from 'loglevel'; import type { IJsonPatch } from 'mobx-state-tree'; import { Action, @@ -41,7 +42,7 @@ class SophieRendererImpl implements SophieRenderer { // `mobx-state-tree` will validate the patch, so we can safely cast here. this.listener?.onPatch(patch as IJsonPatch); } catch (err) { - console.error('Shared store listener onPatch failed', err); + log.error('Shared store listener onPatch failed', err); this.listener = null; } }); @@ -58,7 +59,7 @@ class SophieRendererImpl implements SophieRenderer { snapshot = await ipcRenderer.invoke(RendererToMainIpcMessage.GetSharedStoreSnapshot); success = true; } catch (err) { - console.error('Failed to get initial shared store snapshot', err); + log.error('Failed to get initial shared store snapshot', err); } if (success) { if (sharedStore.is(snapshot)) { @@ -66,7 +67,7 @@ class SophieRendererImpl implements SophieRenderer { this.listener = listener; return; } - console.error('Got invalid initial shared store snapshot', snapshot); + log.error('Got invalid initial shared store snapshot', snapshot); } throw new Error('Failed to connect to shared store'); } @@ -80,7 +81,7 @@ class SophieRendererImpl implements SophieRenderer { } catch (err) { // Do not leak IPC failure details into the main world. const message = 'Failed to dispatch action'; - console.error(message, actionToDispatch, err); + log.error(message, actionToDispatch, err); throw new Error(message); } } diff --git a/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts b/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts index e40dcf5..41937c2 100644 --- a/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts +++ b/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts @@ -20,6 +20,7 @@ import { describe, it, jest } from '@jest/globals'; import { mocked } from 'jest-mock'; +import log from 'loglevel'; import type { IJsonPatch } from 'mobx-state-tree'; import { Action, @@ -66,6 +67,8 @@ const invalidAction = { action: 'not-a-valid-action', } as unknown as Action; +log.disableAll(); + describe('createSophieRenderer', () => { it('registers a shared store patch listener', () => { createSophieRenderer(false); -- cgit v1.2.3-54-g00ecf