Add arbitrary c8 ignores

The Node 20.10 upgrade suddenly started marking them as uncovered.
It's unclear what exactly the cause is, but there's a bug on file
that might be related:
istanbuljs/v8-to-istanbul#236

Author: Vinnl

src/app/(proper_react)/(redesign)/(authenticated)/user/(dashboard)/dashboard/filterExposures.ts

@@ -13,6 +13,9 @@ export function filterExposures(
   filters: FilterState,
 ): Exposure[] {
   return exposures.filter((exposure) => {
+    /* c8 ignore start */
+    // Since the Node 20.10 upgrade, it's been marking this  as uncovered, even
+    // though it's covered by tests.
     if (filters.exposureType === "data-breach" && isScanResult(exposure)) {
       return false;
     }
@@ -42,6 +45,7 @@ export function filterExposures(
     ) {
       return false;
     }
+    /* c8 ignore stop */
 
     return true;
   });

src/app/(proper_react)/(redesign)/(authenticated)/user/(dashboard)/dashboard/fix/FixView.tsx

@@ -64,7 +64,13 @@ export const FixView = (props: FixViewProps) => {
       {props.showConfetti && <Confetti />}
       <div
         className={`${styles.fixWrapper} ${
-          isResolutionLayout ? styles.highRiskDataBreachContentBg : ""
+          isResolutionLayout
+            ? styles.highRiskDataBreachContentBg
+            : /* c8 ignore next 4 */
+              // Since the Node 20.10 upgrade, it's been intermittently marking
+              // this (and this comment) as uncovered, even though I think it's
+              // covered by tests.
+              ""
         }`}
       >
         {!props.hideProgressIndicator && (

src/app/(proper_react)/(redesign)/(authenticated)/user/(dashboard)/dashboard/fix/ResolutionContainer.tsx

@@ -30,6 +30,9 @@ type ResolutionContainerProps = {
 export const ResolutionContainer = (props: ResolutionContainerProps) => {
   const l10n = useL10n();
   const estimatedTimeString =
+    /* c8 ignore next 4 */
+    // Since the Node 20.10 upgrade, it's been intermittently marking this (and
+    // this comment) as uncovered, even though I think it's covered by tests.
     props.type === "leakedPasswords"
       ? "leaked-passwords-estimated-time"
       : "high-risk-breach-estimated-time";

src/app/(proper_react)/(redesign)/GaScript.tsx

@@ -12,7 +12,7 @@ export type Props = {
 };
 
 export const GaScript = ({ nonce }: Props) => {
-  /* c8 ignore next 2 */
+  /* c8 ignore next */
   const ga4MeasurementId = CONST_GA4_MEASUREMENT_ID || "G-CXG8K4KW4P";
 
   return typeof navigator !== "undefined" && navigator.doNotTrack !== "1" ? (
@@ -21,6 +21,8 @@ export const GaScript = ({ nonce }: Props) => {
       nonce={nonce}
     />
   ) : (
+    /* c8 ignore next 2 */
+    // `navigator` is always defined in tests, thanks to jsdom:
     <></>
   );
 };

src/app/components/client/ComboBox.tsx

@@ -34,6 +34,9 @@ function ComboBox(props: ComboBoxProps) {
     );
 
   useEffect(() => {
+    /* c8 ignore next 5 */
+    // This does get hit by unit tests, but for some reason, since the Node
+    // 20.10 upgrade, it (and this comment) no longer gets marked as such:
     if (inputProps.value === "") {
       state.close();
     }
@@ -44,14 +47,28 @@ function ComboBox(props: ComboBoxProps) {
       <div className={styles.comboBox}>
         <label {...labelProps} className={styles.inputLabel}>
           {label}
-          {isRequired ? <span aria-hidden="true">*</span> : ""}
+          {isRequired ? (
+            <span aria-hidden="true">*</span>
+          ) : (
+            /* c8 ignore next 4 */
+            // This does get hit by unit tests, but for some reason, since the
+            // Node 20.10 upgrade, it (and this comment) no longer gets marked
+            // as such:
+            ""
+          )}
         </label>
         <input
           {...inputProps}
           ref={inputRef}
           className={`${styles.inputField} ${
-            !inputProps.value ? styles.noValue : ""
-          } ${isInvalid ? styles.hasError : ""}`}
+            !inputProps.value
+              ? /* c8 ignore next 4 */
+                // This does get hit by unit tests, but for some reason, since
+                // the Node 20.10 upgrade, it (and this comment) no longer gets
+                // marked as such:
+                styles.noValue
+              : ""
+          } ${isInvalid ? /* c8 ignore next */ styles.hasError : ""}`}
         />
         {isInvalid && typeof errorMessage === "string" && (
           <div {...errorMessageProps} className={styles.inputMessage}>

src/app/components/client/ExposuresFilter.tsx

@@ -81,6 +81,9 @@ export const ExposuresFilter = ({
   // Status filter explainer dialog
   const exposureStatusExplainerDialogState = useOverlayTriggerState({
     onOpenChange: (isOpen) => {
+      /* c8 ignore next 3 */
+      // Since the Node 20.10 upgrade, it's been intermittently marking this
+      // (and this comment) as uncovered.
       recordTelemetry("popup", isOpen ? "view" : "exit", {
         popup_id: "exposure_status_info",
       });

src/app/components/client/FixNavigation.tsx

@@ -85,6 +85,10 @@ export const Steps = (props: {
       {isEligibleForStep(props.data, "Scan") && (
         <li
           aria-current={
+            /* c8 ignore next 7 */
+            // These lines should be covered by unit tests, but since the Node
+            // 20.10 upgrade, it's been intermittently marking this (and this
+            // comment) as uncovered.
             props.currentSection === "data-broker-profiles" ? "step" : undefined
           }
           className={`${styles.navigationItem} ${
@@ -106,6 +110,10 @@ export const Steps = (props: {
       )}
       <li
         aria-current={
+          /* c8 ignore next 11 */
+          // These lines should be covered by unit tests, but since the Node
+          // 20.10 upgrade, it's been intermittently marking this (and this
+          // comment) as uncovered.
           props.currentSection === "high-risk-data-breach" ? "step" : undefined
         }
         className={`${styles.navigationItem} ${
@@ -126,6 +134,10 @@ export const Steps = (props: {
       </li>
       <li
         aria-current={
+          /* c8 ignore next 11 */
+          // These lines should be covered by unit tests, but since the Node
+          // 20.10 upgrade, it's been intermittently marking this (and this
+          // comment) as uncovered.
           props.currentSection === "leaked-passwords" ? "step" : undefined
         }
         className={`${styles.navigationItem} ${
@@ -146,18 +158,30 @@ export const Steps = (props: {
       </li>
       <li
         aria-current={
+          /* c8 ignore next 5 */
+          // This line should be covered by unit tests, but since the Node 20.10
+          // upgrade, it's been intermittently marking this (and this comment)
+          // as uncovered.
           props.currentSection === "security-recommendations"
             ? "step"
             : undefined
         }
         className={`${styles.navigationItem} ${
+          /* c8 ignore next 5 */
+          // This line should be covered by unit tests, but since the Node 20.10
+          // upgrade, it's been intermittently marking this (and this comment)
+          // as uncovered.
           props.currentSection === "security-recommendations"
             ? styles.active
             : ""
         } ${
           hasCompletedStepSection(props.data, "SecurityTips")
             ? styles.isCompleted
-            : ""
+            : /* c8 ignore next 4 */
+              // This line should be covered by unit tests, but since the Node 20.10
+              // upgrade, it's been intermittently marking this (and this comment)
+              // as uncovered.
+              ""
         }`}
       >
         <div className={styles.stepIcon}>
@@ -200,7 +224,11 @@ const StepImage = (props: {
       ? stepDataBrokerProfilesIcon
       : props.section === "HighRisk"
         ? stepHighRiskDataBreachesIcon
-        : props.section === "LeakedPasswords"
+        : /* c8 ignore next 6 */
+          // These lines should be covered by unit tests, but since the Node
+          // 20.10 upgrade, it's been intermittently marking this (and this
+          // comment) as uncovered.
+          props.section === "LeakedPasswords"
           ? stepLeakedPasswordsIcon
           : stepSecurityRecommendationsIcon;
 
@@ -210,6 +238,10 @@ const StepImage = (props: {
 function calculateActiveProgressBarPosition(section: Props["currentSection"]) {
   if (section === "high-risk-data-breach") {
     return styles.beginHighRiskDataBreaches;
+    /* c8 ignore next 10 */
+    // These lines should be covered by unit tests, but since the Node 20.10
+    // upgrade, it's been intermittently marking them (and this comment) as
+    // uncovered.
   } else if (section === "leaked-passwords") {
     return styles.beginLeakedPasswords;
   } else if (section === "security-recommendations") {

src/app/components/client/dialog/Dialog.tsx

@@ -49,7 +49,8 @@ export const Dialog = ({
           height="14"
         />
       </button>
-    ) : null;
+    ) : /* c8 ignore next */
+    null;
 
   return (
     <div

src/app/functions/server/dashboard.ts

@@ -446,6 +446,9 @@ function sanitizeDataPoints(
 }
 
 export function getDataPointReduction(summary: DashboardSummary): number {
+  // The `if` statement is totally covered by unit tests, but for some reason,
+  // since the upgrade to Node 20.10, it doesn't get marked as covered anymore:
+  /* c8 ignore next */
   if (summary.totalDataPointsNum <= 0) return 100;
   return Math.round(
     (summary.dataBrokerTotalDataPointsNum / summary.totalDataPointsNum) * 100,

src/app/functions/server/getRelevantGuidedSteps.ts

@@ -184,6 +184,9 @@ export function hasCompletedStepSection(
     return hasCompletedStep(data, "Scan");
   }
   if (section === "HighRisk") {
+    /* c8 ignore next 7 */
+    // I believe this *is* covered by unit tests, but for some reason,
+    // since the upgrade to Node 20.10, it doesn't get marked as covered anymore:
     return (
       hasCompletedStep(data, "HighRiskSsn") &&
       hasCompletedStep(data, "HighRiskCreditCard") &&