fix: change storage layout sidepanel MAASENG-2798 (#5335)

canonical · Mar 11, 2024 · cb8f225 · cb8f225
1 parent 38fc68e
commit cb8f225
Show file tree

Hide file tree

Showing 6 changed files with 9 additions and 4 deletions.
diff --git a/src/app/machines/components/MachineForms/MachineForms.tsx b/src/app/machines/components/MachineForms/MachineForms.tsx
@@ -69,7 +69,7 @@ export const MachineForms = ({
       return <AddChassisForm clearSidePanelContent={clearSidePanelContent} />;
     case MachineSidePanelViews.ADD_MACHINE:
       return <AddMachineForm clearSidePanelContent={clearSidePanelContent} />;
-    case MachineSidePanelViews.CHANGE_STORAGE_LAYOUT: {
+    case MachineSidePanelViews.APPLY_STORAGE_LAYOUT: {
       if (!systemId || !selectedLayout) {
         return null;
       }

diff --git a/src/app/machines/constants.ts b/src/app/machines/constants.ts
@@ -38,7 +38,7 @@ export const MachineNonActionSidePanelViews = {
   ADD_MACHINE: ["machineNonActionForm", "addMachine"],
   ADD_SPECIAL_FILESYSTEM: ["machineNonActionForm", "addSpecialFilesystem"],
   ADD_VLAN: ["machineNonActionForm", "addVlan"],
-  CHANGE_STORAGE_LAYOUT: ["machineNonActionForm", "changeStorageLayout"],
+  APPLY_STORAGE_LAYOUT: ["machineNonActionForm", "applyStorageLayout"],
   CREATE_DATASTORE: ["machineNonActionForm", "createDatastore"],
   CREATE_RAID: ["machineNonActionForm", "createRaid"],
   CREATE_VOLUME_GROUP: ["machineNonActionForm", "createVolumeGroup"],

diff --git a/...ines/views/MachineDetails/MachineStorage/ChangeStorageLayout/ChangeStorageLayout.test.tsx b/...ines/views/MachineDetails/MachineStorage/ChangeStorageLayout/ChangeStorageLayout.test.tsx
@@ -86,6 +86,7 @@ describe("ChangeStorageLayout", () => {
   });
 
   it("correctly dispatches an action to update a machine's storage layout", async () => {
+    const handleClearSidePanelContent = vi.fn();
     const state = rootStateFactory({
       machine: machineStateFactory({
         items: [machineDetailsFactory({ system_id: "abc123" })],
@@ -97,7 +98,7 @@ describe("ChangeStorageLayout", () => {
     const store = mockStore(state);
     renderWithBrowserRouter(
       <ChangeStorageLayout
-        clearSidePanelContent={vi.fn()}
+        clearSidePanelContent={handleClearSidePanelContent}
         selectedLayout={sampleStoragelayout}
         systemId="abc123"
       />,
@@ -128,5 +129,6 @@ describe("ChangeStorageLayout", () => {
       },
       type: "machine/applyStorageLayout",
     });
+    expect(handleClearSidePanelContent).toHaveBeenCalled();
   });
 });
diff --git a/.../machines/views/MachineDetails/MachineStorage/ChangeStorageLayout/ChangeStorageLayout.tsx b/.../machines/views/MachineDetails/MachineStorage/ChangeStorageLayout/ChangeStorageLayout.tsx
@@ -49,6 +49,7 @@ export const ChangeStorageLayout = ({
             storageLayout: selectedLayout.value,
           })
         );
+        clearSidePanelContent();
       }}
       saved={saved}
       saving={saving}

diff --git a/...s/views/MachineDetails/MachineStorage/ChangeStorageLayoutMenu/ChangeStorageLayoutMenu.tsx b/...s/views/MachineDetails/MachineStorage/ChangeStorageLayoutMenu/ChangeStorageLayoutMenu.tsx
@@ -52,7 +52,7 @@ const ChangeStorageLayoutMenu = ({ systemId }: Props) => {
             children: option.label,
             onClick: () =>
               setSidePanelContent({
-                view: MachineSidePanelViews.CHANGE_STORAGE_LAYOUT,
+                view: MachineSidePanelViews.APPLY_STORAGE_LAYOUT,
                 extras: {
                   systemId,
                   selectedLayout: option,

diff --git a/src/app/store/utils/node/base.ts b/src/app/store/utils/node/base.ts
@@ -218,6 +218,8 @@ export const getSidePanelTitle = (
         return "Add VLAN";
       case SidePanelViews.CHANGE_SOURCE[1]:
         return "Change source";
+      case SidePanelViews.APPLY_STORAGE_LAYOUT[1]:
+        return "Change storage layout";
       case SidePanelViews.CLEAR_ALL_DISCOVERIES[1]:
         return "Clear all discoveries";
       case SidePanelViews.CREATE_DATASTORE[1]: