🐛 Address in progress inherited status gap (#1652)

<img width="1323" alt="Screenshot 2024-01-03 at 5 43 10 PM" src="https://github.com/konveyor/tackle2-ui/assets/11218376/d4311c95-341b-4e59-9280-361139d5e878"> <img width="1420" alt="Screenshot 2024-01-03 at 5 42 58 PM" src="https://github.com/konveyor/tackle2-ui/assets/11218376/c5191eab-1436-4500-a8f2-69b87595272b"> <img width="1423" alt="Screenshot 2024-01-03 at 5 42 54 PM" src="https://github.com/konveyor/tackle2-ui/assets/11218376/ab4d077e-0432-4e92-be1f-be707eebd861"> https://issues.redhat.com/browse/MTA-1878 https://issues.redhat.com/browse/MTA-1954 Signed-off-by: ibolton336 <ibolton@redhat.com>
konveyor · Jan 4, 2024 · cc40cb8 · cc40cb8
1 parent 34ffd17
commit cc40cb8
Show file tree

Hide file tree

Showing 3 changed files with 56 additions and 8 deletions.
diff --git a/client/src/app/components/IconedStatus.tsx b/client/src/app/components/IconedStatus.tsx
@@ -10,6 +10,8 @@ import TopologyIcon from "@patternfly/react-icons/dist/esm/icons/topology-icon";
 
 export type IconedStatusPreset =
   | "InheritedReviews"
+  | "InProgressInheritedReviews"
+  | "InProgressInheritedAssessments"
   | "InheritedAssessments"
   | "Canceled"
   | "Completed"
@@ -52,6 +54,22 @@ export const IconedStatus: React.FC<IIconedStatusProps> = ({
 }: IIconedStatusProps) => {
   const { t } = useTranslation();
   const presets: IconedStatusPresetType = {
+    InProgressInheritedReviews: {
+      icon: <InProgressIcon />,
+      status: "info",
+      label: t("terms.inProgress"),
+      tooltipMessage: t("message.inheritedReviewTooltip", {
+        count: tooltipCount,
+      }),
+    },
+    InProgressInheritedAssessments: {
+      icon: <InProgressIcon />,
+      status: "info",
+      label: t("terms.inProgress"),
+      tooltipMessage: t("message.inheritedAssessmentTooltip", {
+        count: tooltipCount,
+      }),
+    },
     InheritedReviews: {
       icon: <CheckCircleIcon />,
       status: "success",
@@ -131,6 +149,15 @@ export const IconedStatus: React.FC<IIconedStatusProps> = ({
         return t("message.inheritedAssessmentTooltip", {
           count: tooltipCount,
         });
+      case "InProgressInheritedReviews":
+        return t("message.inheritedReviewTooltip", {
+          count: tooltipCount,
+        });
+      case "InProgressInheritedAssessments":
+        return t("message.inheritedAssessmentTooltip", {
+          count: tooltipCount,
+        });
+
       default:
         return "";
     }
@@ -149,7 +176,10 @@ export const IconedStatus: React.FC<IIconedStatusProps> = ({
         </IconWithOptionalTooltip>
       </FlexItem>
       <FlexItem>{label || presetProps?.label}</FlexItem>
-      {(preset === "InheritedReviews" || preset === "InheritedAssessments") && (
+      {(preset === "InheritedReviews" ||
+        preset === "InheritedAssessments" ||
+        preset === "InProgressInheritedAssessments" ||
+        preset === "InProgressInheritedReviews") && (
         <FlexItem>
           <Tooltip content={getTooltipContent()}>
             <TopologyIcon />

diff --git a/...s/applications/components/application-assessment-status/application-assessment-status.tsx b/...s/applications/components/application-assessment-status/application-assessment-status.tsx
@@ -22,9 +22,13 @@ export const ApplicationAssessmentStatus: React.FC<
     return archetypes?.find((archetype) => archetype.id === archetypeRef.id);
   });
 
-  const hasAssessedArchetype = applicationArchetypes?.some(
+  const someArchetypesAssessed = applicationArchetypes?.some(
     (archetype) => !!archetype?.assessments?.length ?? 0 > 0
   );
+  const areAllArchetypesAssessed =
+    applicationArchetypes?.every(
+      (archetype) => archetype?.assessments?.length ?? 0 > 0
+    ) ?? false;
 
   const {
     assessments,
@@ -42,16 +46,22 @@ export const ApplicationAssessmentStatus: React.FC<
 
   let statusPreset: IconedStatusPreset = "NotStarted"; // Default status
   let tooltipCount: number = 0;
+
+  const assessedArchetypeCount =
+    applicationArchetypes?.filter(
+      (archetype) => archetype?.assessments?.length ?? 0 > 0
+    ).length || 0;
+
   const isDirectlyAssessed =
     application.assessed && (application.assessments?.length ?? 0) > 0;
+
   if (isDirectlyAssessed) {
     statusPreset = "Completed";
-  } else if (hasAssessedArchetype) {
+  } else if (areAllArchetypesAssessed) {
     statusPreset = "InheritedAssessments";
-    const assessedArchetypeCount =
-      applicationArchetypes?.filter(
-        (archetype) => archetype?.assessments?.length ?? 0 > 0
-      ).length || 0;
+    tooltipCount = assessedArchetypeCount;
+  } else if (someArchetypesAssessed) {
+    statusPreset = "InProgressInheritedAssessments";
     tooltipCount = assessedArchetypeCount;
   } else if (
     assessments?.some(

diff --git a/...app/pages/applications/components/application-review-status/application-review-status.tsx b/...app/pages/applications/components/application-review-status/application-review-status.tsx
@@ -38,10 +38,18 @@ export const ApplicationReviewStatus: React.FC<
   let statusPreset: IconedStatusPreset;
   let tooltipCount = 0;
 
+  const areAllArchetypesReviewed =
+    applicationArchetypes?.every((archetype) => !!archetype?.review) ?? false;
+  const someArchetypesReviewed =
+    applicationArchetypes?.some((archetype) => !!archetype?.review) ?? false;
+
   if (isAppReviewed) {
     statusPreset = "Completed";
-  } else if (reviewedArchetypeCount > 0) {
+  } else if (areAllArchetypesReviewed) {
+    tooltipCount = reviewedArchetypeCount;
     statusPreset = "InheritedReviews";
+  } else if (someArchetypesReviewed) {
+    statusPreset = "InProgressInheritedReviews";
     tooltipCount = reviewedArchetypeCount;
   } else {
     statusPreset = "NotStarted";