From a22af3fd550c8c94988dfb403c581193bc4b5367 Mon Sep 17 00:00:00 2001
From: Sakshi Bobade <bobadesakshi21@gmail.com>
Date: Thu, 10 Oct 2024 12:18:39 +0530
Subject: [PATCH] Updating the completion status for the inifinispan

---
 .../components/stages/ReviewInstallation.tsx  |  2 +-
 src/renderer/components/stages/Vsam.tsx       | 40 ++++++++++---------
 2 files changed, 23 insertions(+), 19 deletions(-)

diff --git a/src/renderer/components/stages/ReviewInstallation.tsx b/src/renderer/components/stages/ReviewInstallation.tsx
index 47eb6aab..d351c43e 100644
--- a/src/renderer/components/stages/ReviewInstallation.tsx
+++ b/src/renderer/components/stages/ReviewInstallation.tsx
@@ -71,7 +71,7 @@ const ReviewInstallation = () => {
     const stageProgress = stageProgressStatus.every(status => status === true);
     const subStageProgress = subStageProgressStatus.every(status => status === true);
 
-    if (stageProgress && subStageProgress) {
+    if(stageProgress && subStageProgress) {
       updateProgress(true);
     } else {
       updateProgress(false);
diff --git a/src/renderer/components/stages/Vsam.tsx b/src/renderer/components/stages/Vsam.tsx
index 409a2286..bd18cc88 100644
--- a/src/renderer/components/stages/Vsam.tsx
+++ b/src/renderer/components/stages/Vsam.tsx
@@ -80,6 +80,12 @@ const Vsam = () => {
 
     (zowePaxVersion < 3) ? setStorageMode('VSAM') : setShowStorageModeOptions(true);
 
+    if(storageMode.toUpperCase() !== 'VSAM' && zowePaxVersion >= 3) {
+      dispatchActions(true);
+      setShowProgress(false);
+      return;
+    }
+
     let nextPosition;
     const stepProgress = getProgress('vsamStatus');
 
@@ -98,7 +104,7 @@ const Vsam = () => {
 
     if(stepProgress) {
       nextPosition = document.getElementById('vsam-progress');
-      nextPosition?.scrollIntoView({ behavior: 'smooth', block: 'end' });
+      nextPosition?.scrollIntoView({ behavior: 'smooth', block: 'center' });
     } else {
       nextPosition = document.getElementById('container-box-id');
       nextPosition?.scrollIntoView({behavior: 'smooth'});
@@ -111,10 +117,6 @@ const Vsam = () => {
       alertEmitter.emit('hideAlert');
       updateSubStepSkipStatus(SUB_STAGE_ID, stageStatusRef.current);
       dispatch(setActiveStep({ activeStepIndex: STAGE_ID, isSubStep: SUB_STAGES, activeSubStepIndex: SUB_STAGE_ID }));
-
-      if(storageMode.toUpperCase() != 'VSAM') {
-        dispatch(setVsamStatus(false));
-      }
     }
   }, []);
 
@@ -151,19 +153,23 @@ const Vsam = () => {
   }, [showProgress, stateUpdated]);
 
   useEffect(() => {
-    const allAttributesTrue = Object.values(vsamInitProgress).every(value => value === true);
-    if(allAttributesTrue) {
-      dispatchActions(true);
-      setShowProgress(initClicked || getProgress('vsamStatus'));
-    }
+    verifyAndSetVsamInitCompletion(vsamInitProgress);
   }, [vsamInitProgress]);
 
   const setVsamInitializationProgress = (vsamInitState: InitSubStepsState) => {
     setVsamInitProgress(vsamInitState);
     setVsamInitState(vsamInitState);
+    verifyAndSetVsamInitCompletion(vsamInitState);
+  }
+
+  const verifyAndSetVsamInitCompletion = (vsamInitState: InitSubStepsState, selectedStorageMode?: string) => {
     const allAttributesTrue = Object.values(vsamInitState).every(value => value === true);
     if(allAttributesTrue) {
       dispatchActions(true);
+      setShowProgress(initClicked || getProgress('vsamStatus'));
+    } else if (selectedStorageMode && selectedStorageMode.toUpperCase() == 'VSAM') {
+      dispatchActions(false);
+      setShowProgress(false);
     }
   }
 
@@ -302,7 +308,6 @@ const Vsam = () => {
     setAllowInitialization(regEx.test(dsName));
   }
 
-
   const cachingServiceChangeHandler = (newValue: string) => {
     alertEmitter.emit('hideAlert');
     datasetValidation(newValue);
@@ -326,14 +331,13 @@ const Vsam = () => {
       }
     };
     setYamlConfig(updatedYaml);
+
     if(event.target.value.toUpperCase() !== 'VSAM') {
-      Object.keys(vsamInitProgress).forEach(key => {
-        vsamInitProgress[key as keyof InitSubStepsState] = false;
-      });
-      setVsamInitializationProgress(vsamInitProgress);
+      dispatchActions(true);
+      setShowProgress(false);
+    } else {
+      verifyAndSetVsamInitCompletion(vsamInitProgress, event.target.value);
     }
-    setShowProgress(false);
-    dispatchActions(false);
   }
 
   return (
@@ -409,7 +413,7 @@ const Vsam = () => {
                 null
               }
 
-              <Box sx={{height: showProgress ? 'calc(100vh - 220px)' : 'auto'}} id="start-vsam-progress">
+              <Box sx={{height: showProgress ? 'calc(100vh - 70vh)' : 'auto'}} id="start-vsam-progress">
                 {!showProgress ? null :
                   <React.Fragment>
                     <ProgressCard label={`Write configuration file locally to temp directory`} id="init-vsam-progress-card" status={vsamInitProgress?.writeYaml}/>