Bedre sortering av sisteDato

navikt · Nov 1, 2024 · 912a787 · 912a787
1 parent a8487b2
commit 912a787
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 6 deletions.
diff --git a/src/utils/Typer.ts b/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;

diff --git a/src/view/dialogliste/DialogListe.tsx b/src/view/dialogliste/DialogListe.tsx
@@ -7,6 +7,7 @@ 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';
 
 interface Res {
     naaverende: DialogData[];
@@ -22,7 +23,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 +44,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 : sisteDatoA === sisteDatoB ? 0 : 1;
 }
 
 const DialogListeFallback = () => {

diff --git a/src/view/dialogliste/sortDialoger.test.ts b/src/view/dialogliste/sortDialoger.test.ts
@@ -2,21 +2,21 @@ import { describe } 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];