From 4a5653ad55a168bbadc69346427e82cec9311c52 Mon Sep 17 00:00:00 2001
From: Petteri Tolonen <petteri.tolonen@knowit.fi>
Date: Tue, 7 Jan 2025 10:47:06 +0200
Subject: [PATCH] =?UTF-8?q?OK-674:=20Ei=20n=C3=A4ytet=C3=A4=20tyhj=C3=A4?=
 =?UTF-8?q?=C3=A4=20"suorittamattomat=20hakukohteet"-haitaria?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../components/suorittamattomat-hakukohteet.tsx            | 5 +++--
 tests/e2e/henkilo.spec.ts                                  | 7 ++++++-
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/app/haku/[oid]/henkilo/[hakemusOid]/components/suorittamattomat-hakukohteet.tsx b/src/app/haku/[oid]/henkilo/[hakemusOid]/components/suorittamattomat-hakukohteet.tsx
index ba1062c3..2b6524ef 100644
--- a/src/app/haku/[oid]/henkilo/[hakemusOid]/components/suorittamattomat-hakukohteet.tsx
+++ b/src/app/haku/[oid]/henkilo/[hakemusOid]/components/suorittamattomat-hakukohteet.tsx
@@ -1,6 +1,7 @@
 import { ErrorWithIcon } from '@/app/components/error-with-icon';
 import { SimpleAccordion } from '@/app/components/simple-accordion';
 import { useTranslations } from '@/app/hooks/useTranslations';
+import { isEmpty } from '@/app/lib/common';
 import { NDASH } from '@/app/lib/constants';
 import { LaskentaActorRef } from '@/app/lib/state/laskenta-state';
 import { Hakukohde } from '@/app/lib/types/kouta-types';
@@ -25,7 +26,7 @@ export const SuorittamattomatHakukohteet = ({
     s.context.summary?.hakukohteet.filter((hk) => hk?.tila !== 'VALMIS'),
   );
 
-  return summaryErrors ? (
+  return isEmpty(summaryErrors) ? null : (
     <SimpleAccordion
       titleOpen={t('henkilo.piilota-suorittamattomat-hakukohteet')}
       titleClosed={t('henkilo.nayta-suorittamattomat-hakukohteet')}
@@ -72,5 +73,5 @@ export const SuorittamattomatHakukohteet = ({
         })}
       </Stack>
     </SimpleAccordion>
-  ) : null;
+  );
 };
diff --git a/tests/e2e/henkilo.spec.ts b/tests/e2e/henkilo.spec.ts
index b6fa8eee..68ce75df 100644
--- a/tests/e2e/henkilo.spec.ts
+++ b/tests/e2e/henkilo.spec.ts
@@ -962,6 +962,11 @@ test.describe('Valintalaskenta', () => {
       page.getByText('Laskenta suoritettu onnistuneesti'),
     ).toBeVisible();
 
+    const suljeTiedotButton = page.getByRole('button', {
+      name: 'Sulje laskennan tiedot',
+    });
+    await expect(suljeTiedotButton).toBeVisible();
+
     await expect(
       page.getByRole('button', { name: 'Näytä suorittamattomat hakukohteet' }),
     ).toBeHidden();
@@ -970,7 +975,7 @@ test.describe('Valintalaskenta', () => {
       page.getByRole('button', { name: 'Suorita valintalaskenta' }),
     ).toBeHidden();
 
-    await page.getByRole('button', { name: 'Sulje laskennan tiedot' }).click();
+    await suljeTiedotButton.click();
 
     await expect(
       page.getByRole('button', { name: 'Suorita valintalaskenta' }),