@geode/opengeodeweb-front 10.3.0 → 10.3.1

package/tests/unit/stores/Geode.nuxt.test.js

@@ -1,164 +1,162 @@
-import { setActivePinia } from "pinia"
-import { createTestingPinia } from "@pinia/testing"
-import { describe, test, expect, expectTypeOf, beforeEach, vi } from "vitest"
-import { registerEndpoint } from "@nuxt/test-utils/runtime"
+// Third party imports
 import back_schemas from "@geode/opengeodeweb-back/opengeodeweb_back_schemas.json"
-import Status from "@ogw_front/utils/status"
+import { registerEndpoint } from "@nuxt/test-utils/runtime"
+import { describe, expect, expectTypeOf, test, vi } from "vitest"
+
+// Local imports
 import { appMode } from "@ogw_front/utils/app_mode"
-import { useInfraStore } from "@ogw_front/stores/infra"
+import { setupActivePinia } from "../../utils"
+import Status from "@ogw_front/utils/status"
 import { useGeodeStore } from "@ogw_front/stores/geode"
+import { useInfraStore } from "@ogw_front/stores/infra"
 
-beforeEach(async () => {
-  const pinia = createTestingPinia({
-    stubActions: false,
-    createSpy: vi.fn,
-  })
-  setActivePinia(pinia)
+// CONSTANTS
+const PORT_443 = "443"
+const PORT_12 = "12"
+const PORT_5000 = "5000"
+const CLOUD_ID = "123456"
+const STATUS_500 = 500
+const EXPECTED_ONE_REQUEST = 1
+const EXPECTED_NO_REQUEST = 0
+
+beforeEach(() => {
+  setupActivePinia()
 })
 
-describe("Geode store", () => {
-  describe("state", () => {
-    test("initial state", () => {
+describe("geode store", () => {
+  test("state", () => {
+    const geodeStore = useGeodeStore()
+    expectTypeOf(geodeStore.default_local_port).toBeString()
+    expectTypeOf(geodeStore.request_counter).toBeNumber()
+    expectTypeOf(geodeStore.status).toBeString()
+  })
+
+  describe("protocol", () => {
+    test("app_mode CLOUD", () => {
+      const infraStore = useInfraStore()
+      const geodeStore = useGeodeStore()
+      infraStore.app_mode = appMode.CLOUD
+      expect(geodeStore.protocol).toBe("https")
+    })
+
+    test("app_mode BROWSER/DESKTOP", () => {
+      const infraStore = useInfraStore()
       const geodeStore = useGeodeStore()
-      expectTypeOf(geodeStore.default_local_port).toBeString()
-      expectTypeOf(geodeStore.request_counter).toBeNumber()
-      expectTypeOf(geodeStore.status).toBeString()
+      infraStore.app_mode = appMode.BROWSER
+      expect(geodeStore.protocol).toBe("http")
+      infraStore.app_mode = appMode.DESKTOP
+      expect(geodeStore.protocol).toBe("http")
     })
   })
 
-  describe("getters", () => {
-    describe("protocol", () => {
-      test("test app_mode CLOUD", () => {
-        const infraStore = useInfraStore()
-        const geodeStore = useGeodeStore()
-        infraStore.app_mode = appMode.CLOUD
-        expect(geodeStore.protocol).toBe("https")
-      })
-      test("test app_mode BROWSER", () => {
-        const infraStore = useInfraStore()
-        const geodeStore = useGeodeStore()
-        infraStore.app_mode = appMode.BROWSER
-        expect(geodeStore.protocol).toBe("http")
-      })
-      test("test app_mode DESKTOP", () => {
-        const infraStore = useInfraStore()
-        const geodeStore = useGeodeStore()
-        infraStore.app_mode = appMode.DESKTOP
-        expect(geodeStore.protocol).toBe("http")
-      })
+  describe("port", () => {
+    test("app_mode CLOUD", () => {
+      const infraStore = useInfraStore()
+      const geodeStore = useGeodeStore()
+      infraStore.app_mode = appMode.CLOUD
+      expect(geodeStore.port).toBe(PORT_443)
     })
 
-    describe("port", () => {
-      test("test app_mode CLOUD", () => {
-        const infraStore = useInfraStore()
-        const geodeStore = useGeodeStore()
-        infraStore.app_mode = appMode.CLOUD
-        expect(geodeStore.port).toBe("443")
-      })
-      test("test app_mode BROWSER", () => {
-        const infraStore = useInfraStore()
-        const geodeStore = useGeodeStore()
-        infraStore.app_mode = appMode.BROWSER
-        expect(geodeStore.port).toBe(geodeStore.default_local_port)
-      })
-      test("test app_mode DESKTOP", () => {
-        const infraStore = useInfraStore()
-        const geodeStore = useGeodeStore()
-        infraStore.app_mode = appMode.DESKTOP
-        expect(geodeStore.port).toBe(geodeStore.default_local_port)
-      })
+    test("app_mode BROWSER/DESKTOP", () => {
+      const infraStore = useInfraStore()
+      const geodeStore = useGeodeStore()
+      infraStore.app_mode = appMode.BROWSER
+      expect(geodeStore.port).toBe(geodeStore.default_local_port)
+      infraStore.app_mode = appMode.DESKTOP
+      expect(geodeStore.port).toBe(geodeStore.default_local_port)
+    })
 
-      test("test override default_local_port", () => {
-        const infraStore = useInfraStore()
-        const geodeStore = useGeodeStore()
-        infraStore.app_mode = appMode.DESKTOP
-        geodeStore.default_local_port = "12"
-        expect(geodeStore.port).toBe("12")
-      })
+    test("override default_local_port", () => {
+      const infraStore = useInfraStore()
+      const geodeStore = useGeodeStore()
+      infraStore.app_mode = appMode.DESKTOP
+      geodeStore.default_local_port = PORT_12
+      expect(geodeStore.port).toBe(PORT_12)
     })
+  })
 
-    describe("base_url", () => {
-      test("test app_mode BROWSER", () => {
-        const infraStore = useInfraStore()
-        const geodeStore = useGeodeStore()
-        infraStore.app_mode = appMode.BROWSER
-        infraStore.domain_name = "localhost"
-        expect(geodeStore.base_url).toBe("http://localhost:5000")
-      })
-      test("test app_mode CLOUD", () => {
-        const infraStore = useInfraStore()
-        const geodeStore = useGeodeStore()
-        infraStore.app_mode = appMode.CLOUD
-        infraStore.ID = "123456"
-        infraStore.domain_name = "example.com"
-        expect(geodeStore.base_url).toBe("https://example.com:443/123456/geode")
-      })
-      test("test app_mode CLOUD, ID empty", () => {
-        const infraStore = useInfraStore()
-        const geodeStore = useGeodeStore()
-        infraStore.app_mode = appMode.CLOUD
-        infraStore.ID = ""
-        infraStore.domain_name = "example.com"
-        expect(() => geodeStore.base_url).toThrowError(
-          "ID must not be empty in cloud mode",
-        )
-      })
+  describe("base_url", () => {
+    test("app_mode BROWSER", () => {
+      const infraStore = useInfraStore()
+      const geodeStore = useGeodeStore()
+      infraStore.app_mode = appMode.BROWSER
+      infraStore.domain_name = "localhost"
+      expect(geodeStore.base_url).toBe(`http://localhost:${PORT_5000}`)
     })
 
-    describe("is_busy", () => {
-      test("test is_busy", () => {
-        const geodeStore = useGeodeStore()
-        geodeStore.request_counter = 1
-        expect(geodeStore.is_busy).toBe(true)
-      })
-      test("test not is_busy", () => {
-        const geodeStore = useGeodeStore()
-        geodeStore.request_counter = 0
-        expect(geodeStore.is_busy).toBe(false)
-      })
+    test("app_mode CLOUD", () => {
+      const infraStore = useInfraStore()
+      const geodeStore = useGeodeStore()
+      infraStore.app_mode = appMode.CLOUD
+      infraStore.ID = CLOUD_ID
+      infraStore.domain_name = "example.com"
+      expect(geodeStore.base_url).toBe(
+        `https://example.com:${PORT_443}/${CLOUD_ID}/geode`,
+      )
+    })
+
+    test("app_mode CLOUD, ID empty", () => {
+      const infraStore = useInfraStore()
+      const geodeStore = useGeodeStore()
+      infraStore.app_mode = appMode.CLOUD
+      infraStore.ID = ""
+      infraStore.domain_name = "example.com"
+      expect(() => geodeStore.base_url).toThrow(
+        "ID must not be empty in cloud mode",
+      )
     })
   })
 
-  describe("actions", () => {
-    describe("ping", () => {
-      const getFakeCall = vi.fn()
-      registerEndpoint(back_schemas.opengeodeweb_back.ping.$id, getFakeCall)
-
-      test("response", async () => {
-        const geodeStore = useGeodeStore()
-        geodeStore.base_url = ""
-        getFakeCall.mockImplementation(() => ({}))
-        await geodeStore.ping()
-        expect(geodeStore.status).toBe(Status.CONNECTED)
-      })
-      test("response_error", async () => {
-        const geodeStore = useGeodeStore()
-        geodeStore.base_url = ""
-        getFakeCall.mockImplementation(() => {
-          throw createError({
-            status: 500,
-          })
-        })
-        await expect(geodeStore.ping()).rejects.toThrow()
-        expect(geodeStore.status).toBe(Status.NOT_CONNECTED)
-      })
+  describe("is_busy", () => {
+    test("is_busy", () => {
+      const geodeStore = useGeodeStore()
+      geodeStore.request_counter = EXPECTED_ONE_REQUEST
+      expect(geodeStore.is_busy).toBeTruthy()
     })
 
-    describe("start_request", () => {
-      test("test increment", async () => {
-        const geodeStore = useGeodeStore()
-        expect(geodeStore.request_counter).toBe(0)
-        await geodeStore.start_request()
-        expect(geodeStore.request_counter).toBe(1)
-      })
+    test("not is_busy", () => {
+      const geodeStore = useGeodeStore()
+      geodeStore.request_counter = EXPECTED_NO_REQUEST
+      expect(geodeStore.is_busy).toBeFalsy()
     })
-    describe("stop_request", () => {
-      test("test decrement", async () => {
-        const geodeStore = useGeodeStore()
-        geodeStore.request_counter = 1
-        await geodeStore.stop_request()
-        expect(geodeStore.request_counter).toBe(0)
+  })
+})
+
+describe("geode store actions", () => {
+  const getFakeCall = vi.fn()
+
+  beforeEach(() => {
+    registerEndpoint(back_schemas.opengeodeweb_back.ping.$id, getFakeCall)
+  })
+
+  describe("ping", () => {
+    test("response", async () => {
+      const geodeStore = useGeodeStore()
+      geodeStore.base_url = ""
+      getFakeCall.mockImplementation(() => ({}))
+      await geodeStore.ping()
+      expect(geodeStore.status).toBe(Status.CONNECTED)
+    })
+
+    test("response_error", async () => {
+      const geodeStore = useGeodeStore()
+      geodeStore.base_url = ""
+      getFakeCall.mockImplementation(() => {
+        throw createError({ status: STATUS_500 })
       })
+      await expect(geodeStore.ping()).rejects.toThrow("500")
+      expect(geodeStore.status).toBe(Status.NOT_CONNECTED)
+    })
+  })
+
+  describe("request counter", () => {
+    test("increment/decrement", async () => {
+      const geodeStore = useGeodeStore()
+      expect(geodeStore.request_counter).toBe(EXPECTED_NO_REQUEST)
+      await geodeStore.start_request()
+      expect(geodeStore.request_counter).toBe(EXPECTED_ONE_REQUEST)
+      await geodeStore.stop_request()
+      expect(geodeStore.request_counter).toBe(EXPECTED_NO_REQUEST)
     })
   })
 })

package/tests/unit/stores/Infra.nuxt.test.js

@@ -1,22 +1,20 @@
-// Global imports
-
 // Third party imports
-import { registerEndpoint } from "@nuxt/test-utils/runtime"
-import { setActivePinia } from "pinia"
-import { createTestingPinia } from "@pinia/testing"
 import { beforeEach, describe, expect, expectTypeOf, test, vi } from "vitest"
+import { registerEndpoint } from "@nuxt/test-utils/runtime"
+
 // Local imports
 import Status from "@ogw_front/utils/status"
 import { appMode } from "@ogw_front/utils/app_mode"
-import { useInfraStore } from "@ogw_front/stores/infra"
+import { setupActivePinia } from "../../utils"
 import { useGeodeStore } from "@ogw_front/stores/geode"
-import { useViewerStore } from "@ogw_front/stores/viewer"
+import { useInfraStore } from "@ogw_front/stores/infra"
 import { useLambdaStore } from "@ogw_front/stores/lambda"
+import { useViewerStore } from "@ogw_front/stores/viewer"
 
 // Mock navigator.locks API
-const mockLockRequest = vi.fn().mockImplementation(async (name, callback) => {
-  return callback({ name })
-})
+const mockLockRequest = vi
+  .fn()
+  .mockImplementation(async (name, handler) => await handler({ name }))
 
 vi.stubGlobal("navigator", {
   ...navigator,
@@ -25,12 +23,8 @@ vi.stubGlobal("navigator", {
   },
 })
 
-beforeEach(async () => {
-  const pinia = createTestingPinia({
-    stubActions: false,
-    createSpy: vi.fn,
-  })
-  setActivePinia(pinia)
+beforeEach(() => {
+  setupActivePinia()
 })
 
 describe("Infra Store", () => {
@@ -71,7 +65,7 @@ describe("Infra Store", () => {
     describe("microservices_connected", () => {
       test("test no microservices registered", () => {
         const infraStore = useInfraStore()
-        expect(infraStore.microservices_connected).toBe(true)
+        expect(infraStore.microservices_connected).toBeTruthy()
       })
       test("test geode false & viewer false", () => {
         const infraStore = useInfraStore()
@@ -91,7 +85,7 @@ describe("Infra Store", () => {
 
         geodeStore.$patch({ status: Status.NOT_CONNECTED })
         viewerStore.$patch({ status: Status.NOT_CONNECTED })
-        expect(infraStore.microservices_connected).toBe(false)
+        expect(infraStore.microservices_connected).toBeFalsy()
       })
       test("test geode true & viewer false", () => {
         const infraStore = useInfraStore()
@@ -111,7 +105,7 @@ describe("Infra Store", () => {
 
         geodeStore.$patch({ status: Status.CONNECTED })
         viewerStore.$patch({ status: Status.NOT_CONNECTED })
-        expect(infraStore.microservices_connected).toBe(false)
+        expect(infraStore.microservices_connected).toBeFalsy()
       })
       test("test geode false & viewer true", () => {
         const infraStore = useInfraStore()
@@ -131,7 +125,7 @@ describe("Infra Store", () => {
 
         geodeStore.$patch({ status: Status.NOT_CONNECTED })
         viewerStore.$patch({ status: Status.CONNECTED })
-        expect(infraStore.microservices_connected).toBe(false)
+        expect(infraStore.microservices_connected).toBeFalsy()
       })
       test("test geode true & viewer true", () => {
         const infraStore = useInfraStore()
@@ -151,14 +145,14 @@ describe("Infra Store", () => {
 
         geodeStore.$patch({ status: Status.CONNECTED })
         viewerStore.$patch({ status: Status.CONNECTED })
-        expect(infraStore.microservices_connected).toBe(true)
+        expect(infraStore.microservices_connected).toBeTruthy()
       })
     })
 
     describe("microservices_busy", () => {
       test("test no microservices registered", () => {
         const infraStore = useInfraStore()
-        expect(infraStore.microservices_busy).toBe(false)
+        expect(infraStore.microservices_busy).toBeFalsy()
       })
       test("test geode false & viewer false", () => {
         const infraStore = useInfraStore()
@@ -178,7 +172,7 @@ describe("Infra Store", () => {
 
         geodeStore.$patch({ request_counter: 0 })
         viewerStore.$patch({ request_counter: 0 })
-        expect(infraStore.microservices_busy).toBe(false)
+        expect(infraStore.microservices_busy).toBeFalsy()
       })
       test("test geode true & viewer false", () => {
         const infraStore = useInfraStore()
@@ -198,7 +192,7 @@ describe("Infra Store", () => {
 
         geodeStore.$patch({ request_counter: 1 })
         viewerStore.$patch({ request_counter: 0 })
-        expect(infraStore.microservices_busy).toBe(true)
+        expect(infraStore.microservices_busy).toBeTruthy()
       })
       test("test geode false & viewer true", () => {
         const infraStore = useInfraStore()
@@ -218,7 +212,7 @@ describe("Infra Store", () => {
 
         geodeStore.$patch({ request_counter: 0 })
         viewerStore.$patch({ request_counter: 1 })
-        expect(infraStore.microservices_busy).toBe(true)
+        expect(infraStore.microservices_busy).toBeTruthy()
       })
       test("test geode true & viewer true", () => {
         const infraStore = useInfraStore()
@@ -238,7 +232,7 @@ describe("Infra Store", () => {
 
         geodeStore.$patch({ request_counter: 1 })
         viewerStore.$patch({ request_counter: 1 })
-        expect(infraStore.microservices_busy).toBe(true)
+        expect(infraStore.microservices_busy).toBeTruthy()
       })
     })
   })

package/tests/unit/stores/Lambda.nuxt.test.js

@@ -1,25 +1,32 @@
-// Global imports
-
 // Third party imports
-import { registerEndpoint } from "@nuxt/test-utils/runtime"
-import { setActivePinia } from "pinia"
-import { createTestingPinia } from "@pinia/testing"
 import { beforeEach, describe, expect, expectTypeOf, test, vi } from "vitest"
+import { registerEndpoint } from "@nuxt/test-utils/runtime"
 
+// Local imports
+import Status from "@ogw_front/utils/status"
+import { setupActivePinia } from "../../utils"
 import { useFeedbackStore } from "@ogw_front/stores/feedback"
 import { useLambdaStore } from "@ogw_front/stores/lambda"
 
-// Local imports
-import Status from "@ogw_front/utils/status"
+// CONSTANTS
+const PORT_443 = "443"
+const API_URL = "api.example.com"
+const SITE_BRANCH = "/test"
+const PROJECT = "/project"
+const TEST_ID = "test-id-123456"
+const STATUS_500 = 500
 
 beforeEach(async () => {
-  const pinia = createTestingPinia({
-    stubActions: false,
-    createSpy: vi.fn,
-  })
-  setActivePinia(pinia)
+  setupActivePinia()
 })
 
+function setupConfig() {
+  const config = useRuntimeConfig()
+  config.public.API_URL = API_URL
+  config.public.SITE_BRANCH = SITE_BRANCH
+  config.public.PROJECT = PROJECT
+}
+
 describe("Lambda Store", () => {
   describe("state", () => {
     test("initial state", () => {
@@ -40,18 +47,16 @@ describe("Lambda Store", () => {
     describe("port", () => {
       test("test port is always 443", () => {
         const lambdaStore = useLambdaStore()
-        expect(lambdaStore.port).toBe("443")
+        expect(lambdaStore.port).toBe(PORT_443)
       })
     })
 
     describe("base_url", () => {
       test("test base_url construction", () => {
+        setupConfig()
         const lambdaStore = useLambdaStore()
-        useRuntimeConfig().public.API_URL = "api.example.com"
-        useRuntimeConfig().public.SITE_BRANCH = "/test"
-        useRuntimeConfig().public.PROJECT = "/project"
         expect(lambdaStore.base_url).toBe(
-          "https://api.example.com:443/test/project/createbackend",
+          `https://${API_URL}:${PORT_443}${SITE_BRANCH}${PROJECT}/createbackend`,
         )
       })
     })
@@ -59,7 +64,7 @@ describe("Lambda Store", () => {
     describe("is_busy", () => {
       test("test is_busy is always false", () => {
         const lambdaStore = useLambdaStore()
-        expect(lambdaStore.is_busy).toBe(false)
+        expect(lambdaStore.is_busy).toBeFalsy()
       })
     })
   })
@@ -69,13 +74,10 @@ describe("Lambda Store", () => {
       const postFakeCall = vi.fn()
 
       test("successful launch", async () => {
+        setupConfig()
         const lambdaStore = useLambdaStore()
         const feedbackStore = useFeedbackStore()
 
-        useRuntimeConfig().public.API_URL = "api.example.com"
-        useRuntimeConfig().public.SITE_BRANCH = "/test"
-        useRuntimeConfig().public.PROJECT = "/project"
-
         lambdaStore.base_url = "test-base-url"
         registerEndpoint(lambdaStore.base_url, {
           method: "POST",
@@ -83,24 +85,21 @@ describe("Lambda Store", () => {
         })
 
         postFakeCall.mockImplementation(() => ({
-          ID: "test-id-123456",
+          ID: TEST_ID,
         }))
 
         const id = await lambdaStore.launch()
 
         expect(lambdaStore.status).toBe(Status.CONNECTED)
-        expect(id).toBe("test-id-123456")
-        expect(feedbackStore.server_error).toBe(false)
+        expect(id).toBe(TEST_ID)
+        expect(feedbackStore.server_error).toBeFalsy()
       })
 
       test("failed launch - error response", async () => {
+        setupConfig()
         const lambdaStore = useLambdaStore()
         const feedbackStore = useFeedbackStore()
 
-        useRuntimeConfig().public.API_URL = "api.example.com"
-        useRuntimeConfig().public.SITE_BRANCH = "/test"
-        useRuntimeConfig().public.PROJECT = "/project"
-
         registerEndpoint(lambdaStore.base_url, {
           method: "POST",
           handler: postFakeCall,
@@ -108,7 +107,7 @@ describe("Lambda Store", () => {
 
         postFakeCall.mockImplementation(() => {
           throw createError({
-            status: 500,
+            status: STATUS_500,
             statusMessage: "Internal Server Error",
           })
         })
@@ -118,7 +117,7 @@ describe("Lambda Store", () => {
         )
 
         expect(lambdaStore.status).toBe(Status.NOT_CONNECTED)
-        expect(feedbackStore.server_error).toBe(true)
+        expect(feedbackStore.server_error).toBeTruthy()
       })
     })