ADM-902: [frontend] fix: fix unit test

au-heartbeat · Apr 16, 2024 · deccf87 · deccf87
1 parent a19e5ae
commit deccf87
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 12 deletions.
diff --git a/frontend/src/containers/MetricsStep/DeploymentFrequencySettings/index.tsx b/frontend/src/containers/MetricsStep/DeploymentFrequencySettings/index.tsx
@@ -44,6 +44,8 @@ export const DeploymentFrequencySettings = () => {
   };
 
   const totalPipelineNumber = realDeploymentFrequencySettings.length;
+  const shouldShowCrews =
+    loadingCompletedNumber !== 0 && totalPipelineNumber !== 0 && loadingCompletedNumber === totalPipelineNumber;
 
   return (
     <>
@@ -73,16 +75,14 @@ export const DeploymentFrequencySettings = () => {
             />
           ))}
           <AddButton onClick={handleAddPipeline} text={'New Pipeline'} />
-          {loadingCompletedNumber !== 0 &&
-            totalPipelineNumber !== 0 &&
-            loadingCompletedNumber === totalPipelineNumber && (
-              <Crews
-                options={pipelineCrews}
-                title={'Crew setting (optional)'}
-                label={'Included Crews'}
-                type={'pipeline'}
-              />
-            )}
+          {shouldShowCrews && (
+            <Crews
+              options={pipelineCrews}
+              title={'Crew setting (optional)'}
+              label={'Included Crews'}
+              type={'pipeline'}
+            />
+          )}
         </>
       )}
     </>

diff --git a/frontend/src/context/Metrics/metricsSlice.ts b/frontend/src/context/Metrics/metricsSlice.ts
@@ -515,7 +515,7 @@ export const metricsSlice = createSlice({
                   : '';
                 return {
                   id,
-                  isStepSelected: isStepSelected ? isStepSelected : false,
+                  isStepSelected: isStepSelected || false,
                   organization: matchedOrganization,
                   pipelineName: matchedPipelineName,
                   step: matchedPipelineName ? step : '',
@@ -597,7 +597,7 @@ export const metricsSlice = createSlice({
           return pipeline?.id === id
             ? {
                 ...pipeline,
-                step: stepWarningMessage(selectedPipelineStep, matchedPipeline.isStepSelected),
+                step: stepWarningMessage(selectedPipelineStep, matchedPipeline?.isStepSelected || false),
               }
             : pipeline;
         });