Bedre sortering av sisteDato (#585)

* Bedre sortering av sisteDato

* Fix testdata

* Håndter like tidspunkt i forskjellige tidsoner

src/utils/Typer.ts

@@ -17,7 +17,7 @@ export interface DialogData {
     aktivitetId: StringOrNull;
     overskrift: StringOrNull;
     sisteTekst: StringOrNull;
-    sisteDato: StringOrNull;
+    sisteDato: string;
     opprettetDato: StringOrNull;
     historisk: boolean;
     lest: boolean;

src/view/dialogliste/DialogListe.tsx

@@ -7,6 +7,8 @@ import HistoriskeDialogerOversikt from './HistoriskDialogListe';
 import { useRootLoaderData } from '../../routing/loaders';
 import { Loader } from '@navikt/ds-react';
 import { isAfter } from 'date-fns';
+import { parseISO } from 'date-fns/parseISO';
+import { isEqual } from 'date-fns/isEqual';
 
 interface Res {
     naaverende: DialogData[];
@@ -22,7 +24,7 @@ export function DialogListe() {
     const dialoger = useDialoger();
     const { dialogId } = useParams();
 
-    const sorterteDialoger = dialoger.sort((a, b) => erNyere(a.sisteDato, b.sisteDato));
+    const sorterteDialoger = dialoger.toSorted((a, b) => erNyere(a.sisteDato, b.sisteDato));
     const { naaverende, historiske } = sorterteDialoger.reduce(splitHistoriske, { naaverende: [], historiske: [] });
 
     const loaderData = useRootLoaderData();
@@ -43,7 +45,9 @@ export function DialogListe() {
 }
 
 export function erNyere(sisteDatoA: string | null, sisteDatoB: string | null): number {
-    return isAfter(sisteDatoA || '', sisteDatoB || '') ? -1 : sisteDatoA === sisteDatoB ? 0 : 1;
+    const a = sisteDatoA ? parseISO(sisteDatoA) : new Date();
+    const b = sisteDatoB ? parseISO(sisteDatoB) : new Date();
+    return isAfter(a, b) ? -1 : isEqual(a, b) ? 0 : 1;
 }
 
 const DialogListeFallback = () => {

src/view/dialogliste/ikon/Ikon.test.tsx

@@ -17,7 +17,7 @@ const dialogUtenAktivitet: DialogData = {
     lestAvBrukerTidspunkt: null,
     opprettetDato: null,
     overskrift: 'Overskrift',
-    sisteDato: null,
+    sisteDato: '2024-11-01T09:58:58.061Z',
     sisteTekst: 'asdfs'
 };
 

src/view/dialogliste/sortDialoger.test.ts

@@ -1,28 +1,34 @@
-import { describe } from 'vitest';
+import { describe, expect } from 'vitest';
 import { erNyere } from './DialogListe';
 
 describe('erNyere', () => {
-    it('skal håndtere når all tid er i zulu', () => {
+    it('skal sortere riktig når all tid er i zulu', () => {
         const gammelTid = '2024-09-09T08:00:30.171+00:00';
         const nyereTid = '2024-09-09T08:00:31.171+00:00';
         const dialoger = [nyereTid, gammelTid];
         expect(dialoger.sort((a, b) => erNyere(a, b))).toStrictEqual([nyereTid, gammelTid]);
     });
 
-    it('skal funke når input har tidssoner', () => {
+    it('skal sortere riktig når input har tidssoner', () => {
         const gammelTid = '2024-09-09T08:00:30.171+02:00';
         const nyereTid = '2024-09-09T08:00:31.171+02:00';
         const dialoger = [nyereTid, gammelTid];
         expect(dialoger.sort((a, b) => erNyere(a, b))).toStrictEqual([nyereTid, gammelTid]);
     });
 
-    it('skal funke når input har forskjellige tidssoner', () => {
+    it('skal sortere riktig når input har forskjellige tidssoner', () => {
         const gammelTid = '2024-09-09T10:00:30.171+02:00';
         const nyereTid = '2024-09-09T08:00:31.171+00:00';
         const dialoger = [nyereTid, gammelTid];
         expect(dialoger.sort((a, b) => erNyere(a, b))).toStrictEqual([nyereTid, gammelTid]);
     });
 
+    it('skal sortere riktig når input er like med forskjellige tidssoner', () => {
+        const gammelTid = '2024-09-09T10:00:30.171+02:00';
+        const nyereTid = '2024-09-09T08:00:30.171+00:00';
+        expect(erNyere(gammelTid, nyereTid)).toBe(0);
+    });
+
     it('null skal være først', () => {
         const gammelTid = '2024-09-09T10:00:30.171+02:00';
         const nullTid = null;