fix: review comment

Author: Elessar1802

src/components/bulkEdits/BulkEdits.tsx

@@ -354,7 +354,7 @@ class BulkEdits extends Component<BulkEditsProps, BulkEditsState> {
                         />
                     </div>
                     <div
-                        className="bulk-output-body cn-9 fs-13 p-20 dc__overflow-auto flexbox-col flex-grow-1"
+                        className="cn-9 fs-13 p-20 dc__overflow-auto flexbox-col flex-grow-1"
                         data-testid="output-message"
                     >
                         {this.renderOutputContent()}
@@ -371,13 +371,25 @@ class BulkEdits extends Component<BulkEditsProps, BulkEditsState> {
             return <Progressing pageLoader />
         }
 
-        return outputResult ? (
-            <div>
+        if (!outputResult) {
+            return (
+                <GenericSectionErrorState
+                    useInfoIcon
+                    title="Nothing to show yet"
+                    subTitle="Enter a valid payload and click Run to see the operation result."
+                    description=""
+                    rootClassName="flex-grow-1"
+                />
+            )
+        }
+
+        return (
+            <div className="bulk-output-body">
                 {renderConfigMapOutput(outputResult.configMap)}
                 {renderDeploymentTemplateOutput(outputResult.deploymentTemplate)}
                 {renderSecretOutput(outputResult.secret)}
             </div>
-        ) : null
+        )
     }
 
     renderConfigMapImpObj = () => {
@@ -467,14 +479,24 @@ class BulkEdits extends Component<BulkEditsProps, BulkEditsState> {
             return <Progressing pageLoader />
         }
 
-        return impactedObjects ? (
-            <div>
+        if (!impactedObjects) {
+            return (
+                <GenericSectionErrorState
+                    useInfoIcon
+                    title="Nothing to show yet"
+                    subTitle="Enter a valid payload and click Show Impacted Objects to see which applications will be affected."
+                    description=""
+                    rootClassName="flex-grow-1"
+                />
+            )
+        }
+
+        return (
+            <div className="bulk-output-body">
                 {this.renderConfigMapImpObj()}
                 {this.renderDeploymentTemplateImpObj()}
                 {this.renderSecretImpObj()}
             </div>
-        ) : (
-            ''
         )
     }
 

src/components/bulkEdits/bulkedit.utils.tsx

@@ -52,7 +52,7 @@ export const renderConfigMapOutput = (configMap) => {
             <div>
                 #Message: <br />
                 <br />
-                {configMap?.message?.map((elm) => {
+                {configMap.message?.map((elm) => {
                     return (
                         <>
                             {elm}
@@ -67,11 +67,11 @@ export const renderConfigMapOutput = (configMap) => {
                 #Failed Operations:
                 <br />
                 <br />
-                {configMap?.failure == null ? (
+                {configMap.failure == null ? (
                     <>No Result Found</>
                 ) : (
                     <>
-                        {configMap?.failure.map((elm) => {
+                        {configMap.failure.map((elm) => {
                             return renderCmAndSecretResponseForOneApp(elm)
                         })}
                     </>
@@ -82,11 +82,11 @@ export const renderConfigMapOutput = (configMap) => {
             <div>
                 #Successful Operations: <br />
                 <br />
-                {configMap?.successful == null ? (
+                {configMap.successful == null ? (
                     <>No Result Found</>
                 ) : (
                     <>
-                        {configMap?.successful.map((elm) => {
+                        {configMap.successful.map((elm) => {
                             return renderCmAndSecretResponseForOneApp(elm)
                         })}
                     </>
@@ -133,7 +133,7 @@ export const renderDeploymentTemplateOutput = (deploymentTemplate) => {
             <div>
                 #Message: <br />
                 <br />
-                {deploymentTemplate?.message?.map((elm) => {
+                {deploymentTemplate.message?.map((elm) => {
                     return (
                         <div>
                             {elm}
@@ -148,11 +148,11 @@ export const renderDeploymentTemplateOutput = (deploymentTemplate) => {
                 #Failed Operations:
                 <br />
                 <br />
-                {deploymentTemplate?.failure === null ? (
+                {deploymentTemplate.failure === null ? (
                     <>No Result Found</>
                 ) : (
                     <>
-                        {deploymentTemplate?.failure.map((elm) => {
+                        {deploymentTemplate.failure.map((elm) => {
                             return renderDTResponseForOneApp(elm)
                         })}
                     </>
@@ -164,11 +164,11 @@ export const renderDeploymentTemplateOutput = (deploymentTemplate) => {
             <div>
                 #Successful Operations: <br />
                 <br />
-                {deploymentTemplate?.successful == null ? (
+                {deploymentTemplate.successful == null ? (
                     <>No Result Found</>
                 ) : (
                     <>
-                        {deploymentTemplate?.successful.map((elm) => {
+                        {deploymentTemplate.successful.map((elm) => {
                             return renderDTResponseForOneApp(elm)
                         })}
                     </>
@@ -189,7 +189,7 @@ export const renderSecretOutput = (secret) => {
             <div>
                 #Message: <br />
                 <br />
-                {secret?.message?.map((elm) => {
+                {secret.message?.map((elm) => {
                     return (
                         <>
                             {elm}
@@ -204,11 +204,11 @@ export const renderSecretOutput = (secret) => {
                 #Failed Operations:
                 <br />
                 <br />
-                {secret?.failure == null ? (
+                {secret.failure == null ? (
                     <>No Result Found</>
                 ) : (
                     <>
-                        {secret?.failure.map((elm) => {
+                        {secret.failure.map((elm) => {
                             return renderCmAndSecretResponseForOneApp(elm)
                         })}
                     </>
@@ -220,11 +220,11 @@ export const renderSecretOutput = (secret) => {
             <div>
                 #Successful Operations: <br />
                 <br />
-                {secret?.successful == null ? (
+                {secret.successful == null ? (
                     <>No Result Found</>
                 ) : (
                     <>
-                        {secret?.successful.map((elm) => {
+                        {secret.successful.map((elm) => {
                             return renderCmAndSecretResponseForOneApp(elm)
                         })}
                     </>