diff options
author | Kristóf Marussy <kristof@marussy.com> | 2021-12-30 23:11:08 +0100 |
---|---|---|
committer | Kristóf Marussy <kristof@marussy.com> | 2021-12-30 23:11:08 +0100 |
commit | 7aa8a5600d0bb44949749893e0bbc630c06e62be (patch) | |
tree | 97dadae766de4959a06d3df552e65e41fa7525ec /packages/preload | |
parent | build: Fully switch over to ESM (diff) | |
download | sophie-7aa8a5600d0bb44949749893e0bbc630c06e62be.tar.gz sophie-7aa8a5600d0bb44949749893e0bbc630c06e62be.tar.zst sophie-7aa8a5600d0bb44949749893e0bbc630c06e62be.zip |
test: Silence logging in preload tests
Diffstat (limited to 'packages/preload')
-rw-r--r-- | packages/preload/src/contextBridge/SophieRendererImpl.ts | 9 | ||||
-rw-r--r-- | packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts | 3 |
2 files changed, 8 insertions, 4 deletions
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 @@ | |||
19 | */ | 19 | */ |
20 | 20 | ||
21 | import { ipcRenderer } from 'electron'; | 21 | import { ipcRenderer } from 'electron'; |
22 | import log from 'loglevel'; | ||
22 | import type { IJsonPatch } from 'mobx-state-tree'; | 23 | import type { IJsonPatch } from 'mobx-state-tree'; |
23 | import { | 24 | import { |
24 | Action, | 25 | Action, |
@@ -41,7 +42,7 @@ class SophieRendererImpl implements SophieRenderer { | |||
41 | // `mobx-state-tree` will validate the patch, so we can safely cast here. | 42 | // `mobx-state-tree` will validate the patch, so we can safely cast here. |
42 | this.listener?.onPatch(patch as IJsonPatch); | 43 | this.listener?.onPatch(patch as IJsonPatch); |
43 | } catch (err) { | 44 | } catch (err) { |
44 | console.error('Shared store listener onPatch failed', err); | 45 | log.error('Shared store listener onPatch failed', err); |
45 | this.listener = null; | 46 | this.listener = null; |
46 | } | 47 | } |
47 | }); | 48 | }); |
@@ -58,7 +59,7 @@ class SophieRendererImpl implements SophieRenderer { | |||
58 | snapshot = await ipcRenderer.invoke(RendererToMainIpcMessage.GetSharedStoreSnapshot); | 59 | snapshot = await ipcRenderer.invoke(RendererToMainIpcMessage.GetSharedStoreSnapshot); |
59 | success = true; | 60 | success = true; |
60 | } catch (err) { | 61 | } catch (err) { |
61 | console.error('Failed to get initial shared store snapshot', err); | 62 | log.error('Failed to get initial shared store snapshot', err); |
62 | } | 63 | } |
63 | if (success) { | 64 | if (success) { |
64 | if (sharedStore.is(snapshot)) { | 65 | if (sharedStore.is(snapshot)) { |
@@ -66,7 +67,7 @@ class SophieRendererImpl implements SophieRenderer { | |||
66 | this.listener = listener; | 67 | this.listener = listener; |
67 | return; | 68 | return; |
68 | } | 69 | } |
69 | console.error('Got invalid initial shared store snapshot', snapshot); | 70 | log.error('Got invalid initial shared store snapshot', snapshot); |
70 | } | 71 | } |
71 | throw new Error('Failed to connect to shared store'); | 72 | throw new Error('Failed to connect to shared store'); |
72 | } | 73 | } |
@@ -80,7 +81,7 @@ class SophieRendererImpl implements SophieRenderer { | |||
80 | } catch (err) { | 81 | } catch (err) { |
81 | // Do not leak IPC failure details into the main world. | 82 | // Do not leak IPC failure details into the main world. |
82 | const message = 'Failed to dispatch action'; | 83 | const message = 'Failed to dispatch action'; |
83 | console.error(message, actionToDispatch, err); | 84 | log.error(message, actionToDispatch, err); |
84 | throw new Error(message); | 85 | throw new Error(message); |
85 | } | 86 | } |
86 | } | 87 | } |
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 @@ | |||
20 | 20 | ||
21 | import { describe, it, jest } from '@jest/globals'; | 21 | import { describe, it, jest } from '@jest/globals'; |
22 | import { mocked } from 'jest-mock'; | 22 | import { mocked } from 'jest-mock'; |
23 | import log from 'loglevel'; | ||
23 | import type { IJsonPatch } from 'mobx-state-tree'; | 24 | import type { IJsonPatch } from 'mobx-state-tree'; |
24 | import { | 25 | import { |
25 | Action, | 26 | Action, |
@@ -66,6 +67,8 @@ const invalidAction = { | |||
66 | action: 'not-a-valid-action', | 67 | action: 'not-a-valid-action', |
67 | } as unknown as Action; | 68 | } as unknown as Action; |
68 | 69 | ||
70 | log.disableAll(); | ||
71 | |||
69 | describe('createSophieRenderer', () => { | 72 | describe('createSophieRenderer', () => { |
70 | it('registers a shared store patch listener', () => { | 73 | it('registers a shared store patch listener', () => { |
71 | createSophieRenderer(false); | 74 | createSophieRenderer(false); |