diff options
author | Vijay A <vraravam@users.noreply.github.com> | 2022-01-04 22:16:09 +0530 |
---|---|---|
committer | Kristóf Marussy <kristof@marussy.com> | 2022-01-09 19:51:48 +0100 |
commit | 649c1ed37ac74765fb7572fa3f703c94827042f4 (patch) | |
tree | c00d13e58cbe1df0a4645b7905bccbab3eb9ff92 /packages/preload | |
parent | Incorporate 'eslint' (fixes #3) (diff) | |
download | sophie-649c1ed37ac74765fb7572fa3f703c94827042f4.tar.gz sophie-649c1ed37ac74765fb7572fa3f703c94827042f4.tar.zst sophie-649c1ed37ac74765fb7572fa3f703c94827042f4.zip |
New configurations based on review comments (WIP)
Signed-off-by: Vijay A <vraravam@users.noreply.github.com>
Diffstat (limited to 'packages/preload')
-rw-r--r-- | packages/preload/package.json | 2 | ||||
-rw-r--r-- | packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts | 8 | ||||
-rw-r--r-- | packages/preload/src/index.ts | 2 |
3 files changed, 9 insertions, 3 deletions
diff --git a/packages/preload/package.json b/packages/preload/package.json index 253b858..0957aaf 100644 --- a/packages/preload/package.json +++ b/packages/preload/package.json | |||
@@ -11,10 +11,12 @@ | |||
11 | "dependencies": { | 11 | "dependencies": { |
12 | "@sophie/shared": "workspace:*", | 12 | "@sophie/shared": "workspace:*", |
13 | "electron": "16.0.6", | 13 | "electron": "16.0.6", |
14 | "loglevel": "^1.8.0", | ||
14 | "mobx": "^6.3.12", | 15 | "mobx": "^6.3.12", |
15 | "mobx-state-tree": "^5.1.0" | 16 | "mobx-state-tree": "^5.1.0" |
16 | }, | 17 | }, |
17 | "devDependencies": { | 18 | "devDependencies": { |
19 | "@jest/globals": "^27.4.4", | ||
18 | "@types/jest": "^27.4.0", | 20 | "@types/jest": "^27.4.0", |
19 | "jest": "^27.4.7", | 21 | "jest": "^27.4.7", |
20 | "jest-mock": "^27.4.6", | 22 | "jest-mock": "^27.4.6", |
diff --git a/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts b/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts index f8b0024..ff77a63 100644 --- a/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts +++ b/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts | |||
@@ -40,7 +40,7 @@ jest.unstable_mockModule('electron', () => ({ | |||
40 | 40 | ||
41 | const { ipcRenderer } = await import('electron'); | 41 | const { ipcRenderer } = await import('electron'); |
42 | 42 | ||
43 | const { createSophieRenderer } = await import('../SophieRendererImpl.js'); | 43 | const { createSophieRenderer } = await import('../SophieRendererImpl'); |
44 | 44 | ||
45 | const event: Electron.IpcRendererEvent = null as unknown as Electron.IpcRendererEvent; | 45 | const event: Electron.IpcRendererEvent = null as unknown as Electron.IpcRendererEvent; |
46 | 46 | ||
@@ -83,9 +83,11 @@ describe('createSophieRenderer', () => { | |||
83 | 83 | ||
84 | describe('SophieRendererImpl', () => { | 84 | describe('SophieRendererImpl', () => { |
85 | let sut: SophieRenderer; | 85 | let sut: SophieRenderer; |
86 | let onSharedStorePatch: (event: Electron.IpcRendererEvent, patch: unknown) => void; | 86 | let onSharedStorePatch: (event1: Electron.IpcRendererEvent, patch1: unknown) => void; |
87 | let listener = { | 87 | let listener = { |
88 | // eslint-disable-next-line @typescript-eslint/no-unused-vars, no-unused-vars | ||
88 | onSnapshot: jest.fn((_snapshot: SharedStoreSnapshotIn) => {}), | 89 | onSnapshot: jest.fn((_snapshot: SharedStoreSnapshotIn) => {}), |
90 | // eslint-disable-next-line @typescript-eslint/no-unused-vars, no-unused-vars | ||
89 | onPatch: jest.fn((_patch: IJsonPatch) => {}), | 91 | onPatch: jest.fn((_patch: IJsonPatch) => {}), |
90 | }; | 92 | }; |
91 | 93 | ||
@@ -233,7 +235,9 @@ describe('SophieRendererImpl', () => { | |||
233 | shouldUseDarkColors: false, | 235 | shouldUseDarkColors: false, |
234 | }; | 236 | }; |
235 | const listener2 = { | 237 | const listener2 = { |
238 | // eslint-disable-next-line @typescript-eslint/no-unused-vars, no-unused-vars | ||
236 | onSnapshot: jest.fn((_snapshot: SharedStoreSnapshotIn) => { }), | 239 | onSnapshot: jest.fn((_snapshot: SharedStoreSnapshotIn) => { }), |
240 | // eslint-disable-next-line @typescript-eslint/no-unused-vars, no-unused-vars | ||
237 | onPatch: jest.fn((_patch: IJsonPatch) => { }), | 241 | onPatch: jest.fn((_patch: IJsonPatch) => { }), |
238 | }; | 242 | }; |
239 | 243 | ||
diff --git a/packages/preload/src/index.ts b/packages/preload/src/index.ts index f6c720a..de91742 100644 --- a/packages/preload/src/index.ts +++ b/packages/preload/src/index.ts | |||
@@ -20,7 +20,7 @@ | |||
20 | 20 | ||
21 | import { contextBridge } from 'electron'; | 21 | import { contextBridge } from 'electron'; |
22 | 22 | ||
23 | import { createSophieRenderer } from './contextBridge/SophieRendererImpl.js'; | 23 | import { createSophieRenderer } from './contextBridge/SophieRendererImpl'; |
24 | 24 | ||
25 | const isDevelopment = import.meta.env.MODE === 'development'; | 25 | const isDevelopment = import.meta.env.MODE === 'development'; |
26 | 26 | ||