QA Forbedringer: Sjekk at streng ikke er tom i tillegg til undefined …

…og null

src/helpers/steg/bosituasjon.ts

@@ -5,22 +5,26 @@ import { IPersonDetaljer } from '../../models/søknad/person';
 import { harValgtSvar } from '../../utils/spørsmålogsvar';
 import { erDatoGyldigOgInnaforBegrensninger } from '../../components/dato/utils';
 import { DatoBegrensning } from '../../components/dato/Datovelger';
+import { stringHarVerdiOgErIkkeTom } from '../../utils/typer';
 
 const harPlanerOmÅBliSamboerEllerSkalGifteSeg = (bosituasjon: IBosituasjon) => {
   const { skalGifteSegEllerBliSamboer } = bosituasjon;
 
-  return skalGifteSegEllerBliSamboer !== undefined && skalGifteSegEllerBliSamboer.svarid === ESvar.JA
-
+  return (
+    skalGifteSegEllerBliSamboer !== undefined &&
+    skalGifteSegEllerBliSamboer.svarid === ESvar.JA
+  );
 };
 
 const harSattFødselsdato = (fødselsdato?: string): boolean =>
-  fødselsdato !== undefined &&
-    erDatoGyldigOgInnaforBegrensninger(
-        fødselsdato,
-        DatoBegrensning.TidligereDatoer
-    );
+  stringHarVerdiOgErIkkeTom(fødselsdato) &&
+  erDatoGyldigOgInnaforBegrensninger(
+    fødselsdato,
+    DatoBegrensning.TidligereDatoer
+  );
 
-const harSattIdent = (ident?: string): boolean => (ident !== undefined);
+const harSattIdent = (ident?: string): boolean =>
+  stringHarVerdiOgErIkkeTom(ident);
 
 const harFerdigUtfyltOmSamboer = (
   samboerDetaljer?: IPersonDetaljer,
@@ -56,11 +60,13 @@ const harFerdigUtfyltPlanerOmÅBliSamboerEllerBliGift = (
 
 const harSattDatoFlyttetFraHverandre = (bosituasjon: IBosituasjon) => {
   const { datoFlyttetFraHverandre } = bosituasjon;
-  return datoFlyttetFraHverandre?.verdi !== undefined &&
+  return (
+    stringHarVerdiOgErIkkeTom(datoFlyttetFraHverandre) &&
     erDatoGyldigOgInnaforBegrensninger(
       datoFlyttetFraHverandre?.verdi,
       DatoBegrensning.AlleDatoer
-    );
+    )
+  );
 };
 
 export const erFerdigUtfylt = (bosituasjon: IBosituasjon) => {

src/utils/typer.ts

@@ -1,3 +1,7 @@
 export function harVerdi<T>(verdi: T | null | undefined): verdi is T {
   return verdi !== null && verdi !== undefined;
 }
+
+export function stringHarVerdiOgErIkkeTom<T>(verdi: T | null | undefined): verdi is T {
+  return harVerdi(verdi) && verdi !== "";
+}