fix: validate keymap props

ueokande · Feb 20, 2024 · 41ef719 · 41ef719
1 parent a4ea0a2
commit 41ef719
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 21 deletions.
diff --git a/src/background/settings/Validator.ts b/src/background/settings/Validator.ts
@@ -27,7 +27,7 @@ export default class Validator {
     });
 
     const keymapEntries = settings.keymaps?.entries() || [];
-    keymapEntries.forEach(([key, { type, ...props }]) => {
+    keymapEntries.forEach(([key, { type, props }]) => {
       const op = this.operatorRegistory.getOperator(type);
       if (typeof op === "undefined") {
         throw new Error("Unknown keymap: " + type);

diff --git a/test/background/settings/Validator.test.ts b/test/background/settings/Validator.test.ts
@@ -20,9 +20,9 @@ describe("Validator", () => {
     sut.validate({});
     sut.validate({
       keymaps: new Keymaps({
-        d: { type: "tabs.close" },
-        D: { type: "tabs.close", select: "left" },
-        zd: { type: "tabs.duplicate" },
+        d: { type: "tabs.close", props: {} },
+        D: { type: "tabs.close", props: { select: "left" } },
+        zd: { type: "tabs.duplicate", props: {} },
       }),
       search: new Search("google", {
         google: "https://google.com/search?q={}",
@@ -65,15 +65,15 @@ describe("Validator", () => {
     expect(() => {
       sut.validate({
         keymaps: new Keymaps({
-          d: { type: "harakiri" },
+          d: { type: "harakiri", props: {} },
         }),
       });
     }).toThrowError("Unknown keymap: harakiri");
 
     expect(() => {
       sut.validate({
         keymaps: new Keymaps({
-          D: { type: "tabs.close", force: "1" },
+          D: { type: "tabs.close", props: { force: "1" }} ,
         }),
       });
     }).toThrowError(

diff --git a/test/shared/Keymaps.test.ts b/test/shared/Keymaps.test.ts
@@ -4,44 +4,44 @@ describe("Keymaps", () => {
   describe("#combine", () => {
     it("returns combined keymaps", () => {
       const keymaps1 = new Keymaps({
-        k: { type: "scroll.vertically", count: -1 },
-        j: { type: "scroll.vertically", count: 1 },
+        k: { type: "scroll.vertically", props: { count: -1 } },
+        j: { type: "scroll.vertically", props: { count: 1 } },
       });
       const keymap2 = new Keymaps({
-        n: { type: "find.next" },
-        N: { type: "find.prev" },
+        n: { type: "find.next", props: {} },
+        N: { type: "find.prev", props: {} },
       });
       const combined = keymaps1.combine(keymap2);
 
       const entries = combined
         .entries()
         .sort(([name1], [name2]) => name1.localeCompare(name2));
       expect(entries).toEqual([
-        ["j", { type: "scroll.vertically", count: 1 }],
-        ["k", { type: "scroll.vertically", count: -1 }],
-        ["n", { type: "find.next" }],
-        ["N", { type: "find.prev" }],
+        ["j", { type: "scroll.vertically", props: { count: 1 } }],
+        ["k", { type: "scroll.vertically", props: { count: -1 } }],
+        ["n", { type: "find.next", props: {} }],
+        ["N", { type: "find.prev", props: {} }],
       ]);
     });
 
     it("overrides current keymaps", () => {
       const keymaps1 = new Keymaps({
-        k: { type: "scroll.vertically", count: -1 },
-        j: { type: "scroll.vertically", count: 1 },
+        k: { type: "scroll.vertically", props: { count: -1 } },
+        j: { type: "scroll.vertically", props: { count: 1 } },
       });
       const keymap2 = new Keymaps({
-        n: { type: "find.next" },
-        j: { type: "find.prev" },
+        n: { type: "find.next", props: {} },
+        j: { type: "find.prev", props: {} },
       });
       const combined = keymaps1.combine(keymap2);
 
       const entries = combined
         .entries()
         .sort(([name1], [name2]) => name1.localeCompare(name2));
       expect(entries).toEqual([
-        ["j", { type: "find.prev" }],
-        ["k", { type: "scroll.vertically", count: -1 }],
-        ["n", { type: "find.next" }],
+        ["j", { type: "find.prev", props: {} }],
+        ["k", { type: "scroll.vertically", props: { count: -1 } }],
+        ["n", { type: "find.next", props: {} }],
       ]);
     });
   });