fix: change storage layout sidepanel MAASENG-2798

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;
       }

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"],

src/app/machines/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();
   });
 });

src/app/machines/views/MachineDetails/MachineStorage/ChangeStorageLayout/ChangeStorageLayout.tsx

@@ -49,6 +49,7 @@ export const ChangeStorageLayout = ({
             storageLayout: selectedLayout.value,
           })
         );
+        clearSidePanelContent();
       }}
       saved={saved}
       saving={saving}

src/app/machines/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,

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]: