Better tests and some refactor

Author: tu55eladd

src/hovedside/Hovedside.tsx

@@ -26,8 +26,6 @@ const Hovedside = () => {
     return (
         <main id="main" className="hovedside">
             <div className="flex items-center flex-col w-full h-full">
-                <OppfolgingStatus>
-                </OppfolgingStatus>
                 <HovedsideFeilmelding />
                 <InformasjonsHenting />
                 <Varslinger />

src/moduler/aktiver-digital-oppfolging/AktiverDigitalOppfolging.tsx

@@ -10,7 +10,7 @@ import AktiverDigitalOppfolgingVarsel from './AktiverDigitalOppfolgingVarsel';
 
 const AktiverDigitalOppfolging = () => {
     const oppfolgingStatus = useSelector(selectOppfolgingStatus);
-    const reservertKrr = useSelector(selectReservasjonKRR);
+    const reservertKrr = useSelector(selectReservasjonKRR) || false;
 
     const settOppfolgingFeilet = oppfolgingStatus === Status.ERROR;
     const lasterOppfolging = oppfolgingStatus === Status.PENDING || oppfolgingStatus === Status.RELOADING;

src/moduler/aktiver-digital-oppfolging/AktiverDigitalOppfolgingVarsel.tsx

@@ -7,6 +7,19 @@ interface Props {
     settDigitalFeilet: boolean;
 }
 
+const tekster = {
+    veileder: {
+        tittel: 'Brukeren er reservert i KRR',
+        tekst: 'Du kan ikke sende meldinger fordi brukeren har reservert seg mot digital kommunikasjon KRR.',
+        lenkeTekst: 'Gå til norge.no for å fjerne reservasjonen.',
+    },
+    bruker: {
+        tittel: 'Du har reservert deg mot digital kommunikasjon',
+        tekst: 'Du kan ikke bruke aktivitetsplanen fordi du har reservert deg mot digital kommunikasjon i kontakt og reservasjonsregisteret (KRR)',
+        lenkeTekst: 'Gå til norge.no for å fjerne reservasjonen.',
+    },
+};
+
 const AktiverDigitalOppfolgingVarsel = (props: Props) => {
     const { reservertIKRR, settDigitalFeilet } = props;
     const erVeileder = useErVeileder();
@@ -22,33 +35,19 @@ const AktiverDigitalOppfolgingVarsel = (props: Props) => {
     //å være enten manuell eller reservert i krr for å kunne komme hit
     if (!reservertIKRR) {
         return (
-            <Alert variant="warning" className="mx-2 mb-5 max-w-2xl">
+            <Alert variant="warning" className="mx-2 mt-6 max-w-2xl">
                 Du har ikke digital oppfølging fra Nav. Du har derfor ikke en digital aktivitetsplan.
             </Alert>
         );
-    } else if (reservertIKRR && !erVeileder) {
-        return (
-            <Alert variant="warning" className="mx-2 mb-5 max-w-2xl">
-                <Heading spacing size="small" level="3">
-                    Du har reservert deg mot digital kommunikasjon
-                </Heading>
-                Du kan ikke bruke aktivitetsplanen fordi du har reservert deg mot digital kommunikasjon i kontakt og
-                reservasjonsregisteret (KRR)
-                <Link href={'https://www.norge.no/nb/digital-borgar/reservasjon'}>
-                    Gå til norge.no for å fjerne reservasjonen.
-                </Link>
-            </Alert>
-        );
-    } else if (reservertIKRR && erVeileder) {
+    } else if (reservertIKRR) {
+        const tekst = erVeileder ? tekster.veileder : tekster.bruker;
         return (
-            <Alert variant="warning" className="mx-2 mb-5 max-w-2xl">
+            <Alert variant="warning" className="mx-2 mt-6 max-w-2xl">
                 <Heading spacing size="small" level="3">
-                    Brukeren er reservert i KRR
+                    {tekst.tittel}
                 </Heading>
-                Du kan ikke sende meldinger fordi brukeren har reservert seg mot digital kommunikasjon KRR.
-                <Link href={'https://www.norge.no/nb/digital-borgar/reservasjon'}>
-                    Gå til norge.no for å fjerne reservasjonen.
-                </Link>
+                {tekst.tekst}
+                <Link href={'https://www.norge.no/nb/digital-borgar/reservasjon'}>{tekst.lenkeTekst}</Link>
             </Alert>
         );
     }

src/moduler/oppfolging-status/IkkeRegistrertIKRRAdvarsel.tsx

@@ -0,0 +1,37 @@
+import { Alert, Heading, Link } from '@navikt/ds-react';
+import React from 'react';
+
+const tekster = {
+    bruker: {
+        tittel: 'Vi har ikke din kontaktinformasjon',
+        tekst: 'Du kan ikke bruke aktivitetsplanen fordi du ikke har registrert e-post eller telefonnummeret ditt i kontakt og reservasjonsregisteret (KRR).',
+        lenkeTekst: 'Gå til norge.no for å registrere.',
+    },
+    veileder: {
+        tittel: 'Brukeren er ikke registrert i KRR',
+        tekst: 'Du kan ikke bruke aktivitetsplanen fordi brukeren ikke har registrert e-post eller telefonnummeret sitt i KRR',
+        lenkeTekst: 'Brukeren må gå til norge.no for å registrere.',
+    },
+};
+
+export function IkkeRegistrertIKRRAdvarsel({
+    erRegistrertIKRR,
+    erVeileder,
+}: {
+    erRegistrertIKRR: boolean;
+    erVeileder: boolean;
+}) {
+    if (erRegistrertIKRR) return null;
+    const tekst = erVeileder ? tekster.veileder : tekster.bruker;
+    return (
+        <div className="flex items-center flex-col">
+            <Alert variant="warning" className="mx-2 mt-6 max-w-2xl">
+                <Heading spacing size="small" level="3">
+                    {tekst.tittel}
+                </Heading>
+                <p>{tekst.tekst}</p>
+                <Link href={'https://www.norge.no/nb/digital-borgar/registrere'}>{tekst.lenkeTekst}</Link>
+            </Alert>
+        </div>
+    );
+}

src/moduler/oppfolging-status/OppfolgingStatus.tsx

@@ -5,9 +5,7 @@ interface Props {
     children: ReactNode;
 }
 
-
-
-const OppfolgingStatus = ({children}: Props) => {
+const OppfolgingStatus = ({ children }: Props) => {
     return (
         <div className="w-full">
             <VidereSendBrukereEllerRenderChildren>{children}</VidereSendBrukereEllerRenderChildren>

src/moduler/oppfolging-status/VidereSendBrukereEllerRenderChildren.tsx

@@ -18,54 +18,12 @@ import {
 import { Status } from '../../createGenericSlice';
 import { selectAktivitetStatus } from '../aktivitet/aktivitet-selector';
 import { selectIdentitetId } from '../identitet/identitet-selector';
-import { Alert, Heading, Link } from '@navikt/ds-react';
+import { IkkeRegistrertIKRRAdvarsel } from './IkkeRegistrertIKRRAdvarsel';
 
 interface VidereSendBrukereEllerRenderChildrenProps {
     children: React.ReactNode;
 }
 
-
-
-function KRRAdvarsel({ erRegistrertIKRR, erVeilder}: { erRegistrertIKRR: boolean; erVeilder: boolean; }) {
-     if (!erRegistrertIKRR && !erVeilder) {
-
-        return (
-            <div className="flex items-center flex-col">
-                <Alert variant="warning" className="mx-2 mb-5 max-w-2xl">
-                    <Heading spacing size="small" level="3">
-                        Vi har ikke din kontaktinformasjon
-                    </Heading>
-                    <p>
-                        Du kan ikke bruke aktivitetsplanen fordi du ikke har registrert e-post eller telefonnummeret
-                        ditt i kontakt og reservasjonsregisteret (KRR).
-                    </p>
-                    <Link href={'https://www.norge.no/nb/digital-borgar/registrere'}>
-                        Gå til norge.no for å registrere.
-                    </Link>
-                </Alert>
-            </div>
-        );
-
-    } else if (erVeilder && !erRegistrertIKRR){
-        return(
-        <div className="flex items-center flex-col">
-            <Alert variant="warning" className="mx-2 mb-5 max-w-2xl">
-                <Heading spacing size="small" level="3">
-                    Brukeren er ikke registrert i KRR
-                </Heading>
-                <p>
-                    Du kan ikke bruke aktivitetsplanen fordi brukeren ikke
-                    har registrert e-post eller telefonnummeret sitt i KRR
-                </p>
-                <Link href={'https://www.norge.no/nb/digital-borgar/registrere'}>
-                    Brukeren må gå til norge.no for å registrere.
-                </Link>
-            </Alert>
-        </div>
-        )
-    }
-}
-
 const VidereSendBrukereEllerRenderChildren = (props: VidereSendBrukereEllerRenderChildrenProps) => {
     const underOppfolging = useSelector(selectErUnderOppfolging);
     const oppfolgingsPerioder = useSelector(selectOppfolgingsPerioder);
@@ -99,14 +57,23 @@ const VidereSendBrukereEllerRenderChildren = (props: VidereSendBrukereEllerRende
     }
 
     if (!erRegistrertIKRR && oppfolgingsStatus === Status.OK) {
-        return <KRRAdvarsel erRegistrertIKRR={erRegistrertIKRR} erVeilder={erVeileder}/>;
-
+        return (
+            <>
+                <IkkeRegistrertIKRRAdvarsel erRegistrertIKRR={erRegistrertIKRR} erVeileder={erVeileder} />
+                {erVeileder ? props.children : null}
+            </>
+        );
     }
     if (ikkeDigitalOppfolging) {
-        return <AktiverDigitalOppfolging />;
+        return (
+            <>
+                <AktiverDigitalOppfolging />
+                {erVeileder ? props.children : null}
+            </>
+        );
     }
 
-     return <>{props.children}</>;
+    return <>{props.children}</>;
 };
 
 export default VidereSendBrukereEllerRenderChildren;

src/moduler/oppfolging-status/Videresend.test.tsx

@@ -1,25 +1,41 @@
 import { configureStore } from '@reduxjs/toolkit';
 import reducer from '../../reducer';
-import { initialLoadedEmptyState } from '../../testUtils/defaultInitialStore';
+import { aktiviteterState, initialLoadedEmptyState } from '../../testUtils/defaultInitialStore';
 import { render, waitFor } from '@testing-library/react';
 import { WrappedHovedside } from '../../testUtils/WrappedHovedside';
 import { mockfnr } from '../../mocks/utils';
 import React from 'react';
-import { afterAll, beforeAll, describe } from 'vitest';
+import { afterAll, beforeAll, describe, expect } from 'vitest';
 import { Status } from '../../createGenericSlice';
 import { mockOppfolging } from '../../mocks/data/oppfolging';
 import { OppfolgingStatus } from '../../datatypes/oppfolgingTypes';
 import { RootState } from '../../store';
 import { setupServer } from 'msw/node';
 import { handlers } from '../../mocks/handlers';
+import { mockTestAktiviteter } from '../../mocks/aktivitet';
+import { VeilarbAktivitet } from '../../datatypes/internAktivitetTypes';
+import { MinimalPeriode } from './oppfolging-selector';
 
 const defaultOppfolging = mockOppfolging;
+const aktivitetTittel = 'Videresend aktivitet';
+const periode: MinimalPeriode = {
+    id: 1,
+    start: new Date().toISOString(),
+    slutt: undefined,
+};
+const aktivitet: VeilarbAktivitet = {
+    ...mockTestAktiviteter[0],
+    tittel: aktivitetTittel,
+    oppfolgingsperiodeId: periode.id,
+};
+const stateMedAktiviteter = aktiviteterState({ aktiviteter: [aktivitet], oppfolgingsPerioder: [periode] });
 
-const storeMedOppfolging = (oppfolging: OppfolgingStatus = defaultOppfolging) => {
+const storeMedOppfolging = (oppfolging: OppfolgingStatus = defaultOppfolging): RootState => {
     return {
         ...initialLoadedEmptyState,
         data: {
             ...initialLoadedEmptyState.data,
+            aktiviteter: stateMedAktiviteter,
             oppfolging: {
                 status: Status.OK,
                 data: oppfolging,
@@ -104,18 +120,19 @@ describe('Videresend brukere eller render children', () => {
             const store = gitt.hentStatus.ikkeRegistrertIKrr();
             const { getByText } = render(<WrappedHovedside fnr={mockfnr} store={store} />);
             await waitFor(() => getByText('Brukeren er ikke registrert i KRR'));
+            getByText(aktivitetTittel);
         });
         it('skal vise varsel når bruker har reservert seg mot digital kommunikasjon i KRR', async () => {
             const store = gitt.hentStatus.reserverIKrr();
             const { getByText } = render(<WrappedHovedside fnr={mockfnr} store={store} />);
             await waitFor(() => getByText('Brukeren er reservert i KRR'));
+            getByText(aktivitetTittel);
         });
-        it('skal vise FEILMEDLING FOR BRUKER? når bruker er manuell', async () => {
+        it('skal vise aktivitetsplan når bruker er manuell', async () => {
             const store = gitt.hentStatus.manuell();
             const { getByText } = render(<WrappedHovedside fnr={mockfnr} store={store} />);
-            await waitFor(() =>
-                getByText('Du har ikke digital oppfølging fra Nav. Du har derfor ikke en digital aktivitetsplan.'),
-            );
+            await waitFor(() => getByText(aktivitetTittel));
+            getByText(aktivitetTittel);
         });
     });
     describe('Brukere:', () => {
@@ -127,20 +144,23 @@ describe('Videresend brukere eller render children', () => {
         });
         it('skal vise varsel når bruker ikke har registrert informasjon i KRR', async () => {
             const store = gitt.hentStatus.ikkeRegistrertIKrr();
-            const { getByText } = render(<WrappedHovedside store={store} />);
+            const { getByText, queryByText } = render(<WrappedHovedside store={store} />);
             await waitFor(() => getByText('Vi har ikke din kontaktinformasjon'));
+            expect(queryByText(aktivitetTittel)).toBeFalsy();
         });
         it('skal vise varsel når bruker har reservert seg mot digital kommunikasjon i KRR', async () => {
             const store = gitt.hentStatus.reserverIKrr();
-            const { getByText } = render(<WrappedHovedside store={store} />);
+            const { getByText, queryByText } = render(<WrappedHovedside store={store} />);
             await waitFor(() => getByText('Du har reservert deg mot digital kommunikasjon'));
+            expect(queryByText(aktivitetTittel)).toBeFalsy();
         });
         it('skal vise dårlig feilmelding når bruker er manuell', async () => {
             const store = gitt.hentStatus.manuell();
-            const { getByText } = render(<WrappedHovedside store={store} />);
+            const { getByText, queryByText } = render(<WrappedHovedside store={store} />);
             await waitFor(() =>
                 getByText('Du har ikke digital oppfølging fra Nav. Du har derfor ikke en digital aktivitetsplan.'),
             );
+            expect(queryByText(aktivitetTittel)).toBeFalsy();
         });
     });
 });