fix(instance) ensure instance name is unique and ot matched against p…

…revious name on renaming Signed-off-by: David Edler <[email protected]>
canonical · Jun 25, 2024 · 1693d00 · 1693d00
1 parent 9f9f47d
commit 1693d00
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 10 deletions.
diff --git a/src/components/RenameHeader.tsx b/src/components/RenameHeader.tsx
@@ -35,14 +35,14 @@ const RenameHeader: FC<Props> = ({
   renameDisabledReason,
 }) => {
   const canRename = renameDisabledReason === undefined;
-  const enableRename = () => {
-    window.dispatchEvent(new Event("resize"));
+  const toggleRename = () => {
+    setTimeout(() => window.dispatchEvent(new Event("resize")), 10);
     if (!canRename || !formik) {
       return;
     }
     void formik.setValues({
       ...formik.values,
-      isRenaming: true,
+      isRenaming: !formik.values.isRenaming,
     });
   };
 
@@ -82,9 +82,7 @@ const RenameHeader: FC<Props> = ({
                   <Button
                     appearance="base"
                     className="cancel"
-                    onClick={() =>
-                      void formik.setFieldValue("isRenaming", false)
-                    }
+                    onClick={toggleRename}
                   >
                     Cancel
                   </Button>
@@ -101,7 +99,7 @@ const RenameHeader: FC<Props> = ({
             ) : (
               <li
                 className="p-heading--4 u-no-margin--bottom name continuous-breadcrumb"
-                onClick={enableRename}
+                onClick={toggleRename}
                 title={`Rename ${name}`}
               >
                 <Tooltip

diff --git a/src/pages/instances/InstanceDetailHeader.tsx b/src/pages/instances/InstanceDetailHeader.tsx
@@ -36,7 +36,7 @@ const InstanceDetailHeader: FC<Props> = ({
   const controllerState = useState<AbortController | null>(null);
 
   const RenameSchema = Yup.object().shape({
-    name: instanceNameValidation(project, controllerState).required(
+    name: instanceNameValidation(project, controllerState, name).required(
       "Instance name is required",
     ),
   });

diff --git a/src/util/instances.tsx b/src/util/instances.tsx
@@ -30,10 +30,15 @@ export const instanceLinkFromOperation = (args: {
 export const instanceNameValidation = (
   project: string,
   controllerState: AbortControllerState,
+  oldName?: string,
 ): Yup.StringSchema =>
   Yup.string()
-    .test("deduplicate", "An instance with this name already exists", (value) =>
-      checkDuplicateName(value, project, controllerState, "instances"),
+    .test(
+      "deduplicate",
+      "An instance with this name already exists",
+      (value) =>
+        oldName === value ||
+        checkDuplicateName(value, project, controllerState, "instances"),
     )
     .test(
       "size",