fix(data-contracts-web) handle other schedule types (datahub-project#…

…10919)
mayurinehate · Jul 16, 2024 · ee18a17 · ee18a17
1 parent ea2216a
commit ee18a17
Showing 1 changed file with 23 additions and 10 deletions.
diff --git a/...eact/src/app/entity/shared/tabs/Dataset/Validations/contract/FreshnessScheduleSummary.tsx b/...eact/src/app/entity/shared/tabs/Dataset/Validations/contract/FreshnessScheduleSummary.tsx
@@ -13,16 +13,29 @@ type Props = {
 };
 
 export const FreshnessScheduleSummary = ({ definition, evaluationSchedule }: Props) => {
-    const scheduleText =
-        definition.type === FreshnessAssertionScheduleType.Cron
-            ? `${capitalizeFirstLetter(cronstrue.toString(definition.cron?.cron as string))}.`
-            : `In the past ${
-                  definition.fixedInterval?.multiple
-              } ${definition.fixedInterval?.unit.toLocaleLowerCase()}s${
-                  (evaluationSchedule &&
-                      `, as of ${cronstrue.toString(evaluationSchedule.cron as string).toLowerCase()}`) ||
-                  ''
-              }`;
+    let scheduleText = '';
+    const cronStr = definition.cron?.cron ?? evaluationSchedule?.cron;
+    switch (definition.type) {
+        case FreshnessAssertionScheduleType.Cron:
+            scheduleText = cronStr
+                ? `${capitalizeFirstLetter(cronstrue.toString(cronStr))}.`
+                : `Unknown freshness schedule.`;
+            break;
+        case FreshnessAssertionScheduleType.SinceTheLastCheck:
+            scheduleText = cronStr
+                ? `Since the previous check, as of ${cronstrue.toString(cronStr).toLowerCase()}`
+                : 'Since the previous check';
+            break;
+        case FreshnessAssertionScheduleType.FixedInterval:
+            scheduleText = `In the past ${
+                definition.fixedInterval?.multiple
+            } ${definition.fixedInterval?.unit.toLocaleLowerCase()}s${
+                cronStr ? `, as of ${cronstrue.toString(cronStr).toLowerCase()}` : ''
+            }`;
+            break;
+        default:
+            break;
+    }
 
     return <>{scheduleText}</>;
 };