feat(operations): add tests for getProjectName

src/util/operations.spec.tsx

@@ -1,4 +1,4 @@
-import { getInstanceName } from "./operations";
+import { getInstanceName, getProjectName } from "./operations";
 import { LxdOperation } from "types/operation";
 
 const craftOperation = (url: string) => {
@@ -44,3 +44,30 @@ describe("getInstanceName", () => {
     expect(name).toBe("testInstance3");
   });
 });
+
+describe("getProjectName", () => {
+  it("identifies project name from an instance operation when no project parameter is present", () => {
+    const operation = craftOperation("/1.0/instances/testInstance1");
+    const name = getProjectName(operation);
+
+    expect(name).toBe("default");
+  });
+
+  it("identifies project name from an instance operation in a custom project", () => {
+    const operation = craftOperation(
+      "/1.0/instances/testInstance2?project=fooProject"
+    );
+    const name = getProjectName(operation);
+
+    expect(name).toBe("fooProject");
+  });
+
+  it("identifies project name from an instance operation in a custom project with other parameters", () => {
+    const operation = craftOperation(
+      "/1.0/instances/testInstance2?foo=bar&project=barProject"
+    );
+    const name = getProjectName(operation);
+
+    expect(name).toBe("barProject");
+  });
+});

src/util/operations.tsx

@@ -17,13 +17,15 @@ export const getInstanceName = (operation: LxdOperation): string => {
 export const getProjectName = (operation: LxdOperation): string => {
   // the url can be
   // /1.0/instances/<instance_name>?project=<project_name>
+  // /1.0/instances/<instance_name>?other=params&project=<project_name>
+  // /1.0/instances/testInstance1?other=params&project=<project_name>&other=params
   // when no project parameter is present, the project will be "default"
 
   return (
     operation.resources?.instances
       ?.filter((item) => item.startsWith("/1.0/instances/"))
-      .map((item) => item.split("/")[3])
+      .map((item) => item.split("project=")[1])
       .pop()
-      ?.split("=")[1] ?? "default"
+      ?.split("&")[0] ?? "default"
   );
 };