Legger til små forbedringer i kodekvalitet (#1323)

* Forbedrer variabel-navn og kommentar for alleRouteEnumMedlemmerGjennomgås * Forbedrer formProgressStegOppsummeringTekst til å bruke template literal for tekster
navikt · Aug 9, 2024 · 44fb530 · 44fb530
1 parent ac47c6f
commit 44fb530
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 12 deletions.
diff --git a/src/frontend/components/Felleskomponenter/Steg/Steg.tsx b/src/frontend/components/Felleskomponenter/Steg/Steg.tsx
@@ -167,14 +167,7 @@ const Steg: React.FC<ISteg> = ({ tittel, guide, skjema, gåVidereCallback, child
 
     const frittståendeOrdTekster = tekster().FELLES.frittståendeOrd;
 
-    const formProgressStegTekst =
-        plainTekst(frittståendeOrdTekster.steg) +
-        ' ' +
-        hentNåværendeStegIndex() +
-        ' ' +
-        plainTekst(frittståendeOrdTekster.av) +
-        ' ' +
-        formProgressSteg.length;
+    const formProgressStegOppsummeringTekst = `${plainTekst(frittståendeOrdTekster.steg)} ${hentNåværendeStegIndex()} ${plainTekst(frittståendeOrdTekster.av)} ${formProgressSteg.length}`;
 
     return (
         <>
@@ -199,7 +192,7 @@ const Steg: React.FC<ISteg> = ({ tittel, guide, skjema, gåVidereCallback, child
                             </div>
                             <FormProgress
                                 translations={{
-                                    step: formProgressStegTekst,
+                                    step: formProgressStegOppsummeringTekst,
                                     showAllSteps: plainTekst(frittståendeOrdTekster.visAlleSteg),
                                     hideAllSteps: plainTekst(frittståendeOrdTekster.skjulAlleSteg),
                                 }}

diff --git a/src/frontend/components/Felleskomponenter/Steg/useFormProgressSteg.tsx b/src/frontend/components/Felleskomponenter/Steg/useFormProgressSteg.tsx
@@ -84,9 +84,14 @@ export const useFormProgressSteg = (): IStegMedTittel[] => {
                     tittelBlock = KVITTERING.kvitteringTittel;
                     break;
                 default:
-                    // Alle routes i RouteEnum må gjennomgås i switch(), ellers feiler _exhaustiveCheck
-                    const _exhaustiveCheck: never = steg.route;
-                    return _exhaustiveCheck;
+                    /*
+                     * Det er viktig at alle enum-medlemmer i RouteEnum blir håndtert i switch-setningen.
+                     * Hvis et medlem utelates, vil koden under feile fordi den forutsetter at hver route har en tilhørende tittel fra Sanity.
+                     * Eslint vil fange opp en ubehandlet enum-verdi og kaste en feil, men dersom dette ikke korrigeres, kan det resultere i runtime-feil eller manglende tittel for enkelte steg.
+                     * Dette bidrar til å sikre at alle routes har en tilhørende titteltekst og at applikasjonen oppfører seg som forventet.
+                     */
+                    const alleRouteEnumMedlemmerGjennomgås: never = steg.route;
+                    return alleRouteEnumMedlemmerGjennomgås;
             }
 
             return {