Remove use of mockFs in hotbar store tests

Signed-off-by: Sebastian Malton <sebastian@malton.name>

src/common/__tests__/hotbar-store.test.ts

@@ -4,10 +4,8 @@
  */
 
 import { anyObject } from "jest-mock-extended";
-import mockFs from "mock-fs";
 import type { CatalogEntity, CatalogEntityData, CatalogEntityKindData } from "../catalog";
 import { getDiForUnitTesting } from "../../main/getDiForUnitTesting";
-import getConfigurationFileModelInjectable from "../get-configuration-file-model/get-configuration-file-model.injectable";
 import type { DiContainer } from "@ogre-tools/injectable";
 import hotbarStoreInjectable from "../hotbars/store.injectable";
 import type { HotbarStore } from "../hotbars/store";
@@ -19,7 +17,7 @@ import loggerInjectable from "../logger.injectable";
 import type { Logger } from "../logger";
 import directoryForUserDataInjectable from "../app-paths/directory-for-user-data/directory-for-user-data.injectable";
 import storeMigrationVersionInjectable from "../vars/store-migration-version.injectable";
-import fsInjectable from "../fs/fs.injectable";
+import writeJsonSyncInjectable from "../fs/write-json-sync.injectable";
 
 function getMockCatalogEntity(data: Partial<CatalogEntityData> & CatalogEntityKindData): CatalogEntity {
   return {
@@ -47,8 +45,6 @@ describe("HotbarStore", () => {
   beforeEach(async () => {
     di = getDiForUnitTesting({ doGeneralOverrides: true });
 
-    di.unoverride(hotbarStoreInjectable);
-
     testCluster = getMockCatalogEntity({
       apiVersion: "v1",
       kind: "Cluster",
@@ -101,7 +97,7 @@ describe("HotbarStore", () => {
 
     di.override(loggerInjectable, () => loggerMock);
 
-    di.override(directoryForUserDataInjectable, () => "some-directory-for-user-data");
+    di.override(directoryForUserDataInjectable, () => "/some-directory-for-user-data");
 
     const catalogEntityRegistry = di.inject(catalogEntityRegistryInjectable);
     const catalogCatalogEntity = di.inject(catalogCatalogEntityInjectable);
@@ -112,20 +108,10 @@ describe("HotbarStore", () => {
       awsCluster,
       catalogCatalogEntity,
     ]));
-
-    di.permitSideEffects(fsInjectable);
-    di.permitSideEffects(getConfigurationFileModelInjectable);
-    di.unoverride(getConfigurationFileModelInjectable);
-  });
-
-  afterEach(() => {
-    mockFs.restore();
   });
 
   describe("given no previous data in store, running all migrations", () => {
     beforeEach(() => {
-      mockFs();
-
       hotbarStore = di.inject(hotbarStoreInjectable);
 
       hotbarStore.load();
@@ -276,9 +262,9 @@ describe("HotbarStore", () => {
 
   describe("given data from 5.0.0-beta.3 and version being 5.0.0-beta.10", () => {
     beforeEach(() => {
-      mockFs({
-        "some-directory-for-user-data": {
-          "lens-hotbar-store.json": JSON.stringify({
+      const writeJsonSync = di.inject(writeJsonSyncInjectable);
+
+      writeJsonSync("/some-directory-for-user-data/lens-hotbar-store.json", {
         __internal__: {
           migrations: {
             version: "5.0.0-beta.3",
@@ -334,8 +320,6 @@ describe("HotbarStore", () => {
             ],
           },
         ],
-          }),
-        },
       });
 
       di.override(storeMigrationVersionInjectable, () => "5.0.0-beta.10");