add tests

qdraw · Feb 22, 2024 · 3bc8c9f · 3bc8c9f
1 parent e24c49e
commit 3bc8c9f
Show file tree

Hide file tree

Showing 2 changed files with 46 additions and 3 deletions.
diff --git a/...es/menu-option-desktop-editor-open-single/menu-option-desktop-editor-open-single.spec.tsx b/...es/menu-option-desktop-editor-open-single/menu-option-desktop-editor-open-single.spec.tsx
@@ -5,7 +5,9 @@ import { IEnvFeatures } from "../../../interfaces/IEnvFeatures";
 import * as FetchPost from "../../../shared/fetch/fetch-post";
 import { UrlQuery } from "../../../shared/url-query";
 import * as Notification from "../../atoms/notification/notification";
-import MenuOptionDesktopEditorOpenSingle from "./menu-option-desktop-editor-open-single";
+import MenuOptionDesktopEditorOpenSingle, {
+  OpenDesktopSingle
+} from "./menu-option-desktop-editor-open-single";
 
 describe("MenuOptionDesktopEditorOpenSingle", () => {
   it("should render without errors", () => {
@@ -62,6 +64,41 @@ describe("MenuOptionDesktopEditorOpenSingle", () => {
     container.unmount();
   });
 
+  it("calls StartMenuOptionDesktopEditorOpenSelection on hotkey trigger", async () => {
+    const mockIConnectionDefaultResolve: Promise<IConnectionDefault> = Promise.resolve({
+      data: true,
+      statusCode: 200
+    } as IConnectionDefault);
+
+    const fetchPostSpy = jest
+      .spyOn(FetchPost, "default")
+      .mockReset()
+      .mockImplementationOnce(() => mockIConnectionDefaultResolve)
+      .mockImplementationOnce(() => mockIConnectionDefaultResolve);
+
+    const container = render(
+      <MenuOptionDesktopEditorOpenSingle
+        subPath={"/test.jpg"}
+        collections={false}
+        isReadOnly={false}
+      />
+    );
+
+    fireEvent.keyDown(document.body, { key: "e", ctrlKey: true });
+
+    await waitFor(() => {
+      expect(fetchPostSpy).toHaveBeenCalled();
+      expect(fetchPostSpy).toHaveBeenCalledTimes(1);
+
+      expect(fetchPostSpy).toHaveBeenNthCalledWith(
+        1,
+        new UrlQuery().UrlApiDesktopEditorOpen(),
+        "f=%2Ftest.jpg&collections=false"
+      );
+      container.unmount();
+    });
+  });
+
   it("feature toggle disabled", () => {
     const mockGetIConnectionDefaultFeatureToggle = {
       statusCode: 200,
@@ -148,4 +185,9 @@ describe("MenuOptionDesktopEditorOpenSingle", () => {
     });
     container.unmount();
   });
+
+  it("OpenDesktopSingle readonly should skip", async () => {
+    const result = await OpenDesktopSingle("/", false, jest.fn(), "error", true);
+    expect(result).toBeFalsy();
+  });
 });
diff --git a/...lecules/menu-option-desktop-editor-open-single/menu-option-desktop-editor-open-single.tsx b/...lecules/menu-option-desktop-editor-open-single/menu-option-desktop-editor-open-single.tsx
@@ -23,9 +23,9 @@ export async function OpenDesktopSingle(
   setIsError: React.Dispatch<React.SetStateAction<string>>,
   messageDesktopEditorUnableToOpen: string,
   isReadOnly: boolean
-) {
+): Promise<boolean> {
   if (isReadOnly) {
-    return;
+    return false;
   }
   const urlOpen = new UrlQuery().UrlApiDesktopEditorOpen();
 
@@ -37,6 +37,7 @@ export async function OpenDesktopSingle(
   if (openDesktopResult.statusCode >= 300) {
     setIsError(messageDesktopEditorUnableToOpen);
   }
+  return true;
 }
 
 const MenuOptionDesktopEditorOpenSingle: React.FunctionComponent<IMenuOptionDesktopEditorOpenSingleProps> =