🐛 Accurately represent upload state in binary upload step (#1487)

- Address https://issues.redhat.com/browse/MTA-1420 
- Removes bg polling while wizard is open 
- Adds a check for ongoing mutations while the file upload operation is
underway.

Signed-off-by: ibolton336 <ibolton@redhat.com>

client/src/app/pages/applications/analysis-wizard/analysis-wizard.tsx

@@ -340,10 +340,9 @@ export const AnalysisWizard: React.FC<IAnalysisWizard> = ({
         <WizardStep
           id={StepId.AnalysisMode}
           name={t("wizard.terms.analysisMode")}
-          isDisabled={!isStepEnabled(StepId.AnalysisMode)}
           footer={{
             isNextDisabled:
-              !isMutating && !isStepEnabled(StepId.AnalysisMode + 1),
+              !!isMutating || !isStepEnabled(StepId.AnalysisMode + 1),
           }}
         >
           <>

client/src/app/pages/applications/analysis-wizard/components/upload-binary.tsx

@@ -46,6 +46,8 @@ export const UploadBinary: React.FC = () => {
       title: "Uploaded binary file.",
       variant: "success",
     });
+    setFileUploadStatus("success");
+    setFileUploadProgress(100);
   };
 
   const failedUpload = (error: AxiosError) => {
@@ -82,10 +84,8 @@ export const UploadBinary: React.FC = () => {
     failedUpload
   );
 
-  const { mutate: removeFile } = useRemoveUploadedFileMutation(
-    completedRemove,
-    failedRemove
-  );
+  const { mutate: removeFile, isLoading: isRemovingFile } =
+    useRemoveUploadedFileMutation(completedRemove, failedRemove);
   const onCreateTaskgroupSuccess = (data: Taskgroup) => {
     updateTaskGroup(data);
   };
@@ -133,9 +133,8 @@ export const UploadBinary: React.FC = () => {
       readFile(droppedFiles[0])
         .then((data) => {
           if (data) {
-            setFileUploadProgress(100);
-            setFileUploadStatus("success");
             setValue("artifact", droppedFiles[0]);
+            setFileUploadProgress(0);
           }
         })
         .catch((error) => {

client/src/app/pages/applications/applications-table-analyze/applications-table-analyze.tsx

@@ -68,7 +68,6 @@ import {
   useFetchApplications,
 } from "@app/queries/applications";
 import { useCancelTaskMutation, useFetchTasks } from "@app/queries/tasks";
-import { useFetchReviews } from "@app/queries/reviews";
 import { useFetchIdentities } from "@app/queries/identities";
 import { useFetchTagCategories } from "@app/queries/tags";
 
@@ -116,7 +115,7 @@ export const ApplicationsTableAnalyze: React.FC = () => {
   const getTask = (application: Application) =>
     tasks.find((task: Task) => task.application?.id === application.id);
 
-  const { tasks } = useFetchTasks({ addon: "analyzer" });
+  const { tasks } = useFetchTasks({ addon: "analyzer" }, isAnalyzeModalOpen);
 
   const { tagCategories: tagCategories } = useFetchTagCategories();
 
@@ -127,7 +126,7 @@ export const ApplicationsTableAnalyze: React.FC = () => {
     isFetching: isFetchingApplications,
     error: applicationsFetchError,
     refetch: fetchApplications,
-  } = useFetchApplications();
+  } = useFetchApplications(isAnalyzeModalOpen);
 
   const onDeleteApplicationSuccess = (appIDCount: number) => {
     pushNotification({
@@ -350,8 +349,6 @@ export const ApplicationsTableAnalyze: React.FC = () => {
     selectionState: { selectedItems: selectedRows },
   } = tableControls;
 
-  const { reviews } = useFetchReviews();
-
   const [
     saveApplicationsCredentialsModalState,
     setSaveApplicationsCredentialsModalState,

client/src/app/queries/applications.ts

@@ -28,13 +28,13 @@ interface DownloadOptions {
   mimeType: MimeType;
 }
 
-export const useFetchApplications = () => {
+export const useFetchApplications = (refetchDisabled: boolean = false) => {
   const queryClient = useQueryClient();
   const { isLoading, error, refetch, data } = useQuery({
     initialData: [],
     queryKey: [ApplicationsQueryKey],
     queryFn: getApplications,
-    refetchInterval: 5000,
+    refetchInterval: !refetchDisabled ? 5000 : false,
     onSuccess: () => {
       queryClient.invalidateQueries([reviewsQueryKey]);
       queryClient.invalidateQueries([assessmentsQueryKey]);

client/src/app/queries/tasks.ts

@@ -8,11 +8,14 @@ interface FetchTasksFilters {
 
 export const TasksQueryKey = "tasks";
 
-export const useFetchTasks = (filters: FetchTasksFilters = {}) => {
+export const useFetchTasks = (
+  filters: FetchTasksFilters = {},
+  refetchDisabled: boolean = false
+) => {
   const { isLoading, error, refetch, data } = useQuery({
     queryKey: [TasksQueryKey],
     queryFn: getTasks,
-    refetchInterval: 5000,
+    refetchInterval: !refetchDisabled ? 5000 : false,
     select: (allTasks) => {
       const uniqSorted = allTasks
         .filter((task) =>