diff options
author | Kristóf Marussy <kristof@marussy.com> | 2022-01-23 17:12:47 +0100 |
---|---|---|
committer | Kristóf Marussy <kristof@marussy.com> | 2022-02-08 21:43:17 +0100 |
commit | 044b2de8c7861504704468ba441d4a6a37eed8f7 (patch) | |
tree | 940d2b7946d07a0c69b5e3ad46c25c599cf2aca7 /packages/renderer/src/stores | |
parent | feat: Add selected service field to SharedStore (diff) | |
download | sophie-044b2de8c7861504704468ba441d4a6a37eed8f7.tar.gz sophie-044b2de8c7861504704468ba441d4a6a37eed8f7.tar.zst sophie-044b2de8c7861504704468ba441d4a6a37eed8f7.zip |
refactor: Move runtime state into shared models
Now the runtime state lives inside the model (instead of being
associated to the static settings via a map), which simplifies state
management. Static settings are now located inside the runtime models,
so we must create tests to make sure that the settings are being
persisted correctly. The contents of the config file are now generated
as a view of store (instead of a snapshot), which adds flexibility.
Signed-off-by: Kristóf Marussy <kristof@marussy.com>
Diffstat (limited to 'packages/renderer/src/stores')
-rw-r--r-- | packages/renderer/src/stores/Config.ts | 32 | ||||
-rw-r--r-- | packages/renderer/src/stores/RendererStore.ts | 24 | ||||
-rw-r--r-- | packages/renderer/src/stores/Service.ts | 38 | ||||
-rw-r--r-- | packages/renderer/src/stores/SharedStore.ts | 37 | ||||
-rw-r--r-- | packages/renderer/src/stores/__tests__/Service.spec.ts | 63 |
5 files changed, 12 insertions, 182 deletions
diff --git a/packages/renderer/src/stores/Config.ts b/packages/renderer/src/stores/Config.ts deleted file mode 100644 index 070c4ec..0000000 --- a/packages/renderer/src/stores/Config.ts +++ /dev/null | |||
@@ -1,32 +0,0 @@ | |||
1 | /* | ||
2 | * Copyright (C) 2022 Kristóf Marussy <kristof@marussy.com> | ||
3 | * | ||
4 | * This file is part of Sophie. | ||
5 | * | ||
6 | * Sophie is free software: you can redistribute it and/or modify | ||
7 | * it under the terms of the GNU Affero General Public License as | ||
8 | * published by the Free Software Foundation, version 3. | ||
9 | * | ||
10 | * This program is distributed in the hope that it will be useful, | ||
11 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
12 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||
13 | * GNU Affero General Public License for more details. | ||
14 | * | ||
15 | * You should have received a copy of the GNU Affero General Public License | ||
16 | * along with this program. If not, see <https://www.gnu.org/licenses/>. | ||
17 | * | ||
18 | * SPDX-License-Identifier: AGPL-3.0-only | ||
19 | */ | ||
20 | |||
21 | import { config as originalConfig } from '@sophie/shared'; | ||
22 | import { Instance, types } from 'mobx-state-tree'; | ||
23 | |||
24 | import { service } from './Service'; | ||
25 | |||
26 | export const config = originalConfig.props({ | ||
27 | services: types.array(service), | ||
28 | }); | ||
29 | |||
30 | export interface Config extends Instance<typeof config> {} | ||
31 | |||
32 | export type { ConfigSnapshotIn, ConfigSnapshotOut } from '@sophie/shared'; | ||
diff --git a/packages/renderer/src/stores/RendererStore.ts b/packages/renderer/src/stores/RendererStore.ts index 731ca28..a4e6197 100644 --- a/packages/renderer/src/stores/RendererStore.ts +++ b/packages/renderer/src/stores/RendererStore.ts | |||
@@ -18,17 +18,19 @@ | |||
18 | * SPDX-License-Identifier: AGPL-3.0-only | 18 | * SPDX-License-Identifier: AGPL-3.0-only |
19 | */ | 19 | */ |
20 | 20 | ||
21 | import { BrowserViewBounds, SophieRenderer, ThemeSource } from '@sophie/shared'; | 21 | import { |
22 | BrowserViewBounds, | ||
23 | sharedStore, | ||
24 | Service, | ||
25 | SophieRenderer, | ||
26 | ThemeSource, | ||
27 | } from '@sophie/shared'; | ||
22 | import { applySnapshot, applyPatch, Instance, types } from 'mobx-state-tree'; | 28 | import { applySnapshot, applyPatch, Instance, types } from 'mobx-state-tree'; |
23 | 29 | ||
30 | import RendererEnv from '../env/RendererEnv'; | ||
24 | import getEnv from '../env/getEnv'; | 31 | import getEnv from '../env/getEnv'; |
25 | import RendererEnvImpl from '../env/impl/RendererEnvImpl'; | ||
26 | import { getLogger } from '../utils/log'; | 32 | import { getLogger } from '../utils/log'; |
27 | 33 | ||
28 | import type { Config } from './Config'; | ||
29 | import type { Service } from './Service'; | ||
30 | import { sharedStore } from './SharedStore'; | ||
31 | |||
32 | const log = getLogger('RendererStore'); | 34 | const log = getLogger('RendererStore'); |
33 | 35 | ||
34 | export const rendererStore = types | 36 | export const rendererStore = types |
@@ -36,11 +38,8 @@ export const rendererStore = types | |||
36 | shared: types.optional(sharedStore, {}), | 38 | shared: types.optional(sharedStore, {}), |
37 | }) | 39 | }) |
38 | .views((self) => ({ | 40 | .views((self) => ({ |
39 | get config(): Config { | ||
40 | return self.shared.config; | ||
41 | }, | ||
42 | get services(): Service[] { | 41 | get services(): Service[] { |
43 | return this.config.services; | 42 | return self.shared.services; |
44 | }, | 43 | }, |
45 | get selectedService(): Service | undefined { | 44 | get selectedService(): Service | undefined { |
46 | return self.shared.selectedService; | 45 | return self.shared.selectedService; |
@@ -87,9 +86,10 @@ export interface RendererStore extends Instance<typeof rendererStore> {} | |||
87 | export function createAndConnectRendererStore( | 86 | export function createAndConnectRendererStore( |
88 | ipc: SophieRenderer, | 87 | ipc: SophieRenderer, |
89 | ): RendererStore { | 88 | ): RendererStore { |
90 | const env = new RendererEnvImpl(ipc); | 89 | const env: RendererEnv = { |
90 | dispatchMainAction: ipc.dispatchAction, | ||
91 | }; | ||
91 | const store = rendererStore.create({}, env); | 92 | const store = rendererStore.create({}, env); |
92 | env.setStore(store); | ||
93 | 93 | ||
94 | ipc | 94 | ipc |
95 | .onSharedStoreChange({ | 95 | .onSharedStoreChange({ |
diff --git a/packages/renderer/src/stores/Service.ts b/packages/renderer/src/stores/Service.ts deleted file mode 100644 index 2f45106..0000000 --- a/packages/renderer/src/stores/Service.ts +++ /dev/null | |||
@@ -1,38 +0,0 @@ | |||
1 | /* | ||
2 | * Copyright (C) 2022 Kristóf Marussy <kristof@marussy.com> | ||
3 | * | ||
4 | * This file is part of Sophie. | ||
5 | * | ||
6 | * Sophie is free software: you can redistribute it and/or modify | ||
7 | * it under the terms of the GNU Affero General Public License as | ||
8 | * published by the Free Software Foundation, version 3. | ||
9 | * | ||
10 | * This program is distributed in the hope that it will be useful, | ||
11 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
12 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||
13 | * GNU Affero General Public License for more details. | ||
14 | * | ||
15 | * You should have received a copy of the GNU Affero General Public License | ||
16 | * along with this program. If not, see <https://www.gnu.org/licenses/>. | ||
17 | * | ||
18 | * SPDX-License-Identifier: AGPL-3.0-only | ||
19 | */ | ||
20 | |||
21 | import { | ||
22 | runtimeService, | ||
23 | RuntimeService, | ||
24 | service as originalService, | ||
25 | } from '@sophie/shared'; | ||
26 | import { Instance } from 'mobx-state-tree'; | ||
27 | |||
28 | import getEnv from '../env/getEnv'; | ||
29 | |||
30 | export const service = originalService.views((self) => ({ | ||
31 | get runtime(): RuntimeService { | ||
32 | return getEnv(self).getRuntimeService(self.id) ?? runtimeService.create(); | ||
33 | }, | ||
34 | })); | ||
35 | |||
36 | export interface Service extends Instance<typeof service> {} | ||
37 | |||
38 | export type { ServiceSnapshotIn, ServiceSnapshotOut } from '@sophie/shared'; | ||
diff --git a/packages/renderer/src/stores/SharedStore.ts b/packages/renderer/src/stores/SharedStore.ts deleted file mode 100644 index 962f7e2..0000000 --- a/packages/renderer/src/stores/SharedStore.ts +++ /dev/null | |||
@@ -1,37 +0,0 @@ | |||
1 | /* | ||
2 | * Copyright (C) 2022 Kristóf Marussy <kristof@marussy.com> | ||
3 | * | ||
4 | * This file is part of Sophie. | ||
5 | * | ||
6 | * Sophie is free software: you can redistribute it and/or modify | ||
7 | * it under the terms of the GNU Affero General Public License as | ||
8 | * published by the Free Software Foundation, version 3. | ||
9 | * | ||
10 | * This program is distributed in the hope that it will be useful, | ||
11 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
12 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||
13 | * GNU Affero General Public License for more details. | ||
14 | * | ||
15 | * You should have received a copy of the GNU Affero General Public License | ||
16 | * along with this program. If not, see <https://www.gnu.org/licenses/>. | ||
17 | * | ||
18 | * SPDX-License-Identifier: AGPL-3.0-only | ||
19 | */ | ||
20 | |||
21 | import { sharedStore as originalSharedStore } from '@sophie/shared'; | ||
22 | import { Instance, types } from 'mobx-state-tree'; | ||
23 | |||
24 | import { config } from './Config'; | ||
25 | import { service } from './Service'; | ||
26 | |||
27 | export const sharedStore = originalSharedStore.props({ | ||
28 | config: types.optional(config, {}), | ||
29 | selectedService: types.safeReference(service), | ||
30 | }); | ||
31 | |||
32 | export interface SharedStore extends Instance<typeof sharedStore> {} | ||
33 | |||
34 | export type { | ||
35 | SharedStoreSnapshotIn, | ||
36 | SharedStoreSnapshotOut, | ||
37 | } from '@sophie/shared'; | ||
diff --git a/packages/renderer/src/stores/__tests__/Service.spec.ts b/packages/renderer/src/stores/__tests__/Service.spec.ts deleted file mode 100644 index f835d41..0000000 --- a/packages/renderer/src/stores/__tests__/Service.spec.ts +++ /dev/null | |||
@@ -1,63 +0,0 @@ | |||
1 | /* | ||
2 | * Copyright (C) 2022 Kristóf Marussy <kristof@marussy.com> | ||
3 | * | ||
4 | * This file is part of Sophie. | ||
5 | * | ||
6 | * Sophie is free software: you can redistribute it and/or modify | ||
7 | * it under the terms of the GNU Affero General Public License as | ||
8 | * published by the Free Software Foundation, version 3. | ||
9 | * | ||
10 | * This program is distributed in the hope that it will be useful, | ||
11 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
12 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||
13 | * GNU Affero General Public License for more details. | ||
14 | * | ||
15 | * You should have received a copy of the GNU Affero General Public License | ||
16 | * along with this program. If not, see <https://www.gnu.org/licenses/>. | ||
17 | * | ||
18 | * SPDX-License-Identifier: AGPL-3.0-only | ||
19 | */ | ||
20 | |||
21 | import { jest } from '@jest/globals'; | ||
22 | import { runtimeService } from '@sophie/shared'; | ||
23 | import { mocked } from 'jest-mock'; | ||
24 | |||
25 | import type RendererEnv from '../../env/RendererEnv'; | ||
26 | import { service, Service } from '../Service'; | ||
27 | |||
28 | const env: RendererEnv = { | ||
29 | dispatchMainAction: jest.fn(), | ||
30 | getRuntimeService: jest.fn(), | ||
31 | }; | ||
32 | let sut: Service; | ||
33 | |||
34 | beforeEach(() => { | ||
35 | sut = service.create( | ||
36 | { | ||
37 | id: 'serviceId', | ||
38 | name: 'Foo', | ||
39 | url: 'https://example.com', | ||
40 | profile: 'profileId', | ||
41 | }, | ||
42 | env, | ||
43 | ); | ||
44 | }); | ||
45 | |||
46 | describe('runtime', () => { | ||
47 | it('should return the runtime service with for the service ID', () => { | ||
48 | const runtimeServiceStore = runtimeService.create({}, env); | ||
49 | mocked(env.getRuntimeService).mockReturnValueOnce(runtimeServiceStore); | ||
50 | const returnedStore = sut.runtime; | ||
51 | expect(env.getRuntimeService).toHaveBeenCalledWith('serviceId'); | ||
52 | expect(returnedStore).toBe(runtimeServiceStore); | ||
53 | }); | ||
54 | |||
55 | it('should return a valid runtime service even if none exists in the environment', () => { | ||
56 | /* | ||
57 | eslint-disable-next-line unicorn/no-useless-undefined -- | ||
58 | `mockReturnValueOnce` expects 1 parameter. | ||
59 | */ | ||
60 | mocked(env.getRuntimeService).mockReturnValueOnce(undefined); | ||
61 | expect(sut.runtime).toHaveProperty('state', 'hibernated'); | ||
62 | }); | ||
63 | }); | ||