Remove comment out test linter

mozilla · Dec 16, 2024 · 68575d2 · 68575d2
1 parent c912c6a
commit 68575d2
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 6 deletions.
diff --git a/src/app/(proper_react)/(redesign)/(authenticated)/user/(dashboard)/dashboard/View.tsx b/src/app/(proper_react)/(redesign)/(authenticated)/user/(dashboard)/dashboard/View.tsx
@@ -179,7 +179,7 @@ export const View = (props: Props) => {
       const exposureStatus = getExposureStatus(
         exposure,
         props.enabledFeatureFlags.includes("AdditionalRemovalStatuses"),
-        // TODO: Waiting for criteria for data brokers under maintenace to be determined
+        // TODO: Waiting for criteria for data brokers under maintenance to be determined
         // isDataBrokerUnderMaintenance(exposure),
       );
 

diff --git a/src/app/components/client/exposure_card/ExposureCard.test.tsx b/src/app/components/client/exposure_card/ExposureCard.test.tsx
@@ -78,6 +78,7 @@ describe("ScanResultCard", () => {
 
   // TODO: Waiting for criteria for data brokers under maintenace to be determined
   // Data broker removal under maintenance
+  // eslint-disable-next-line jest/no-commented-out-tests
   // it("shows the right description for a scan result card with removal under maintenance status", () => {
   //   const ComposedProgressCard = composeStory(
   //     DataBrokerRemovalUnderMaintenance,
@@ -94,6 +95,7 @@ describe("ScanResultCard", () => {
 
   // TODO: Waiting for criteria for data brokers under maintenace to be determined
   // Data broker removal under maintenance resolved
+  // eslint-disable-next-line jest/no-commented-out-tests
   // it("shows the right description for a scan result card with removal under maintenance status that's been resolved", () => {
   //   const ComposedProgressCard = composeStory(
   //     DataBrokerRemovalUnderMaintenanceFixed,
@@ -122,6 +124,7 @@ describe("ScanResultCard", () => {
 
   // TODO: Waiting for criteria for data brokers under maintenace to be determined
   // Data broker removal under maintenance
+  // eslint-disable-next-line jest/no-commented-out-tests
   // it("shows the right description for a scan result card where removal is under maintenance", () => {
   //   const ComposedProgressCard = composeStory(
   //     DataBrokerRemovalUnderMaintenance,

diff --git a/src/app/components/server/StatusPill.tsx b/src/app/components/server/StatusPill.tsx
@@ -44,7 +44,7 @@ export const StatusPill = (props: Props) => {
         props.exposure,
         props.enabledFeatureFlags?.includes("AdditionalRemovalStatuses") ??
           false,
-        // TODO: Waiting for criteria for data brokers under maintenace to be determined
+        // TODO: Waiting for criteria for data brokers under maintenance to be determined
         // props.isRemovalUnderMaintenance || false,
       );
 
@@ -56,7 +56,7 @@ export const StatusPill = (props: Props) => {
               exposure: props.exposure,
               pillType,
               l10n,
-              // TODO: Waiting for criteria for data brokers under maintenace to be determined
+              // TODO: Waiting for criteria for data brokers under maintenance to be determined
               // isDataBrokerUnderMaintenance: props.isRemovalUnderMaintenance,
             })
           : getStatusLabel({
@@ -119,6 +119,7 @@ export const getExposureStatus = (
       return "fixed";
     }
 
+    // TODO: Waiting for criteria for data brokers under maintenance to be determined
     // if (isRemovalUnderMaintenance) {
     //   return "actionNeeded";
     // }

diff --git a/src/app/functions/server/dashboard.ts b/src/app/functions/server/dashboard.ts
@@ -204,12 +204,12 @@ export function getDashboardSummary(
           r.status === RemovalStatusMap.WaitingForVerification) &&
         !isManuallyResolved;
 
-      // TODO: Waiting for criteria for data brokers under maintenace to be determined
+      // TODO: Waiting for criteria for data brokers under maintenance to be determined
       // const isRemovalUnderMaintenance =
       //   r.broker_status === DataBrokerRemovalStatusMap.RemovalUnderMaintenance;
 
       if (isInProgress) {
-        // TODO: Waiting for criteria for data brokers under maintenace to be determined
+        // TODO: Waiting for criteria for data brokers under maintenance to be determined
         // if (!isRemovalUnderMaintenance) {
         summary.dataBrokerInProgressNum++;
         // }
@@ -234,7 +234,7 @@ export function getDashboardSummary(
       summary.allDataPoints.familyMembers += r.relatives.length;
 
       if (isInProgress) {
-        // TODO: Waiting for criteria for data brokers under maintenace to be determined
+        // TODO: Waiting for criteria for data brokers under maintenance to be determined
         // if (!isRemovalUnderMaintenance) {
         summary.inProgressDataPoints.emailAddresses += r.emails.length;
         summary.inProgressDataPoints.phoneNumbers += r.phones.length;

diff --git a/src/app/functions/server/getRelevantGuidedSteps.test.ts b/src/app/functions/server/getRelevantGuidedSteps.test.ts
@@ -562,6 +562,7 @@ describe("getNextGuidedStep", () => {
     });
 
     // TODO: Waiting for criteria for data brokers under maintenace to be determined
+    // eslint-disable-next-line jest/no-commented-out-tests
     // it("links to the removal under maintenance step if a user has scan resutls with a data broker that has a removal under maintenance status", () => {
     //   expect(
     //     getNextGuidedStep({