Merge pull request #62 from Opetushallitus/OK-674_suorittamattomat-ha…

…kukohteet-fix

OK-674: Ei näytetä tyhjää "suorittamattomat hakukohteet"-haitaria

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;
+  );
 };

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' }),