diff options
author | Kristóf Marussy <kristof@marussy.com> | 2021-12-27 02:24:22 +0100 |
---|---|---|
committer | Kristóf Marussy <kristof@marussy.com> | 2021-12-27 02:37:38 +0100 |
commit | 11cf57ce92aff4dcdb504194ea5c381ac400c5c5 (patch) | |
tree | 2fb6fbc28903d61d1a79f2cf95753d8b83eaf4e4 /packages/preload/src/contextBridge/__tests__ | |
parent | test: Add preload unit tests (diff) | |
download | sophie-11cf57ce92aff4dcdb504194ea5c381ac400c5c5.tar.gz sophie-11cf57ce92aff4dcdb504194ea5c381ac400c5c5.tar.zst sophie-11cf57ce92aff4dcdb504194ea5c381ac400c5c5.zip |
refactor: Simplify IpcRendererService and its spec
Diffstat (limited to 'packages/preload/src/contextBridge/__tests__')
-rw-r--r-- | packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts b/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts index c0e5ec2..bd2b1d5 100644 --- a/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts +++ b/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts | |||
@@ -22,10 +22,10 @@ import { mocked } from 'jest-mock'; | |||
22 | import type { IJsonPatch } from 'mobx-state-tree'; | 22 | import type { IJsonPatch } from 'mobx-state-tree'; |
23 | import type { Action, SharedStoreSnapshotIn, SophieRenderer } from '@sophie/shared'; | 23 | import type { Action, SharedStoreSnapshotIn, SophieRenderer } from '@sophie/shared'; |
24 | 24 | ||
25 | import { RendererToMainIpcService } from '../../services/RendererToMainIpcService'; | 25 | import { IpcRendererService } from '../../services/IpcRendererService'; |
26 | import { createSophieRenderer } from '../SophieRendererImpl'; | 26 | import { createSophieRenderer } from '../SophieRendererImpl'; |
27 | 27 | ||
28 | jest.mock('../../services/RendererToMainIpcService'); | 28 | jest.mock('../../services/IpcRendererService'); |
29 | 29 | ||
30 | const snapshot: SharedStoreSnapshotIn = { | 30 | const snapshot: SharedStoreSnapshotIn = { |
31 | shouldUseDarkColors: true, | 31 | shouldUseDarkColors: true, |
@@ -52,16 +52,15 @@ const invalidAction = { | |||
52 | 52 | ||
53 | describe('constructor', () => { | 53 | describe('constructor', () => { |
54 | it('registers a shared store patch listener', () => { | 54 | it('registers a shared store patch listener', () => { |
55 | createSophieRenderer(); | 55 | const service = new IpcRendererService(); |
56 | expect(RendererToMainIpcService).toHaveBeenCalledTimes(1); | 56 | createSophieRenderer(service); |
57 | const service = mocked(RendererToMainIpcService).mock.instances[0]; | ||
58 | expect(service.onSharedStorePatch).toHaveBeenCalledTimes(1); | 57 | expect(service.onSharedStorePatch).toHaveBeenCalledTimes(1); |
59 | }); | 58 | }); |
60 | }); | 59 | }); |
61 | 60 | ||
62 | describe('instance', () => { | 61 | describe('instance', () => { |
63 | let sut: SophieRenderer; | 62 | let sut: SophieRenderer; |
64 | let service: RendererToMainIpcService; | 63 | let service: IpcRendererService; |
65 | let onSharedStorePatch: (patch: unknown) => void; | 64 | let onSharedStorePatch: (patch: unknown) => void; |
66 | let listener = { | 65 | let listener = { |
67 | onSnapshot: jest.fn((_snapshot: SharedStoreSnapshotIn) => {}), | 66 | onSnapshot: jest.fn((_snapshot: SharedStoreSnapshotIn) => {}), |
@@ -69,8 +68,8 @@ describe('instance', () => { | |||
69 | }; | 68 | }; |
70 | 69 | ||
71 | beforeEach(() => { | 70 | beforeEach(() => { |
72 | sut = createSophieRenderer(); | 71 | service = new IpcRendererService(); |
73 | service = mocked(RendererToMainIpcService).mock.instances[0]; | 72 | sut = createSophieRenderer(service); |
74 | onSharedStorePatch = mocked(service.onSharedStorePatch).mock.calls[0][0]; | 73 | onSharedStorePatch = mocked(service.onSharedStorePatch).mock.calls[0][0]; |
75 | }); | 74 | }); |
76 | 75 | ||