RI-7187 fix behavior after verification. popup was shown while loading was in progress (#4756)

Author: ArtemHoruzhenko

redisinsight/ui/src/pages/rdi/pipeline-management/components/source-pipeline-dialog/SourcePipelineModal.spec.tsx

@@ -139,4 +139,20 @@ describe('SourcePipelineDialog', () => {
 
     expect(screen.queryByTestId('file-source-pipeline-dialog')).not.toBeInTheDocument()
   })
+
+  it('should not show dialog when config is fetching', () => {
+    const sendEventTelemetryMock = jest.fn()
+    ;(sendEventTelemetry as jest.Mock).mockImplementation(
+      () => sendEventTelemetryMock,
+    )
+    ;(rdiPipelineSelector as jest.Mock).mockReturnValue({
+      ...initialStateDefault.rdi.pipeline,
+      loading: true,
+      config: '',
+    })
+
+    render(<SourcePipelineDialog />)
+
+    expect(screen.queryByTestId('file-source-pipeline-dialog')).not.toBeInTheDocument()
+  })
 })

redisinsight/ui/src/pages/rdi/pipeline-management/components/source-pipeline-dialog/SourcePipelineModal.tsx

@@ -1,4 +1,4 @@
-import React, { useEffect, useState } from 'react'
+import React, { useState } from 'react'
 import {
   EuiIcon,
   EuiModal,
@@ -40,8 +40,6 @@ export enum PipelineSourceOptions {
 
 const SourcePipelineDialog = () => {
   const [isShowDownloadDialog, setIsShowDownloadDialog] = useState(false)
-  const [hasRdiPipelineDeployed, setHasRdiPipelineDeployed] =
-    useState(false)
 
   const { rdiInstanceId } = useParams<{ rdiInstanceId: string }>()
 
@@ -50,10 +48,6 @@ const SourcePipelineDialog = () => {
   const { loading: pipelineLoading, config: pipelineConfig } =
     useSelector(rdiPipelineSelector)
 
-  useEffect(() => {
-    setHasRdiPipelineDeployed(!pipelineLoading && pipelineConfig?.length > 0)
-  }, [pipelineConfig, pipelineLoading])
-
   const dispatch = useDispatch()
 
   const onSelect = (option: PipelineSourceOptions) => {
@@ -106,7 +100,7 @@ const SourcePipelineDialog = () => {
     )
   }
 
-  if (!isOpenDialog || hasRdiPipelineDeployed) {
+  if (!isOpenDialog || pipelineConfig?.length > 0 || pipelineLoading) {
     return null
   }