Skip to content

Commit

Permalink
IS-2260: Filter inaktiv veileder
Browse files Browse the repository at this point in the history
  • Loading branch information
andersrognstad committed Jan 30, 2025
1 parent 15490ea commit 9a19cf4
Show file tree
Hide file tree
Showing 4 changed files with 32 additions and 2 deletions.
4 changes: 3 additions & 1 deletion src/components/TildeltVeileder.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,9 @@ const toVeilederOptions = (veiledere: Veileder[], aktivVeileder: Veileder) => {
...sortedVeiledere
.filter(
(veileder) =>
veileder.fornavn.length > 0 && veileder.etternavn.length > 0
veileder.enabled &&
veileder.fornavn.length > 0 &&
veileder.etternavn.length > 0
)
.filter((veileder) => veileder.ident !== aktivVeileder.ident)
.map(toVeilederOption),
Expand Down
4 changes: 4 additions & 0 deletions src/data/veilederinfo/types/Veileder.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ export interface Veileder {
etternavn: string;
epost: string;
telefonnummer?: string;
enabled: boolean | null;
}

export class Veileder {
Expand All @@ -12,19 +13,22 @@ export class Veileder {
etternavn: string;
epost: string;
telefonnummer?: string;
enabled: boolean | null;

constructor(
ident: string,
fornavn: string,
etternavn: string,
epost: string,
enabled: boolean | null,
telefonnummer?: string
) {
this.ident = ident;
this.fornavn = fornavn;
this.etternavn = etternavn;
this.epost = epost;
this.telefonnummer = telefonnummer;
this.enabled = enabled;
}

fulltNavn(): string {
Expand Down
13 changes: 13 additions & 0 deletions src/mocks/common/mockConstants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@ export const VEILEDER_DEFAULT = new Veileder(
"Vetle",
"Veileder",
"[email protected]",
true,
"12345678"
);

Expand All @@ -61,6 +62,18 @@ export const ANNEN_VEILEDER = new Veileder(
"Valdemar",
"Vaileder",
"[email protected]",
true,
"12345678"
);

export const INAKTIV_VEILEDER_IDENT = "Z980000";

export const INAKTIV_VEILEDER = new Veileder(
INAKTIV_VEILEDER_IDENT,
"Viktor",
"Villeder",
"[email protected]",
false,
"12345678"
);

Expand Down
13 changes: 12 additions & 1 deletion test/components/TildeltVeilederTest.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
import {
ANNEN_VEILEDER,
ARBEIDSTAKER_DEFAULT,
INAKTIV_VEILEDER,
VEILEDER_BRUKER_KNYTNING_DEFAULT,
VEILEDER_DEFAULT,
VEILEDER_IDENT_DEFAULT,
Expand Down Expand Up @@ -89,7 +90,7 @@ describe("TildeltVeileder", () => {
);
queryClient.setQueryData(
veilederinfoQueryKeys.veiledereByEnhet(navEnhet.id),
() => [VEILEDER_DEFAULT, ANNEN_VEILEDER]
() => [VEILEDER_DEFAULT, ANNEN_VEILEDER, INAKTIV_VEILEDER]
);
});
it("viser modal for endring av tildelt veileder", async () => {
Expand All @@ -110,6 +111,16 @@ describe("TildeltVeileder", () => {
})
).to.exist;
});
it("viser ikke inaktiv veileder i modal for endring av tildelt veileder", async () => {
renderTildelVeileder();
await clickButton("Endre");

expect(screen.getAllByRole("option")).to.have.length(2);
const inaktivVeilederOption = screen.queryByRole("option", {
name: `${INAKTIV_VEILEDER.etternavn}, ${INAKTIV_VEILEDER.fornavn}`,
});
expect(inaktivVeilederOption).to.not.exist;
});
it("validerer valgt veileder", async () => {
renderTildelVeileder();
await clickButton("Endre");
Expand Down

0 comments on commit 9a19cf4

Please sign in to comment.