Legger til ja nei spørsmål for tidligere samboer (#1297)

* Små endringer * Bytter spørsmål om tidligere samboere fra knapp til ja-nei spørsmål * Fikset lagring av tidligereSamboere * Fjerner unødvendig typing av felt * Fikser lagring av hattFlereSamboereForSøktPeriode spørsmål * Stopper hattFlereSamboereForSøktPeriode og tidligereSamboere fra å lagres etter søknad startes på nytt * Resetter utvidet.spørsmål.årsak og utvidet.spørsmål.separertEnkeSkilt ettersom de hang igjen selv når søknad ble startet på nytt * Legger til oppsett av spørsmål for test slik at tidligereSamboere kan renderes og brukes i test * Fikser hattAnnenSamboerForSøktPeriode og relaterte testdata slik at testene fungerer som de skal * Legger til steg i test for å velge jaKnapp slik at leggTilSamboerKnapp renderes * Fjerner kode for å vise lengen på tidligere samboere som ble brukt til feilsøking * Legger til søknadsvar for hattAnnenSamboerForSøktPeriode og reduserer kode for å liste ut tidligereSamboere * Legger til persontype barn * Revert "Legger til persontype barn" This reverts commit 5e1ba21.
navikt · Jul 25, 2024 · 5114c58 · 5114c58
1 parent 4b67133
commit 5114c58
Show file tree

Hide file tree

Showing 22 changed files with 181 additions and 101 deletions.
diff --git a/src/frontend/assets/lang/en.json b/src/frontend/assets/lang/en.json
@@ -495,7 +495,7 @@
   "ombarnet.andre-forelder.utenlandspensjon.land.spm": "What country does {barn}'s other parent receive a pension from?",
   "ombarnet.andreforelderpensjonutland.feilmelding": "You must state whether {navn}'s other parent receives or has received a pension from abroad in order to be able to proceed",
   "ombarnet.andreforelderpensjonutland.spm": "Does {navn}'s other parent receive, or have they received a pension from abroad?",
-  "ombarnet.barnetrygd-eøs": "You have stated that you are receiving or have applied for child benefit for {navn} from another EEA country.",
+  "ombarnet.barnetrygd-eøs": "You have stated that you are receiving, have received or have applied for child benefit for {navn} from another EEA country.",
   "ombarnet.barnetrygd-eøs-fortid": "You have stated that you are receiving, have received or have applied for child benefit for {navn} from another EEA country.",
   "ombarnet.barnetrygd-eøs.land.feilmelding": "You must state which country you are receiving or have applied for child benefit from in order to be able to proceed",
   "ombarnet.barnetrygd-eøs.land.spm": "From which country are you receiving or have you applied for child benefit?",
@@ -630,6 +630,7 @@
   "omdeg.nårstartetsamboerforhold.spm": "When did you start living together?",
   "omdeg.nåværendeSamboer.ident.ukjent": "I don't know the National identity number or-D number",
   "omdeg.nåværendesamboer.fødselsdato.ukjent": "You must provide your cohabiting partners date of birth in order to be able to proceed",
+  "omdeg.tidligereSamboer.feilmelding": "You must add at least one former partner to proceed",
   "omdeg.oppholderi.eøs-info": "Since you are staying in an EEA country you must download and fill out a supplementary form and provide it later on in the application.",
   "omdeg.oppholdti.eøs-info": "Since you have stayed in an EEA country you must download and fill out a supplementary form and provide it later on in the application.",
   "omdeg.oppholdtsammenhengende.feilmelding": "You must state whether you have stayed continuously in Norway for the last twelve months in order to be able to proceed",

diff --git a/src/frontend/assets/lang/nb.json b/src/frontend/assets/lang/nb.json
@@ -495,7 +495,7 @@
   "ombarnet.andre-forelder.utenlandspensjon.land.spm": "Hvilket land får den andre forelderen til {barn} pensjon fra?",
   "ombarnet.andreforelderpensjonutland.feilmelding": "Du må oppgi om den andre forelderen til {navn} får eller har fått pensjon fra utlandet for å gå videre",
   "ombarnet.andreforelderpensjonutland.spm": "Får eller har den andre forelderen til {navn} fått pensjon fra utlandet?",
-  "ombarnet.barnetrygd-eøs": "Du har opplyst at du får eller har søkt om barnetrygd for {navn} fra et annet EØS land.",
+  "ombarnet.barnetrygd-eøs": "Du har opplyst at du får, har fått eller har søkt om barnetrygd for {navn} fra et annet EØS land.",
   "ombarnet.barnetrygd-eøs-fortid": "Du har opplyst at du får, har fått eller har søkt om barnetrygd for {navn} fra et annet EØS land.",
   "ombarnet.barnetrygd-eøs.land.feilmelding": "Du må oppgi hvilket land du får eller har søkt om barnetrygd fra for å gå videre",
   "ombarnet.barnetrygd-eøs.land.spm": "Hvilket land får eller har du søkt om barnetrygd fra?",
@@ -630,6 +630,7 @@
   "omdeg.nårstartetsamboerforhold.spm": "Når startet samboerforholdet?",
   "omdeg.nåværendeSamboer.ident.ukjent": "Jeg kjenner ikke fødselsnummer eller d-nummer",
   "omdeg.nåværendesamboer.fødselsdato.ukjent": "Du må oppgi samboerens fødselsdato for å gå videre",
+  "omdeg.tidligereSamboer.feilmelding": "Du må legge til minst en tidligere samboer for å gå videre",
   "omdeg.oppholderi.eøs-info": "Du må laste ned og fylle ut tilleggsskjema og legge ved senere i søknaden, siden du oppholder deg i et EØS-land.",
   "omdeg.oppholdti.eøs-info": "Du må laste ned og fylle ut tilleggsskjema og legge ved senere i søknaden, siden du har oppholdt deg i et EØS-land.",
   "omdeg.oppholdtsammenhengende.feilmelding": "Du må oppgi om du har oppholdt deg sammenhengende i Norge de siste tolv månedene for å gå videre",

diff --git a/src/frontend/assets/lang/nn.json b/src/frontend/assets/lang/nn.json
@@ -495,7 +495,7 @@
   "ombarnet.andre-forelder.utenlandspensjon.land.spm": "Kva land får den andre forelderen til {barn} pensjon frå?",
   "ombarnet.andreforelderpensjonutland.feilmelding": "Du må oppgje om den andre forelderen til {navn} får eller har fått pensjon frå utlandet for å gå vidare",
   "ombarnet.andreforelderpensjonutland.spm": "Får eller har den andre forelderen til {navn} fått pensjon frå utlandet?",
-  "ombarnet.barnetrygd-eøs": "Du har opplyst at du får eller har søkt om barnetrygd for {navn} frå eit anna EØS land.",
+  "ombarnet.barnetrygd-eøs": "Du har opplyst at du får, har fått eller har søkt om barnetrygd for {navn} frå eit anna EØS land.",
   "ombarnet.barnetrygd-eøs-fortid": "Du har opplyst at du får, har fått eller har søkt om barnetrygd for {navn} frå eit anna EØS land.",
   "ombarnet.barnetrygd-eøs.land.feilmelding": "Du må oppgje kva land du får eller har søkt barnetrygd frå for å gå vidare",
   "ombarnet.barnetrygd-eøs.land.spm": "Kva land får eller har du søkt barnetrygd frå?",
@@ -630,6 +630,7 @@
   "omdeg.nårstartetsamboerforhold.spm": "Når starta sambuarforholdet?",
   "omdeg.nåværendeSamboer.ident.ukjent": "Eg kjenner ikkje fødselsnummer eller d-nummer",
   "omdeg.nåværendesamboer.fødselsdato.ukjent": "Du må oppgje sambuarens fødselsdato for å gå vidare",
+  "omdeg.tidligereSamboer.feilmelding": "Du må leggja til minst ein tidlegare sambuar for å gå vidare",
   "omdeg.oppholderi.eøs-info": "Du må laste ned og fylle ut tilleggsskjema og legge ved seinare i søknaden, sidan du oppheld deg i eit EØS-land.",
   "omdeg.oppholdti.eøs-info": "Du må laste ned og fylle ut tilleggsskjema og legge ved seinare i søknaden, sidan du har opphalde deg eit EØS-land.",
   "omdeg.oppholdtsammenhengende.feilmelding": "Du må oppgje om du har opphalde deg samanhengande i Noreg dei siste tolv månadene for å gå vidare",

diff --git a/src/frontend/components/SøknadsSteg/DinLivssituasjon/DinLivssituasjon.tsx b/src/frontend/components/SøknadsSteg/DinLivssituasjon/DinLivssituasjon.tsx
@@ -29,7 +29,6 @@ const DinLivssituasjon: React.FC = () => {
         validerFelterOgVisFeilmelding,
         valideringErOk,
         oppdaterSøknad,
-        tidligereSamboere,
         leggTilTidligereSamboer,
         fjernTidligereSamboer,
         leggTilArbeidsperiode,
@@ -151,9 +150,13 @@ const DinLivssituasjon: React.FC = () => {
                         )}
 
                         <TidligereSamboere
-                            tidligereSamboere={tidligereSamboere}
+                            skjema={skjema}
                             leggTilTidligereSamboer={leggTilTidligereSamboer}
                             fjernTidligereSamboer={fjernTidligereSamboer}
+                            hattAnnenSamboerForSøktPeriodeFelt={
+                                skjema.felter.hattAnnenSamboerForSøktPeriode
+                            }
+                            tidligereSamboere={skjema.felter.tidligereSamboere}
                         />
                     </KomponentGruppe>
                 </>

diff --git a/src/frontend/components/SøknadsSteg/DinLivssituasjon/LeggTilSamboerModal.test.tsx b/src/frontend/components/SøknadsSteg/DinLivssituasjon/LeggTilSamboerModal.test.tsx
@@ -1,6 +1,6 @@
 import React from 'react';
 
-import { render } from '@testing-library/react';
+import { render, within } from '@testing-library/react';
 import { mockDeep } from 'jest-mock-extended';
 import { act } from 'react-dom/test-utils';
 
@@ -40,15 +40,22 @@ describe('LeggTilSamboerModal', () => {
     beforeEach(() => {
         silenceConsoleErrors();
     });
-    it('Viser riktige feilmeldinger ved ingen utfylte felt av tidligere samboer', () => {
+    it('Viser riktige feilmeldinger ved ingen utfylte felt av tidligere samboer', async () => {
         spyOnUseApp(søknad);
 
-        const { getByText, getAllByText, queryByText } = render(
+        const { getByText, getAllByText, findByRole, queryByText } = render(
             <TestProvidereMedEkteTekster mocketNettleserHistorikk={['/din-livssituasjon']}>
                 <DinLivssituasjon />
             </TestProvidereMedEkteTekster>
         );
 
+        const hattAnnenSamboerForSøktPeriode: HTMLElement = await findByRole('group', {
+            name: /Har du hatt samboer tidligere i perioden du søker barnetrygd for?/i,
+        });
+
+        const jaKnapp: HTMLElement = within(hattAnnenSamboerForSøktPeriode).getByText('Ja');
+        act(() => jaKnapp.click());
+
         const leggTilSamboerKnapp: HTMLElement = getByText('Legg til samboer');
         act(() => leggTilSamboerKnapp.click());
 

diff --git a/src/frontend/components/SøknadsSteg/DinLivssituasjon/TidligereSamboere.test.tsx b/src/frontend/components/SøknadsSteg/DinLivssituasjon/TidligereSamboere.test.tsx
@@ -79,6 +79,13 @@ describe('TidligereSamboere', () => {
                 ...søknad.søker,
                 utvidet: {
                     ...søknad.søker.utvidet,
+                    spørsmål: {
+                        ...søknad.søker.utvidet.spørsmål,
+                        hattAnnenSamboerForSøktPeriode: {
+                            id: DinLivssituasjonSpørsmålId.hattAnnenSamboerForSøktPeriode,
+                            svar: ESvar.JA,
+                        },
+                    },
                     tidligereSamboere: [
                         {
                             navn: {

diff --git a/src/frontend/components/SøknadsSteg/DinLivssituasjon/TidligereSamboere.tsx b/src/frontend/components/SøknadsSteg/DinLivssituasjon/TidligereSamboere.tsx
@@ -1,12 +1,14 @@
 import React from 'react';
 
-import styled from 'styled-components';
-
-import { Label } from '@navikt/ds-react';
+import { ESvar } from '@navikt/familie-form-elements';
+import { Felt, ISkjema } from '@navikt/familie-skjema';
 
 import { ITidligereSamboer } from '../../../typer/person';
 import { PersonType } from '../../../typer/personType';
+import { IDinLivssituasjonFeltTyper } from '../../../typer/skjema';
 import { hentLeggTilPeriodeTekster } from '../../../utils/modaler';
+import { genererPeriodeId } from '../../../utils/perioder';
+import JaNeiSpm from '../../Felleskomponenter/JaNeiSpm/JaNeiSpm';
 import { LeggTilKnapp } from '../../Felleskomponenter/LeggTilKnapp/LeggTilKnapp';
 import PerioderContainer from '../../Felleskomponenter/PerioderContainer';
 import useModal from '../../Felleskomponenter/SkjemaModal/useModal';
@@ -17,35 +19,27 @@ import SamboerOpplysninger from './SamboerOpplysninger';
 import { DinLivssituasjonSpørsmålId, dinLivssituasjonSpørsmålSpråkId } from './spørsmål';
 
 interface Props {
+    skjema: ISkjema<IDinLivssituasjonFeltTyper, string>;
     leggTilTidligereSamboer: (samboer: ITidligereSamboer) => void;
-    tidligereSamboere: ITidligereSamboer[];
     fjernTidligereSamboer: (samboer: ITidligereSamboer) => void;
+    hattAnnenSamboerForSøktPeriodeFelt: Felt<ESvar | null>;
+    tidligereSamboere: Felt<ITidligereSamboer[]>;
 }
 
-const StyledLabel = styled(Label)`
-    && {
-        margin-bottom: 0.5rem;
-    }
-`;
-
-const Spørsmål: React.FC<{ språkId: string }> = ({ språkId }) => (
-    <StyledLabel forwardedAs="p">
-        <SpråkTekst id={språkId} />
-    </StyledLabel>
-);
-
 const TidligereSamboere: React.FC<Props> = ({
+    skjema,
     leggTilTidligereSamboer,
-    tidligereSamboere,
     fjernTidligereSamboer,
+    hattAnnenSamboerForSøktPeriodeFelt,
+    tidligereSamboere,
 }) => {
     const {
         lukkModal: lukkLeggTilSamboerModal,
         åpneModal: åpneLeggTilSamboerModal,
         erÅpen: erLeggTilSamboerModalÅpen,
     } = useModal();
 
-    const antallPerioder = tidligereSamboere.length;
+    const antallPerioder = tidligereSamboere.verdi.length;
     const leggTilPeriodeTekster = hentLeggTilPeriodeTekster(
         'tidligereSamboere',
         PersonType.Søker,
@@ -54,34 +48,49 @@ const TidligereSamboere: React.FC<Props> = ({
 
     return (
         <>
-            <Spørsmål
-                språkId={
+            <JaNeiSpm
+                skjema={skjema}
+                felt={hattAnnenSamboerForSøktPeriodeFelt}
+                spørsmålTekstId={
                     dinLivssituasjonSpørsmålSpråkId[
                         DinLivssituasjonSpørsmålId.hattAnnenSamboerForSøktPeriode
                     ]
                 }
             />
-            <PerioderContainer>
-                {tidligereSamboere?.map((samboer: ITidligereSamboer, index: number) => (
-                    <SamboerOpplysninger
-                        key={index}
-                        samboer={samboer}
-                        fjernTidligereSamboer={fjernTidligereSamboer}
+            {hattAnnenSamboerForSøktPeriodeFelt.verdi === ESvar.JA && (
+                <PerioderContainer>
+                    {tidligereSamboere.verdi.map((samboer: ITidligereSamboer, index: number) => (
+                        <SamboerOpplysninger
+                            key={index}
+                            samboer={samboer}
+                            fjernTidligereSamboer={fjernTidligereSamboer}
+                        />
+                    ))}
+                    <LeggTilKnapp
+                        språkTekst="omdeg.leggtilfleresamboere.leggtil"
+                        forklaring={leggTilPeriodeTekster?.tekstForKnapp}
+                        onClick={åpneLeggTilSamboerModal}
+                        id={genererPeriodeId({
+                            personType: PersonType.Søker,
+                            spørsmålsId: DinLivssituasjonSpørsmålId.hattAnnenSamboerForSøktPeriode,
+                        })}
+                        feilmelding={
+                            tidligereSamboere.erSynlig &&
+                            tidligereSamboere.feilmelding &&
+                            skjema.visFeilmeldinger && (
+                                <SpråkTekst id="omdeg.tidligereSamboer.feilmelding" />
+                            )
+                        }
                     />
-                ))}
-                <LeggTilKnapp
-                    språkTekst="omdeg.leggtilfleresamboere.leggtil"
-                    forklaring={leggTilPeriodeTekster?.tekstForKnapp}
-                    onClick={åpneLeggTilSamboerModal}
-                />
-            </PerioderContainer>
-            {erLeggTilSamboerModalÅpen && (
-                <LeggTilSamboerModal
-                    leggTilTidligereSamboer={leggTilTidligereSamboer}
-                    lukkModal={lukkLeggTilSamboerModal}
-                    erÅpen={erLeggTilSamboerModalÅpen}
-                    forklaring={leggTilPeriodeTekster?.tekstForModal}
-                />
+                    {erLeggTilSamboerModalÅpen && (
+                        <LeggTilSamboerModal
+                            leggTilTidligereSamboer={leggTilTidligereSamboer}
+                            lukkModal={lukkLeggTilSamboerModal}
+                            erÅpen={erLeggTilSamboerModalÅpen}
+                            forklaring={leggTilPeriodeTekster?.tekstForModal}
+                        />
+                    )}
+                </PerioderContainer>
             )}
         </>
     );

diff --git a/src/frontend/components/SøknadsSteg/DinLivssituasjon/spørsmål.ts b/src/frontend/components/SøknadsSteg/DinLivssituasjon/spørsmål.ts
@@ -6,7 +6,6 @@ export enum DinLivssituasjonSpørsmålId {
     harSamboerNå = 'har-samboer-nå',
     harSamboerNåGift = 'har-samboer-nå-og-gift',
     hattAnnenSamboerForSøktPeriode = 'hatt-annen-samboer-i-perioden',
-    hattFlereSamboereForSøktPeriode = 'hatt-flere-samboere-i-perioden',
     erAsylsøker = 'er-asylsøker',
     arbeidIUtlandet = 'arbeid-i-utlandet',
     mottarUtenlandspensjon = 'mottar-utenlandspensjon',
@@ -20,7 +19,6 @@ export const dinLivssituasjonSpørsmålSpråkId: Record<DinLivssituasjonSpørsm
     [DinLivssituasjonSpørsmålId.harSamboerNå]: 'omdeg.samboernå.spm',
     [DinLivssituasjonSpørsmålId.harSamboerNåGift]: 'omdeg.samboernå.gift.spm',
     [DinLivssituasjonSpørsmålId.hattAnnenSamboerForSøktPeriode]: 'omdeg.annensamboer.spm',
-    [DinLivssituasjonSpørsmålId.hattFlereSamboereForSøktPeriode]: 'omdeg.leggtilfleresamboere.spm',
     [DinLivssituasjonSpørsmålId.erAsylsøker]: 'omdeg.asylsøker.spm',
     [DinLivssituasjonSpørsmålId.arbeidIUtlandet]: 'eøs.arbeid-utland.spm',
     [DinLivssituasjonSpørsmålId.mottarUtenlandspensjon]: 'omdeg.pensjonutland.spm',