added test for removing shrink field in serialization and re-ordered …

…fields in hot phase advanced
elastic · Dec 9, 2020 · 3b09eb1 · 3b09eb1
1 parent 1f28720
commit 3b09eb1
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 3 deletions.
diff --git a/...ndex_lifecycle_management/__jest__/client_integration/edit_policy/edit_policy.helpers.tsx b/...ndex_lifecycle_management/__jest__/client_integration/edit_policy/edit_policy.helpers.tsx
@@ -192,7 +192,7 @@ export const setup = async (arg?: { appServicesContext: Partial<AppServicesConte
     await createFormSetValueAction(`${phase}-selectedPrimaryShardCount`)(value);
   };
 
-  const shrinkExists = () => exists('shrinkSwitch');
+  const shrinkExists = (phase: Phases) => () => exists(`${phase}-shrinkSwitch`);
 
   const setFreeze = createFormToggleAction('freezeSwitch');
   const freezeExists = () => exists('freezeSwitch');
@@ -238,6 +238,7 @@ export const setup = async (arg?: { appServicesContext: Partial<AppServicesConte
         ...createForceMergeActions('hot'),
         setIndexPriority: setIndexPriority('hot'),
         setShrink: setShrink('hot'),
+        shrinkExists: shrinkExists('hot'),
         ...createSearchableSnapshotActions('hot'),
       },
       warm: {
@@ -249,7 +250,7 @@ export const setup = async (arg?: { appServicesContext: Partial<AppServicesConte
         setSelectedNodeAttribute: setSelectedNodeAttribute('warm'),
         setReplicas: setReplicas('warm'),
         setShrink: setShrink('warm'),
-        shrinkExists,
+        shrinkExists: shrinkExists('warm'),
         ...createForceMergeActions('warm'),
         setIndexPriority: setIndexPriority('warm'),
       },

diff --git a/...agement/public/application/sections/edit_policy/components/phases/hot_phase/hot_phase.tsx b/...agement/public/application/sections/edit_policy/components/phases/hot_phase/hot_phase.tsx
@@ -236,8 +236,8 @@ export const HotPhase: FunctionComponent = () => {
         {isRolloverEnabled && (
           <>
             {<ForcemergeField phase="hot" />}
-            {license.canUseSearchableSnapshot() && <SearchableSnapshotField phase="hot" />}
             <ShrinkField phase="hot" />
+            {license.canUseSearchableSnapshot() && <SearchableSnapshotField phase="hot" />}
           </>
         )}
         <SetPriorityInputField phase={hotProperty} />

diff --git a/...nagement/public/application/sections/edit_policy/form/deserializer_and_serializer.test.ts b/...nagement/public/application/sections/edit_policy/form/deserializer_and_serializer.test.ts
@@ -290,4 +290,14 @@ describe('deserializer and serializer', () => {
       enabled: false,
     });
   });
+
+  it('removes shrink from hot and warm when unset', () => {
+    delete formInternal.phases.hot!.actions!.shrink;
+    delete formInternal.phases.warm!.actions!.shrink;
+
+    const result = serializer(formInternal);
+
+    expect(result.phases.hot!.actions.shrink).toBeUndefined();
+    expect(result.phases.warm!.actions.shrink).toBeUndefined();
+  });
 });
diff --git a/...ifecycle_management/public/application/sections/edit_policy/form/serializer/serializer.ts b/...ifecycle_management/public/application/sections/edit_policy/form/serializer/serializer.ts
@@ -69,6 +69,10 @@ export const createSerializer = (originalPolicy?: SerializedPolicy) => (
         delete hotPhaseActions.set_priority;
       }
 
+      if (!updatedPolicy.phases.hot?.actions?.shrink) {
+        delete hotPhaseActions.shrink;
+      }
+
       if (!updatedPolicy.phases.hot!.actions?.searchable_snapshot) {
         delete hotPhaseActions.searchable_snapshot;
       }