add unit test for fetchVipMetadataInformation method

Author: Salipa-Gurung

src/App.vue

@@ -226,13 +226,12 @@ export default defineComponent({
     // get resource, ensure resource url is not empty!
     if (this.url != null && this.url != undefined && this.url != '') {
       this.dicomUrl = await this.addWadouriPrefix(this.url)
-    }
-
-    // get vip metadata
-    await this.fetchVipMetadataInformation(await this.addWadouriPrefix(this.url))
+      // get vip metadata
+      await this.fetchVipMetadataInformation(this.dicomUrl)
 
-    // prefetch all other metadata (in separate function for performance reasons)
-    await this.fetchMetadataInformation(await this.addWadouriPrefix(this.url))
+      // prefetch all other metadata (in separate function for performance reasons)
+      await this.fetchMetadataInformation(this.dicomUrl)
+    }
   },
   async mounted() {
     // check if cornerstone core is initialized
@@ -312,11 +311,9 @@ export default defineComponent({
 
       if (!this.isDicomImageDataFetched) {
         this.dicomImageData = await fetchDicomImageData(imageId)
-        if (!this.isDicomImageDataFetched) {
           if (this.dicomImageData != null){
             this.isDicomImageDataFetched = true
           }
-        }
       }
 
       this.vipInformation.patientName = this.dicomImageData.string(findDicomTagByValue('patientName'))

tests/unit/App.spec.ts

@@ -1,4 +1,4 @@
-import { shallowMount } from "@vue/test-utils";
+import { shallowMount, flushPromises } from "@vue/test-utils";
 import App from "../../src/App.vue";
 import { vi } from "vitest";
 import { defaultPlugins } from "../../src/helper/defaultPlugins";
@@ -21,6 +21,12 @@ vi.mock("@cornerstonejs/core", () => {
         return {
           setStack: vi.fn(),
           render: vi.fn(),
+          getCamera: vi.fn().mockImplementation(() => {
+            return { parallelScale: 137.3853139193763 };
+          }),
+          getImageData: vi.fn().mockImplementation(() => {
+            return { dimensions: [] };
+          }),
         };
       }
       enableElement() {}
@@ -60,16 +66,40 @@ vi.mock("dicom-parser", () => ({
 }));
 
 describe("App component", () => {
+  const spyFetchVipMetadataInformation = vi
+    .spyOn(App.methods, "fetchVipMetadataInformation")
+    .mockImplementation(vi.fn());
+  const spyFetchMetadataInformation = vi
+    .spyOn(App.methods, "fetchMetadataInformation")
+    .mockImplementation(vi.fn());
+  const spyAddWadouriPrefix = vi
+    .spyOn(App.methods, "addWadouriPrefix")
+    .mockImplementation(vi.fn());
+
+  it("should not fetch overlay metadata when the url is not provided", async () => {
+    getWrapper();
+    await flushPromises();
+
+    expect(spyAddWadouriPrefix).toHaveBeenCalledTimes(0);
+    expect(spyFetchVipMetadataInformation).toHaveBeenCalledTimes(0);
+    expect(spyFetchMetadataInformation).toHaveBeenCalledTimes(0);
+  });
+  it("should fetch overlay metadata when the url is provided", async () => {
+    getWrapper({ props: { url: "https://test" } });
+    await flushPromises();
+
+    expect(spyAddWadouriPrefix).toHaveBeenCalledTimes(2);
+    expect(spyFetchVipMetadataInformation).toHaveBeenCalledTimes(1);
+    expect(spyFetchMetadataInformation).toHaveBeenCalledTimes(1);
+  });
+
   describe("Methods", () => {
-    vi.spyOn(App.methods, "fetchVipMetadataInformation").mockImplementation(
-      vi.fn()
-    );
-    vi.spyOn(App.methods, "fetchMetadataInformation").mockImplementation(
-      vi.fn()
-    );
     const wrapper = getWrapper();
-
     describe("method: wadouri", () => {
+      beforeAll(() => {
+        spyAddWadouriPrefix.mockRestore();
+      });
+
       it('should add "wadouri" prefix', async () => {
         expect(await wrapper.vm.addWadouriPrefix("https://dummy_url")).toBe(
           "wadouri:https://dummy_url"
@@ -124,7 +154,9 @@ describe("App component", () => {
       ])(
         `should return $expected for $description`,
         ({ date, time, expected }) => {
-          expect(wrapper.vm.formatOverlayDateAndTime(date, time)).toBe(expected);
+          expect(wrapper.vm.formatOverlayDateAndTime(date, time)).toBe(
+            expected
+          );
         }
       );
     });