Tillate 0 dager på enkeltinnsending uten å gjøre det samme på bulk (#380

)

src/components/bulk/BulkDager.tsx

@@ -15,7 +15,7 @@ export const BulkDager = (props: BulkDagerInterface) => {
   const aktuellAnsatt = ansatte.find((ansatt) => ansatt.id === props.id);
   const handleChange = (evt) => {
     if (aktuellAnsatt) {
-      aktuellAnsatt.antallDagerMedRefusjon = parseInt(evt.target.selectedIndex);
+      aktuellAnsatt.antallDagerMedRefusjon = parseInt(evt.target.value);
       aktuellAnsatt.dagerError = validateDager(
         aktuellAnsatt.antallDagerMedRefusjon
       );
@@ -26,6 +26,7 @@ export const BulkDager = (props: BulkDagerInterface) => {
   const refusjonsdager = antallRefusjonsdager(props.startdato);
 
   const optionArr = Array.from(Array(refusjonsdager + 1).keys());
+
   return (
     <Select
       id={'dager_' + props.id}

src/components/bulk/ValideringAnsatte.test.ts

@@ -104,25 +104,29 @@ describe('ValideringAnsatte', () => {
         fnr: TestFnr.GyldigeFraDolly.TestPerson1,
         fnrError: undefined,
         fom: '2020-01-01',
+        fomError: undefined,
         id: 123,
         oppdatert: 1,
         periodeError: undefined,
         status: 0,
-        tom: '2020-02-02'
+        tom: '2020-02-02',
+        tomError: undefined
       },
       {
         antallDagerMedRefusjon: -1,
         beloep: 123,
         beloepError: undefined,
-        dagerError: 'Dager må være 0 eller høyere',
+        dagerError: 'Dager må være høyere enn 0',
         fnr: TestFnr.GyldigeFraDolly.TestPerson2,
         fnrError: undefined,
         fom: '2020-03-03',
+        fomError: undefined,
         id: 123,
         oppdatert: 1,
         periodeError: undefined,
         status: 0,
-        tom: '2020-04-04'
+        tom: '2020-04-04',
+        tomError: undefined
       }
     ];
 
@@ -267,15 +271,17 @@ describe('ValideringAnsatte', () => {
         antallDagerMedRefusjon: 0,
         beloep: 666,
         beloepError: 'Beløpet må være 0 når antall dager med refusjon er 0',
-        dagerError: undefined,
+        dagerError: 'Dager må være høyere enn 0',
         fnr: TestFnr.GyldigeFraDolly.TestPerson2,
         fnrError: undefined,
         fom: '2020-03-03',
+        fomError: undefined,
         id: 123,
         oppdatert: 1,
         periodeError: undefined,
         status: 0,
-        tom: '2020-04-04'
+        tom: '2020-04-04',
+        tomError: undefined
       }
     ];
 

src/components/bulk/ValideringAnsatte.ts

@@ -1,6 +1,6 @@
 import { validateAnsatteFnr } from './validateAnsatteFnr';
 import { validatePerioder } from './validatePerioder';
-import validateDager from '../dager/validateDager';
+import validateDagerEnkeltinnsending from '../dager/validateDagerEnkeltinnsending';
 import validateRefusjon from '../refusjon/validateRefusjon';
 import { Ansatt } from './Ansatt';
 import { validateDato } from './validateDato';
@@ -12,7 +12,7 @@ export const valideringAnsatte = (ansatte: Ansatt[]) => {
     a.periodeError = validatePerioder(a.fom, a.tom);
     a.fomError = validateDato(dayjs(a.fom, 'YYYY-MM-DD').toDate());
     a.tomError = validateDato(dayjs(a.tom, 'YYYY-MM-DD').toDate());
-    a.dagerError = validateDager(a.antallDagerMedRefusjon);
+    a.dagerError = validateDagerEnkeltinnsending(a.antallDagerMedRefusjon);
     a.beloepError = validateRefusjon(a.beloep, a.antallDagerMedRefusjon);
   });
 

src/components/dager/DagerInput.tsx

@@ -23,6 +23,7 @@ export const DagerInput = ({
   const [lokalFeil, setLokalFeil] = useState<string | undefined>();
   const valgbareDager = antallRefusjonsdager(startdato);
   const optionArr = Array.from(Array(valgbareDager + 1).keys());
+  optionArr.unshift(-1);
 
   const handleDagerBlur = (evt) => {
     const feilmelding = validateDager(evt.target.value);
@@ -55,7 +56,7 @@ export const DagerInput = ({
     >
       {optionArr.map((optionValue) => (
         <option key={optionValue} value={optionValue}>
-          {optionValue ? optionValue : '-'}
+          {optionValue !== -1 ? optionValue : '-'}
         </option>
       ))}
     </Select>

src/components/dager/validateDagerEnkeltinnsending.test.tsx

@@ -0,0 +1,30 @@
+import validateDagerEnkeltinnsending from './validateDagerEnkeltinnsending';
+
+describe('validateDagerEnkeltinnsending', () => {
+  it('should not allow zero', () => {
+    expect(validateDagerEnkeltinnsending(0)).toEqual(
+      'Dager må være høyere enn 0'
+    );
+  });
+
+  it('should not accept undefined', () => {
+    expect(validateDagerEnkeltinnsending()).toBe('Feltet må fylles ut');
+  });
+
+  it('should accept 1-13', () => {
+    expect(validateDagerEnkeltinnsending(1)).toBeUndefined();
+    expect(validateDagerEnkeltinnsending(13)).toBeUndefined();
+  });
+
+  it('should not accept negative', () => {
+    expect(validateDagerEnkeltinnsending(-1)).toEqual(
+      'Dager må være høyere enn 0'
+    );
+  });
+
+  it('should not allow to high value', () => {
+    expect(validateDagerEnkeltinnsending(14)).toEqual(
+      'Dager må være 13 eller lavere'
+    );
+  });
+});

src/components/dager/validateDagerEnkeltinnsending.tsx

@@ -0,0 +1,12 @@
+const validateDager = (value?: number): string | undefined => {
+  if (value === undefined) {
+    return 'Feltet må fylles ut';
+  } else if (value <= 0) {
+    return 'Dager må være høyere enn 0';
+  } else if (value > 13) {
+    return 'Dager må være 13 eller lavere';
+  }
+  return undefined;
+};
+
+export default validateDager;

src/components/enkel/EnkelDager.tsx

@@ -16,7 +16,7 @@ const EnkelDager = (props: EnkelDagerProps) => {
   } = useFormContext();
   const componentId = 'dager_' + props.index;
   const onChange = (dager?: number) => {
-    if (!dager) {
+    if (dager < 0) {
       setError(componentId, {
         type: 'Antall dager må fylles ut',
         message: 'Antall dager må fylles ut'

src/pages/Sykepenger.tsx

@@ -337,7 +337,7 @@ function validateValuesAreSet(
         }
         break;
       case 'dager':
-        if (!formAsJson[element] || formAsJson[element] === '0') {
+        if (!formAsJson[element] || Number(formAsJson[element]) < 0) {
           methods.setError(element, {
             type: 'Antall dager må fylles ut',
             message: 'Antall dager må fylles ut'