hendrik/lens
1
0
Fork 0
mirror of https://github.com/lensapp/lens.git synced 2025-05-20 05:10:56 +00:00
Code Issues Projects Releases Wiki Activity

Remove flag for causing side effects from too broad scope

Browse Source 
Co-authored-by: Mikko Aspiala <mikko.aspiala@gmail.com>

Signed-off-by: Janne Savolainen <janne.savolainen@live.fi>
This commit is contained in:
Janne Savolainen 2022-03-29 09:34:14 +03:00
parent 5113729e87
commit 504557fc62
No known key found for this signature in database
GPG Key ID: 8C6CFB2FFFE8F68A
2 changed files with 1 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/common/__tests__/hotbar-store.test.ts
View File

@ -9,7 +9,6 @@ import logger from "../../main/logger";
import type { CatalogEntity, CatalogEntityData, CatalogEntityKindData } from "../catalog";
import { HotbarStore } from "../hotbar-store";
import { getDiForUnitTesting } from "../../main/getDiForUnitTesting";
import hotbarStoreInjectable from "../hotbar-store.injectable";
import getConfigurationFileModelInjectable from "../get-configuration-file-model/get-configuration-file-model.injectable";
import appVersionInjectable from "../get-configuration-file-model/app-version/app-version.injectable";
import type { DiContainer } from "@ogre-tools/injectable";
@ -128,7 +127,6 @@ describe("HotbarStore", () => {
beforeEach(async () => {
di = getDiForUnitTesting({ doGeneralOverrides: true });
di.permitSideEffects(hotbarStoreInjectable);
di.permitSideEffects(getConfigurationFileModelInjectable);
di.permitSideEffects(appVersionInjectable);
});

6
src/common/hotbar-store.injectable.ts
View File

@ -369,14 +369,10 @@ const hotbarStoreInjectable = getInjectable({
instantiate: (di) => {
HotbarStore.resetInstance();
const instance = HotbarStore.createInstance({
return HotbarStore.createInstance({
catalogCatalogEntity: di.inject(catalogCatalogEntityInjectable),
});
return instance;
},
causesSideEffects: true,
});
export default hotbarStoreInjectable;