From 493394649f8ce5db668d30fba126c9b307083251 Mon Sep 17 00:00:00 2001
From: Anders Rognstad <anders.rognstad@nav.no>
Date: Mon, 4 Dec 2023 10:09:31 +0100
Subject: [PATCH] IS-1760: Always fetch from syfoperson

Replace tag with icon

fix icon and color

fix test

Adjust mock data
---
 mock/data/personoversiktEnhetMock.ts | 26 +++++++++++++-------------
 src/components/Labels.tsx            | 23 ++++++++++++-----------
 src/data/personregisterHooks.ts      | 14 ++++----------
 test/components/Personrad.test.tsx   |  3 +--
 4 files changed, 30 insertions(+), 36 deletions(-)

diff --git a/mock/data/personoversiktEnhetMock.ts b/mock/data/personoversiktEnhetMock.ts
index 3fcc80db..07458f15 100644
--- a/mock/data/personoversiktEnhetMock.ts
+++ b/mock/data/personoversiktEnhetMock.ts
@@ -7,7 +7,7 @@ import {
 export const personoversiktEnhetMock: PersonOversiktStatusDTO[] = [
   {
     fnr: '01999911111',
-    navn: '',
+    navn: 'Korrupt Heis',
     enhet: '0316',
     veilederIdent: null,
     motebehovUbehandlet: null,
@@ -27,7 +27,7 @@ export const personoversiktEnhetMock: PersonOversiktStatusDTO[] = [
   },
   {
     fnr: '99999922222',
-    navn: '',
+    navn: 'Korrupt Bordsen',
     enhet: '0316',
     veilederIdent: null,
     motebehovUbehandlet: true,
@@ -107,7 +107,7 @@ export const personoversiktEnhetMock: PersonOversiktStatusDTO[] = [
   },
   {
     fnr: '59999933333',
-    navn: '',
+    navn: 'Korrupt Bolle',
     enhet: '0316',
     veilederIdent: 'Z101010',
     motebehovUbehandlet: true,
@@ -149,7 +149,7 @@ export const personoversiktEnhetMock: PersonOversiktStatusDTO[] = [
   },
   {
     fnr: '99999944444',
-    navn: '',
+    navn: 'Stol Bordsen',
     enhet: '0316',
     veilederIdent: 'Z101010',
     motebehovUbehandlet: true,
@@ -187,7 +187,7 @@ export const personoversiktEnhetMock: PersonOversiktStatusDTO[] = [
   },
   {
     fnr: '18999955555',
-    navn: '',
+    navn: 'Bord Stolesen',
     enhet: '0316',
     veilederIdent: null,
     motebehovUbehandlet: true,
@@ -221,7 +221,7 @@ export const personoversiktEnhetMock: PersonOversiktStatusDTO[] = [
   },
   {
     fnr: '99999966666',
-    navn: '',
+    navn: 'Gulv Heisen',
     enhet: '0316',
     veilederIdent: 'M987654',
     motebehovUbehandlet: null,
@@ -251,7 +251,7 @@ export const personoversiktEnhetMock: PersonOversiktStatusDTO[] = [
   },
   {
     fnr: '99999955556',
-    navn: '',
+    navn: 'Skjerm Visen',
     enhet: '0316',
     veilederIdent: null,
     motebehovUbehandlet: true,
@@ -281,7 +281,7 @@ export const personoversiktEnhetMock: PersonOversiktStatusDTO[] = [
   },
   {
     fnr: '99999966667',
-    navn: '',
+    navn: 'Stol Sengestad',
     enhet: '0316',
     veilederIdent: 'M987654',
     motebehovUbehandlet: null,
@@ -311,7 +311,7 @@ export const personoversiktEnhetMock: PersonOversiktStatusDTO[] = [
   },
   {
     fnr: '99999966668',
-    navn: '',
+    navn: 'Bord Plantesen',
     enhet: '0316',
     veilederIdent: 'M987654',
     motebehovUbehandlet: null,
@@ -341,7 +341,7 @@ export const personoversiktEnhetMock: PersonOversiktStatusDTO[] = [
   },
   {
     fnr: '99999966669',
-    navn: '',
+    navn: 'Lun Gange',
     enhet: '0316',
     veilederIdent: 'Wienerbrød',
     motebehovUbehandlet: null,
@@ -375,7 +375,7 @@ export const personoversiktEnhetMock: PersonOversiktStatusDTO[] = [
   },
   {
     fnr: '99999966670',
-    navn: '',
+    navn: 'Vissen Plass',
     enhet: '0316',
     veilederIdent: 'Wienerbrød',
     motebehovUbehandlet: null,
@@ -405,7 +405,7 @@ export const personoversiktEnhetMock: PersonOversiktStatusDTO[] = [
   },
   {
     fnr: '99999966671',
-    navn: '',
+    navn: 'Mør Benk',
     enhet: '0316',
     veilederIdent: 'Z999991',
     motebehovUbehandlet: null,
@@ -435,7 +435,7 @@ export const personoversiktEnhetMock: PersonOversiktStatusDTO[] = [
   },
   {
     fnr: '99999966672',
-    navn: '',
+    navn: 'Grønn Due',
     enhet: '0316',
     veilederIdent: 'Z999991',
     motebehovUbehandlet: null,
diff --git a/src/components/Labels.tsx b/src/components/Labels.tsx
index adcf6b04..31706b5c 100644
--- a/src/components/Labels.tsx
+++ b/src/components/Labels.tsx
@@ -2,14 +2,11 @@ import React from 'react';
 import { PersonData } from '@/api/types/personregisterTypes';
 import { getReadableSkjermingskode } from '@/utils/personDataUtil';
 import styled from 'styled-components';
-import { Tag } from '@navikt/ds-react';
+import { Tooltip } from '@navikt/ds-react';
+import { ExclamationmarkTriangleFillIcon } from '@navikt/aksel-icons';
 
 const LabelColumnWrapper = styled.div`
-  padding: 0.5em 0;
-
-  > * {
-    margin: 0.2em;
-  }
+  display: flex;
 `;
 
 interface LabelColumnProps {
@@ -17,15 +14,19 @@ interface LabelColumnProps {
 }
 
 export const Labels = ({ personData }: LabelColumnProps) => {
-  const showSkjermingskode =
-    personData.skjermingskode && personData.skjermingskode !== 'INGEN';
+  const { skjermingskode } = personData;
+  const showSkjermingskode = skjermingskode && skjermingskode !== 'INGEN';
 
   return (
     <LabelColumnWrapper>
       {showSkjermingskode && (
-        <Tag variant="warning" size="small">
-          {getReadableSkjermingskode(personData.skjermingskode)}
-        </Tag>
+        <Tooltip content={getReadableSkjermingskode(skjermingskode)}>
+          <ExclamationmarkTriangleFillIcon
+            aria-hidden
+            fontSize="1.5rem"
+            color="var(--a-orange-600)"
+          />
+        </Tooltip>
       )}
     </LabelColumnWrapper>
   );
diff --git a/src/data/personregisterHooks.ts b/src/data/personregisterHooks.ts
index 7e42f7da..5e07b0f8 100644
--- a/src/data/personregisterHooks.ts
+++ b/src/data/personregisterHooks.ts
@@ -13,22 +13,16 @@ export const personregisterQueryKeys = {
 };
 
 export const usePersonregisterQuery = () => {
-  const personoversiktQuery = usePersonoversiktQuery();
+  const { data } = usePersonoversiktQuery();
   const { displayNotification, clearNotification } = useNotifications();
   const throwError = useAsyncError();
 
-  const fnrForPersonerUtenNavnListe =
-    personoversiktQuery.data &&
-    personoversiktQuery.data
-      .filter((p) => !p.navn)
-      .map((person) => ({
-        fnr: person.fnr,
-      }));
+  const fnrForPersonerListe = data.map((person) => ({ fnr: person.fnr }));
 
   const fetchPersonregister = () => {
     const personregisterData = post<PersonregisterData[]>(
       `${SYFOPERSON_ROOT}/person/info`,
-      fnrForPersonerUtenNavnListe || []
+      fnrForPersonerListe
     );
 
     return personregisterData || [];
@@ -37,7 +31,7 @@ export const usePersonregisterQuery = () => {
   return useQuery({
     queryKey: personregisterQueryKeys.personregister,
     queryFn: fetchPersonregister,
-    enabled: fnrForPersonerUtenNavnListe.length > 0,
+    enabled: fnrForPersonerListe.length > 0,
     onError: (error) => {
       if (error instanceof ApiErrorException && error.code === 403) {
         throwError(error);
diff --git a/test/components/Personrad.test.tsx b/test/components/Personrad.test.tsx
index b44c72de..70222f6c 100644
--- a/test/components/Personrad.test.tsx
+++ b/test/components/Personrad.test.tsx
@@ -93,7 +93,7 @@ describe('Personrad', () => {
     );
   });
 
-  it('Skal rendre riktig navn, fodselsnummer og skjermingskode', () => {
+  it('Skal rendre riktig navn og fodselsnummer', () => {
     renderPersonrad(defaultPersonData);
 
     expect(
@@ -102,7 +102,6 @@ describe('Personrad', () => {
       })
     ).to.exist;
     expect(screen.getByText(testdata.fnr1)).to.exist;
-    expect(screen.getByText('diskresjonsmerket')).to.exist;
   });
 
   it('Skal rendre frist-dato for aktivitetskrav AVVENT', () => {