hanskataan tyhjä lähetys

Opetushallitus · Jan 25, 2024 · a2ef687 · a2ef687
1 parent 8de7cc5
commit a2ef687
Show file tree

Hide file tree

Showing 7 changed files with 32 additions and 36 deletions.
diff --git a/integraatio/src/test/scala/fi/oph/viestinvalitys/LocalUtil.scala b/integraatio/src/test/scala/fi/oph/viestinvalitys/LocalUtil.scala
@@ -226,6 +226,9 @@ object LocalUtil {
       omistaja: String
        */
       //kantaOperaatiot.tallennaViesti()
+      // viesti lähetystunnuksella
+      // viesti ilman lähetystunnusta
+
     }
 
 }

diff --git a/...raportointi/src/main/scala/fi/oph/viestinvalitys/raportointi/resource/LoginResource.scala b/...raportointi/src/main/scala/fi/oph/viestinvalitys/raportointi/resource/LoginResource.scala
@@ -18,6 +18,6 @@ class LoginResource {
 
   @GetMapping(path = Array(RaportointiAPIConstants.LOGIN_PATH))
   def redirect(response: HttpServletResponse): Unit = {
-    response.sendRedirect(RaportointiAPIConstants.HEALTHCHECK_PATH)
+    response.sendRedirect("localhost:3000")
   }
 }
diff --git a/viestinvalitys-raportointi/middleware.ts b/viestinvalitys-raportointi/middleware.ts
diff --git a/viestinvalitys-raportointi/src/app/layout.tsx b/viestinvalitys-raportointi/src/app/layout.tsx
@@ -2,6 +2,13 @@ import type { Metadata } from 'next'
 import { AppRouterCacheProvider } from '@mui/material-nextjs/v13-appRouter';
 import './globals.css'
 import Link from 'next/link';
+import { Roboto } from 'next/font/google'
+
+const roboto = Roboto({
+  weight: ['400', '500'],
+  display: 'swap',
+  subsets: ['latin'],
+})
 
 export const metadata: Metadata = {
   title: 'Viestinvälityspalvelun raportointi',
@@ -14,7 +21,7 @@ export default function RootLayout({
   children: React.ReactNode
 }) {
   return (
-    <html lang="fi">
+    <html lang="fi" className={roboto.className}>
       <body>
         <header>
           <nav>

diff --git a/viestinvalitys-raportointi/src/app/lib/data.ts b/viestinvalitys-raportointi/src/app/lib/data.ts
@@ -1,12 +1,19 @@
-import { headers } from 'next/headers'
+import { cookies, headers } from 'next/headers'
 import { LahetysHakuParams } from './types'
-import { apiUrl, loginUrl } from './configurations'
+import { apiUrl, cookieName, loginUrl } from './configurations'
 import { redirect } from 'next/navigation'
 
 // TODO apuwrapperi headerien asettamiseen 
 export async function fetchLahetykset(hakuParams: LahetysHakuParams) {
-    console.info(hakuParams)
+    const sessionCookie = cookies().get(cookieName)
+    console.info(sessionCookie)
     const headersInstance = headers()
+    console.info(headersInstance.get('cookie'))
+    if (sessionCookie === undefined) {
+      console.info('no session cookie, redirect to login')
+      redirect(loginUrl)
+    }
+    console.info(hakuParams)
     const fetchUrlBase = `${apiUrl}/lahetykset/lista?enintaan=2`
     const fetchParams = hakuParams.seuraavatAlkaen ? `&alkaen=${hakuParams.seuraavatAlkaen}` : ''
     console.info(fetchUrlBase.concat(fetchParams))
@@ -16,6 +23,7 @@ export async function fetchLahetykset(hakuParams: LahetysHakuParams) {
     console.info(res.status)
     if (!res.ok) {
       if(res.status===401) {
+        console.info('http 401, redirect to login')
         redirect(loginUrl)
       }
       // This will activate the closest `error.js` Error Boundary

diff --git a/viestinvalitys-raportointi/src/app/lib/util.test.ts b/viestinvalitys-raportointi/src/app/lib/util.test.ts
@@ -1,5 +1,5 @@
 import { expect, test } from 'vitest'
-import { getLahetysStatus, getVastaanottajatPerStatus } from './util';
+import { getLahetysStatus, getVastaanottajatPerStatus, lahetyksenStatus } from './util';
 import { Status, VastaanotonTila, VastaanottajaTila } from './types';
 
 const onnistunutTila = [VastaanotonTila.DELIVERY]
@@ -42,4 +42,9 @@ test('Vastaavassa tilassa olevien viestien lukumäärät summataan', () => {
             vastaanottajaLkm: 2
         }
     ]))).toEqual(3);
-});
+});
+
+test('Tyhjä lähetys osataan käsitellä'), () => {
+    expect(lahetyksenStatus(undefined)).toEqual('-')
+    expect(lahetyksenStatus(undefined)).toEqual('-')
+}
diff --git a/viestinvalitys-raportointi/src/app/lib/util.ts b/viestinvalitys-raportointi/src/app/lib/util.ts
@@ -6,8 +6,8 @@ export const getLahetyksenVastaanottajia = (tilat: VastaanottajaTila[]): number
     { return a + b;})
   }
 
-export const lahetyksenStatus = (tilat: VastaanottajaTila[]): string => {
-    if(tilat.length<1) {
+export const lahetyksenStatus = (tilat: VastaanottajaTila[] | undefined): string => {
+    if(!tilat || tilat.length<1) {
       return '-'
     }
     const status = getVastaanottajatPerStatus(tilat)+'/'+getLahetyksenVastaanottajia(tilat)+ ' viestin lähetys ' + getLahetysStatus(tilat.map(tila => tila.vastaanottotila))