fixes

Signed-off-by: Jari Kolehmainen <jari.kolehmainen@gmail.com>

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

@@ -17,4 +17,11 @@ describe("HotbarStore", () => {
       expect(hotbarStore.hotbars.length).toEqual(1);
     });
   });
+
+  describe("add", () => {
+    it("adds a hotbar", () => {
+      hotbarStore.add({ name: "hottest" });
+      expect(hotbarStore.hotbars.length).toEqual(2);
+    });
+  });
 });

src/common/hotbar-store.ts

@@ -1,7 +1,7 @@
 import { action, comparer, observable, toJS } from "mobx";
 import { BaseStore } from "./base-store";
 import migrations from "../migrations/hotbar-store";
-import uuid from "uuid";
+import * as uuid from "uuid";
 
 export interface HotbarItem {
   entity: {
@@ -18,6 +18,12 @@ export interface Hotbar {
   items: HotbarItem[];
 }
 
+export interface HotbarCreateOptions {
+  id?: string;
+  name: string;
+  items?: HotbarItem[];
+}
+
 export interface HotbarStoreModel {
   hotbars: Hotbar[];
   activeHotbarId: string;
@@ -86,18 +92,21 @@ export class HotbarStore extends BaseStore<HotbarStoreModel> {
     return this.hotbars.find((hotbar) => hotbar.id === id);
   }
 
-  add(hotbar: Partial<Hotbar>) {
-    hotbar.id = uuid.v4();
+  add(data: HotbarCreateOptions) {
+    const {
+      id = uuid.v4(),
+      items = [],
+      name,
+    } = data;
 
-    if (!hotbar.items) {
-      hotbar.items = [];
-    }
+    const hotbar = { id, name, items };
 
     this.hotbars.push(hotbar as Hotbar);
 
     return hotbar as Hotbar;
   }
 
+  @action
   remove(hotbar: Hotbar) {
     this.hotbars = this.hotbars.filter((h) => h !== hotbar);
 

src/migrations/hotbar-store/5.0.0-alpha.2.ts

@@ -1,7 +1,7 @@
 // Cleans up a store that had the state related data stored
 import { Hotbar } from "../../common/hotbar-store";
 import { migration } from "../migration-wrapper";
-import uuid from "uuid";
+import * as uuid from "uuid";
 
 export default migration({
   version: "5.0.0-alpha.2",

src/renderer/components/hotbar/hotbar-add-command.tsx

@@ -6,7 +6,7 @@ import { Input, InputValidator } from "../input";
 
 const uniqueHotbarName: InputValidator = {
   condition: ({ required }) => required,
-  message: () => `Hotbar with this name already exists`,
+  message: () => "Hotbar with this name already exists",
   validate: value => !hotbarStore.getByName(value),
 };