add some basic workspace store tests (#680)

* add some basic workspace store tests

Signed-off-by: Sebastian Malton <smalton@mirantis.com>

Co-authored-by: Sebastian Malton <smalton@mirantis.com>

src/common/workspace-store.ts

@@ -51,6 +51,10 @@ export class WorkspaceStore extends BaseStore<WorkspaceStoreModel> {
 
   @action
   setActive(id = WorkspaceStore.defaultId) {
+    if (!this.getById(id)) {
+      throw new Error(`workspace ${id} doesn't exist`);
+    }
+
     this.currentWorkspaceId = id;
   }
 

src/common/workspace-store_test.ts

@@ -0,0 +1,128 @@
+import mockFs from "mock-fs"
+
+jest.mock("electron", () => {
+  return {
+    app: {
+      getVersion: () => '99.99.99',
+      getPath: () => 'tmp',
+      getLocale: () => 'en'
+    }
+  }
+})
+
+import { WorkspaceStore } from "./workspace-store"
+
+describe("workspace store tests", () => {
+  describe("for an empty config", () => {
+    beforeEach(async () => {
+      WorkspaceStore.resetInstance()
+      mockFs({ tmp: { 'lens-workspace-store.json': "{}" } })
+
+      await WorkspaceStore.getInstance<WorkspaceStore>().load();
+    })
+
+    afterEach(() => {
+      mockFs.restore()
+    })
+
+    it("default workspace should always exist", () => {
+      const ws = WorkspaceStore.getInstance<WorkspaceStore>();
+
+      expect(ws.workspaces.size).toBe(1);
+      expect(ws.getById(WorkspaceStore.defaultId)).not.toBe(null);
+    })
+
+    it("cannot remove the default workspace", () => {
+      const ws = WorkspaceStore.getInstance<WorkspaceStore>();
+
+      expect(() => ws.removeWorkspace(WorkspaceStore.defaultId)).toThrowError("Cannot remove");
+    })
+
+    it("can update default workspace name", () => {
+      const ws = WorkspaceStore.getInstance<WorkspaceStore>();
+
+      ws.saveWorkspace({
+        id: WorkspaceStore.defaultId,
+        name: "foobar",
+      });
+
+      expect(ws.currentWorkspace.name).toBe("foobar");
+    })
+
+    it("can add workspaces", () => {
+      const ws = WorkspaceStore.getInstance<WorkspaceStore>();
+
+      ws.saveWorkspace({
+        id: "123",
+        name: "foobar",
+      });
+
+      expect(ws.getById("123").name).toBe("foobar");
+    })
+
+    it("cannot set a non-existent workspace to be active", () => {
+      const ws = WorkspaceStore.getInstance<WorkspaceStore>();
+
+      expect(() => ws.setActive("abc")).toThrow("doesn't exist");
+    })
+
+    it("can set a existent workspace to be active", () => {
+      const ws = WorkspaceStore.getInstance<WorkspaceStore>();
+
+      ws.saveWorkspace({
+        id: "abc",
+        name: "foobar",
+      });
+
+      expect(() => ws.setActive("abc")).not.toThrowError();
+    })
+
+    it("can remove a workspace", () => {
+      const ws = WorkspaceStore.getInstance<WorkspaceStore>();
+
+      ws.saveWorkspace({
+        id: "123",
+        name: "foobar",
+      });
+      ws.saveWorkspace({
+        id: "1234",
+        name: "foobar 1",
+      });
+      ws.removeWorkspace("123");
+
+      expect(ws.workspaces.size).toBe(2);
+    })
+  })
+
+  describe("for a non-empty config", () => {
+    beforeEach(async () => {
+      WorkspaceStore.resetInstance()
+      mockFs({
+        tmp: {
+          'lens-workspace-store.json': JSON.stringify({
+            currentWorkspace: "abc",
+            workspaces: [{
+              id: "abc",
+              name: "test"
+            }, {
+              id: "default",
+              name: "default"
+            }]
+          })
+        }
+      })
+
+      await WorkspaceStore.getInstance<WorkspaceStore>().load();
+    })
+
+    afterEach(() => {
+      mockFs.restore()
+    })
+
+    it("doesn't revert to default workspace", async () => {
+      const ws = WorkspaceStore.getInstance<WorkspaceStore>();
+
+      expect(ws.currentWorkspaceId).toBe("abc");
+    })
+  })
+})