🐛 optimize assessment fetching

konveyor · Feb 28, 2024 · 8b08063 · 8b08063
1 parent 59ad200
commit 8b08063
Show file tree

Hide file tree

Showing 4 changed files with 30 additions and 67 deletions.
diff --git a/...s/applications/components/application-assessment-status/application-assessment-status.tsx b/...s/applications/components/application-assessment-status/application-assessment-status.tsx
@@ -2,9 +2,9 @@ import React from "react";
 import { useTranslation } from "react-i18next";
 import { Spinner } from "@patternfly/react-core";
 import { EmptyTextMessage } from "@app/components/EmptyTextMessage";
-import { Application } from "@app/api/models";
+import { Application, Assessment } from "@app/api/models";
 import { IconedStatus, IconedStatusPreset } from "@app/components/IconedStatus";
-import { useFetchAssessmentsByItemId } from "@app/queries/assessments";
+import { useFetchAssessments } from "@app/queries/assessments";
 import { useFetchArchetypes } from "@app/queries/archetypes";
 interface ApplicationAssessmentStatusProps {
   application: Application;
@@ -17,6 +17,11 @@ export const ApplicationAssessmentStatus: React.FC<
   const { t } = useTranslation();
 
   const { archetypes, isFetching } = useFetchArchetypes();
+  const { assessments, fetchError } = useFetchAssessments();
+
+  const filteredAssessments = assessments?.filter(
+    (assessment: Assessment) => assessment.application?.id === application.id
+  );
 
   const applicationArchetypes = application.archetypes?.map((archetypeRef) => {
     return archetypes?.find((archetype) => archetype.id === archetypeRef.id);
@@ -30,17 +35,11 @@ export const ApplicationAssessmentStatus: React.FC<
       (archetype) => archetype?.assessments?.length ?? 0 > 0
     ) ?? false;
 
-  const {
-    assessments,
-    isFetching: isFetchingAssessmentsById,
-    fetchError,
-  } = useFetchAssessmentsByItemId(false, application.id);
-
   if (fetchError) {
     return <EmptyTextMessage message={t("terms.notAvailable")} />;
   }
 
-  if (isFetching || isFetchingAssessmentsById) {
+  if (isFetching || isFetching) {
     return <Spinner size="md" />;
   }
 
@@ -64,7 +63,7 @@ export const ApplicationAssessmentStatus: React.FC<
     statusPreset = "InProgressInheritedAssessments";
     tooltipCount = assessedArchetypeCount;
   } else if (
-    assessments?.some(
+    filteredAssessments?.some(
       (assessment) =>
         assessment.status === "started" || assessment.status === "complete"
     )

diff --git a/...ages/applications/components/application-detail-drawer/components/assessed-archetypes.tsx b/...ages/applications/components/application-detail-drawer/components/assessed-archetypes.tsx
@@ -1,10 +1,10 @@
 import React from "react";
-import { Application } from "@app/api/models";
+import { Application, AssessmentWithSectionOrder } from "@app/api/models";
 import { Label, LabelGroup, Spinner } from "@patternfly/react-core";
 import { EmptyTextMessage } from "@app/components/EmptyTextMessage";
 import { useTranslation } from "react-i18next";
 import { useFetchArchetypes } from "@app/queries/archetypes";
-import { useFetchAllAssessmentsWithArchetypes } from "@app/queries/assessments";
+import { useFetchAssessments } from "@app/queries/assessments";
 
 interface IAssessedArchetypesProps {
   application: Application | null;
@@ -14,35 +14,31 @@ export const AssessedArchetypes: React.FC<IAssessedArchetypesProps> = ({
   application,
 }) => {
   const { t } = useTranslation();
-  const {
-    archetypes: applicationArchetypes,
-    isFetching: isFetchingArchetypes,
-  } = useFetchArchetypes(application);
+  const { archetypes, isFetching: isFetchingArchetypes } =
+    useFetchArchetypes(application);
+  const { assessments, isFetching: isFetchingAssessments } =
+    useFetchAssessments();
 
-  const {
-    assessmentsWithArchetypes,
-    isLoading: isFetchingAllAssessmentsWithArchetypesLoading,
-  } = useFetchAllAssessmentsWithArchetypes(applicationArchetypes);
+  const assessedArchetypes = React.useMemo(() => {
+    if (!archetypes || !assessments) return [];
 
-  const assessedArchetypesWithARequiredAssessment = assessmentsWithArchetypes
-    ?.filter((assessmentsWithArchetype) => {
-      return (
-        assessmentsWithArchetype.archetype.assessed &&
-        assessmentsWithArchetype.assessments.some(
-          (assessment) => assessment?.required === true
-        )
-      );
-    })
-    .map((assessmentsWithArchetype) => assessmentsWithArchetype.archetype);
+    return archetypes.filter((archetype) =>
+      assessments.some(
+        (assessment: AssessmentWithSectionOrder) =>
+          assessment.archetype?.id === archetype.id && assessment.required
+      )
+    );
+  }, [archetypes, assessments]);
 
-  if (isFetchingArchetypes || isFetchingAllAssessmentsWithArchetypesLoading) {
+  if (isFetchingArchetypes || isFetchingAssessments) {
     return <Spinner size="md" />;
   }
+
   return (
     <LabelGroup>
-      {assessedArchetypesWithARequiredAssessment?.length ? (
-        assessedArchetypesWithARequiredAssessment?.map((archetype) => (
-          <Label key={archetype?.id}>{archetype?.name}</Label>
+      {assessedArchetypes.length ? (
+        assessedArchetypes.map((archetype) => (
+          <Label key={archetype.id}>{archetype.name}</Label>
         ))
       ) : (
         <EmptyTextMessage message={t("terms.none")} />

diff --git a/client/src/app/queries/archetypes.ts b/client/src/app/queries/archetypes.ts
@@ -27,7 +27,6 @@ export const useFetchArchetypes = (forApplication?: Application | null) => {
     initialData: [],
     queryKey: [ARCHETYPES_QUERY_KEY, forApplication?.id],
     queryFn: getArchetypes,
-    refetchInterval: 5000,
     onSuccess: (fetchedArchetypes) => {
       if (!forApplication) {
         setFilteredArchetypes(fetchedArchetypes);
@@ -42,6 +41,7 @@ export const useFetchArchetypes = (forApplication?: Application | null) => {
       } else {
         setFilteredArchetypes([]);
       }
+
       queryClient.invalidateQueries([reviewsQueryKey]);
       queryClient.invalidateQueries([assessmentsQueryKey]);
       queryClient.invalidateQueries([assessmentsByItemIdQueryKey]);

diff --git a/client/src/app/queries/assessments.ts b/client/src/app/queries/assessments.ts
@@ -17,11 +17,9 @@ import {
 } from "@app/api/rest";
 import { AxiosError } from "axios";
 import {
-  Archetype,
   Assessment,
   AssessmentWithArchetypeApplications,
   AssessmentWithSectionOrder,
-  AssessmentsWithArchetype,
   InitialAssessment,
 } from "@app/api/models";
 import { QuestionnairesQueryKey } from "./questionnaires";
@@ -264,33 +262,3 @@ export const useFetchAssessmentsWithArchetypeApplications = () => {
     isLoading: assessmentsLoading || isArchetypesLoading,
   };
 };
-
-export const useFetchAllAssessmentsWithArchetypes = (
-  archetypes: Archetype[] = []
-) => {
-  const assessmentQueries = useQueries({
-    queries: archetypes.map((archetype) => ({
-      queryKey: ["assessmentsForArchetype", archetype.id],
-      queryFn: () => getAssessmentsByItemId(true, archetype.id),
-    })),
-  });
-
-  const assessmentsWithArchetypes: AssessmentsWithArchetype[] =
-    assessmentQueries
-      .map((query, index) => {
-        if (query.isSuccess) {
-          return {
-            archetype: archetypes[index],
-            assessments: query.data,
-          };
-        }
-        return null;
-      })
-      .filter(Boolean);
-
-  return {
-    assessmentsWithArchetypes,
-    isLoading: assessmentQueries.some((query) => query.isLoading),
-    isError: assessmentQueries.some((query) => query.isError),
-  };
-};