feat(ping_route): fix tests and cleanup

Author: JulienChampagnol

nuxt.config.js

@@ -9,7 +9,10 @@ export default defineNuxtConfig({
     },
   },
 
-  modules: [["@pinia/nuxt", { autoImports: ["defineStore"] }], "@vueuse/nuxt"],
+  modules: [
+    ["@pinia/nuxt", { autoImports: ["defineStore", "storeToRefs"] }],
+    "@vueuse/nuxt",
+  ],
   imports: {
     dirs: ["stores"],
   },

stores/viewer.js

@@ -21,22 +21,23 @@ export const use_viewer_store = defineStore("viewer", {
         return "ws"
       }
     },
-    port() {
+    port(state) {
       if (use_infra_store().is_cloud) {
         return "443"
       } else {
-        return this.default_local_port
+        return state.default_local_port
       }
     },
-    base_url() {
+    base_url(state) {
       const infra_store = use_infra_store()
-      var viewer_url = `${this.protocol}://${infra_store.domain_name}:${this.port}`
+      let viewer_url = `${state.protocol}://${infra_store.domain_name}:${state.port}`
       if (infra_store.is_cloud) {
         if (infra_store.ID == "") {
           throw new Error("ID must not be empty in cloud mode")
         }
         viewer_url += `/${infra_store.ID}/viewer`
       }
+      viewer_url += "/ws"
       return viewer_url
     },
     is_busy(state) {

test/stores/Feedback.test.js

@@ -1,7 +1,7 @@
 import { setActivePinia } from "pinia"
 import { createTestingPinia } from "@pinia/testing"
 import { use_feedback_store } from "@/stores/feedback"
-import { describe, test, expect, beforeEach } from "vitest"
+import { describe, test, expect, expectTypeOf, beforeEach } from "vitest"
 
 describe("Feedback Store", () => {
   const pinia = createTestingPinia({

test/stores/Geode.test.js

@@ -97,34 +97,30 @@ describe("Geode Store", () => {
 
   describe("actions", () => {
     describe("do_ping", () => {
-      // test("request_error", async () => {
-      //   geode_store.base_url = ""
-      //   await geode_store.do_ping()
-
-      //   expect(geode_store.is_running).toBe(false)
-      //   expect(feedback_store.server_error).toBe(true)
-      // })
+      test("request_error", async () => {
+        geode_store.base_url = ""
+        geode_store.is_running = true
+        try {
+          await geode_store.do_ping()
+        } catch (e) {
+          console.log(e)
+        }
+
+        expect(geode_store.is_running).toBe(false)
+        expect(feedback_store.server_error).toBe(true)
+        expect(feedback_store.feedbacks.length).toBe(1)
+      })
 
       test("response", async () => {
         geode_store.base_url = ""
-
         registerEndpoint(back_schemas.opengeodeweb_back.ping.$id, {
           method: "POST",
           handler: () => ({}),
         })
         await geode_store.do_ping()
         expect(geode_store.is_running).toBe(true)
+        expect(feedback_store.server_error).toBe(false)
       })
-
-      // test("response_error", async () => {
-      //   geode_store.base_url = ""
-      //   registerEndpoint(back_schemas.opengeodeweb_back.ping.$id, {
-      //     method: "POST",
-      //     handler: () => ({}),
-      //   })
-      //   await geode_store.do_ping()
-      //   expect(geode_store.is_running).toBe(false)
-      // })
     })
 
     describe("start_request", () => {

test/stores/Infra.nuxt.test.js

@@ -1,9 +1,9 @@
-import { describe, test, expect, beforeEach } from "vitest"
+import { describe, test, expect, expectTypeOf, beforeEach } from "vitest"
 import { registerEndpoint } from "@nuxt/test-utils/runtime"
 import { setActivePinia } from "pinia"
 import { createTestingPinia } from "@pinia/testing"
 
-describe("Cloud Store", () => {
+describe("Infra Store", () => {
   const pinia = createTestingPinia({
     stubActions: false,
   })
@@ -46,6 +46,17 @@ describe("Cloud Store", () => {
         expect(infra_store.domain_name).toBe("api.geode-solutions.com")
       })
     })
+
+    describe("lambda_url", () => {
+      test("test is cloud true", () => {
+        useRuntimeConfig().public.SITE_BRANCH = "/test"
+        useRuntimeConfig().public.PROJECT = "/project"
+        infra_store.is_cloud = true
+        expect(infra_store.lambda_url).toBe(
+          "https://api.geode-solutions.com:443/test/project/createbackend",
+        )
+      })
+    })
     describe("is_running", () => {
       test("test geode false & viewer false", () => {
         geode_store.$patch({ is_running: false })
@@ -93,22 +104,28 @@ describe("Cloud Store", () => {
     })
   })
 
-  // describe("actions", () => {
-  //   describe("create_backend", async () => {
-  //     test("test without end-point", async () => {
-  //       expect(geode_store.is_running).toBe(false)
-  //       await infra_store.create_backend()
-  //       expect(geode_store.is_running).toBe(false)
-  //     })
-  //     test("test with end-point", async () => {
-  //       expect(geode_store.is_running).toBe(false)
-  //       registerEndpoint("/createbackend", {
-  //         method: "POST",
-  //         handler: () => ({ ID: "123456" }),
-  //       })
-  //       await infra_store.create_backend()
-  //       await flushPromises()
-  //     })
-  //   })
-  // })
+  describe("actions", () => {
+    describe("create_connexion", async () => {
+      test("test without end-point", async () => {
+        await infra_store.create_connexion()
+        expect(infra_store.is_connexion_launched).toBe(true)
+        expect(feedback_store.server_error).toBe(true)
+      })
+    })
+    describe("create_backend", async () => {
+      test("test without end-point", async () => {
+        await infra_store.create_backend()
+        expect(geode_store.is_running).toBe(false)
+        expect(feedback_store.server_error).toBe(true)
+      })
+      test("test with end-point", async () => {
+        registerEndpoint(infra_store.lambda_url, {
+          method: "POST",
+          handler: () => ({ ID: "123456" }),
+        })
+        await infra_store.create_backend()
+        expect(geode_store.is_running).toBe(true)
+      })
+    })
+  })
 })

test/stores/Viewer.nuxt.test.js

@@ -1,6 +1,6 @@
 import { setActivePinia } from "pinia"
 import { createTestingPinia } from "@pinia/testing"
-import { describe, test, expect, beforeEach } from "vitest"
+import { describe, test, expect, expectTypeOf, beforeEach } from "vitest"
 
 describe("Viewer Store", () => {
   const pinia = createTestingPinia({
@@ -61,15 +61,15 @@ describe("Viewer Store", () => {
       test("test is_cloud false", () => {
         infra_store.is_cloud = false
         infra_store.domain_name = "localhost"
-        expect(viewer_store.base_url).toBe("ws://localhost:1234")
+        expect(viewer_store.base_url).toBe("ws://localhost:1234/ws")
       })
 
       test("test is_cloud true", async () => {
         infra_store.is_cloud = true
         infra_store.ID = "123456"
         infra_store.domain_name = "example.com"
         expect(viewer_store.base_url).toBe(
-          "wss://example.com:443/123456/viewer",
+          "wss://example.com:443/123456/viewer/ws",
         )
       })