Merge pull request #917 from ueokande/operation-as-a-operator

refactor: Make each operation as an operator

1 files changed, 8 insertions, 108 deletions

diff --git a/src/background/controllers/OperationController.ts b/src/background/controllers/OperationController.ts
index 5a7047d..d606ac0 100644
--- a/src/background/controllers/OperationController.ts
+++ b/src/background/controllers/OperationController.ts
@@ -1,23 +1,14 @@
-import { injectable } from "tsyringe";
+import { inject, injectable } from "tsyringe";
 import * as operations from "../../shared/operations";
-import FindUseCase from "../usecases/FindUseCase";
-import ConsoleUseCase from "../usecases/ConsoleUseCase";
-import TabUseCase from "../usecases/TabUseCase";
-import TabSelectUseCase from "../usecases/TabSelectUseCase";
-import ZoomUseCase from "../usecases/ZoomUseCase";
-import NavigateUseCase from "../usecases/NavigateUseCase";
+import OperatorFactory from "../operators/OperatorFactory";
 import RepeatUseCase from "../usecases/RepeatUseCase";
 
 @injectable()
 export default class OperationController {
   constructor(
-    private findUseCase: FindUseCase,
-    private consoleUseCase: ConsoleUseCase,
-    private tabUseCase: TabUseCase,
-    private tabSelectUseCase: TabSelectUseCase,
-    private zoomUseCase: ZoomUseCase,
-    private navigateUseCase: NavigateUseCase,
-    private repeatUseCase: RepeatUseCase
+    private readonly repeatUseCase: RepeatUseCase,
+    @inject("OperatorFactory")
+    private readonly operatorFactory: OperatorFactory
   ) {}
 
   async exec(repeat: number, op: operations.Operation): Promise<any> {
@@ -27,105 +18,14 @@ export default class OperationController {
     }
   }
 
-  // eslint-disable-next-line complexity, max-lines-per-function
-  async doOperation(
+  private async doOperation(
     repeat: number,
     operation: operations.Operation
   ): Promise<any> {
-    // eslint-disable-next-line complexity, max-lines-per-function
-    const opFunc = (() => {
-      switch (operation.type) {
-        case operations.TAB_CLOSE:
-          return () =>
-            this.tabUseCase.close(false, operation.select === "left");
-        case operations.TAB_CLOSE_RIGHT:
-          return () => this.tabUseCase.closeRight();
-        case operations.TAB_CLOSE_FORCE:
-          return () => this.tabUseCase.close(true);
-        case operations.TAB_REOPEN:
-          return () => this.tabUseCase.reopen();
-        case operations.TAB_PREV:
-          return () => this.tabSelectUseCase.selectPrev(1);
-        case operations.TAB_NEXT:
-          return () => this.tabSelectUseCase.selectNext(1);
-        case operations.TAB_FIRST:
-          return () => this.tabSelectUseCase.selectFirst();
-        case operations.TAB_LAST:
-          return () => this.tabSelectUseCase.selectLast();
-        case operations.TAB_PREV_SEL:
-          return () => this.tabSelectUseCase.selectPrevSelected();
-        case operations.TAB_RELOAD:
-          return () => this.tabUseCase.reload(operation.cache);
-        case operations.TAB_PIN:
-          return () => this.tabUseCase.setPinned(true);
-        case operations.TAB_UNPIN:
-          return () => this.tabUseCase.setPinned(false);
-        case operations.TAB_TOGGLE_PINNED:
-          return () => this.tabUseCase.togglePinned();
-        case operations.TAB_DUPLICATE:
-          return () => this.tabUseCase.duplicate();
-        case operations.PAGE_SOURCE:
-          return () => this.tabUseCase.openPageSource();
-        case operations.PAGE_HOME:
-          return () => this.tabUseCase.openHome(operation.newTab);
-        case operations.ZOOM_IN:
-          return () => this.zoomUseCase.zoomIn();
-        case operations.ZOOM_OUT:
-          return () => this.zoomUseCase.zoomOut();
-        case operations.ZOOM_NEUTRAL:
-          return () => this.zoomUseCase.zoomNutoral();
-        case operations.COMMAND_SHOW:
-          return () => this.consoleUseCase.showCommand();
-        case operations.COMMAND_SHOW_OPEN:
-          return () => this.consoleUseCase.showOpenCommand(operation.alter);
-        case operations.COMMAND_SHOW_TABOPEN:
-          return () => this.consoleUseCase.showTabopenCommand(operation.alter);
-        case operations.COMMAND_SHOW_WINOPEN:
-          return () => this.consoleUseCase.showWinopenCommand(operation.alter);
-        case operations.COMMAND_SHOW_BUFFER:
-          return () => this.consoleUseCase.showBufferCommand();
-        case operations.COMMAND_SHOW_ADDBOOKMARK:
-          return () =>
-            this.consoleUseCase.showAddbookmarkCommand(operation.alter);
-        case operations.FIND_START:
-          return () => this.findUseCase.findStart();
-        case operations.CANCEL:
-          return () => this.consoleUseCase.hideConsole();
-        case operations.NAVIGATE_HISTORY_PREV:
-          return () => this.navigateUseCase.openHistoryPrev();
-        case operations.NAVIGATE_HISTORY_NEXT:
-          return () => this.navigateUseCase.openHistoryNext();
-        case operations.NAVIGATE_LINK_PREV:
-          return () => this.navigateUseCase.openLinkPrev();
-        case operations.NAVIGATE_LINK_NEXT:
-          return () => this.navigateUseCase.openLinkNext();
-        case operations.NAVIGATE_PARENT:
-          return () => this.navigateUseCase.openParent();
-        case operations.NAVIGATE_ROOT:
-          return () => this.navigateUseCase.openRoot();
-        case operations.REPEAT_LAST:
-          return () => {
-            const last = this.repeatUseCase.getLastOperation();
-            if (typeof last !== "undefined") {
-              return this.doOperation(1, last);
-            }
-            return Promise.resolve();
-          };
-        case operations.INTERNAL_OPEN_URL:
-          return () =>
-            this.tabUseCase.openURL(
-              operation.url,
-              operation.newTab,
-              operation.newWindow
-            );
-        default:
-          throw new Error("unknown operation: " + operation.type);
-      }
-    })();
-
+    const operator = this.operatorFactory.create(operation);
     for (let i = 0; i < repeat; ++i) {
       // eslint-disable-next-line no-await-in-loop
-      await opFunc();
+      await operator.run();
     }
   }
 }