Skip to content

Commit

Permalink
Sender metrikker med kilde Forebyggingsplan i stedet for Sykefraværss… (
Browse files Browse the repository at this point in the history
#177)

* Sender metrikker med kilde Forebyggingsplan i stedet for Sykefraværsstatistikk
  • Loading branch information
runska authored Nov 23, 2023
1 parent 402fd18 commit 0a89da3
Show file tree
Hide file tree
Showing 14 changed files with 55 additions and 54 deletions.
3 changes: 2 additions & 1 deletion src/Aktiviteter/context/aktivitetStatus.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ import {
AggregertStatistikkDto,
tomtDataobjekt,
} from "../../integrasjoner/aggregert-statistikk-api";
import {MetrikkKilde} from "@navikt/ia-metrikker-client";

const AktivitetContext = React.createContext<{
aktivitetStatuser: AktivitetBrukerStatus[];
Expand Down Expand Up @@ -109,7 +110,7 @@ export const useOppdaterStatus = (
(status: StatusType) => {
if (orgnr) {
oppdaterStatus(aktivitetId, orgnr, status);
sendIaMetrikkInteraksjonstjeneste(orgnr);
sendIaMetrikkInteraksjonstjeneste(MetrikkKilde.FOREBYGGINGSPLAN, orgnr);

setLokaleEndringer((tidligereEndringer) => {
const aktivitetIndex = tidligereEndringer.findIndex(
Expand Down
8 changes: 4 additions & 4 deletions src/Lenkeflis/Lenkeflis.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import userEvent from "@testing-library/user-event";
import {Lenkeflis} from "./Lenkeflis";
import logEvent from "../amplitude/logEvent";
import {axe} from "jest-axe";
import {sendIaTjenesteMetrikk} from "../integrasjoner/ia-tjenestemetrikker-api";
import {sendDigitalIaTjenesteMetrikk} from "../integrasjoner/ia-tjenestemetrikker-api";

jest.mock("../amplitude/logEvent");
jest.mock("../hooks/useOrgnr", () => ({
Expand All @@ -12,7 +12,7 @@ jest.mock("../hooks/useOrgnr", () => ({
jest.mock("../integrasjoner/ia-tjenestemetrikker-api", () => ({
__esModule: true,
...jest.requireActual("../integrasjoner/ia-tjenestemetrikker-api"),
sendIaTjenesteMetrikk: jest.fn(),
sendDigitalIaTjenesteMetrikk: jest.fn(),
}));

beforeEach(() => {
Expand Down Expand Up @@ -60,11 +60,11 @@ describe("Lenkeflis", () => {
name: "Link Klikk her",
});

expect(sendIaTjenesteMetrikk).toHaveBeenCalledTimes(0);
expect(sendDigitalIaTjenesteMetrikk).toHaveBeenCalledTimes(0);

await user.click(statistikklenke);

expect(sendIaTjenesteMetrikk).toHaveBeenCalledTimes(1);
expect(sendDigitalIaTjenesteMetrikk).toHaveBeenCalledTimes(1);
});

test("inneholder ikke uu-feil", async () => {
Expand Down
4 changes: 2 additions & 2 deletions src/Lenkeflis/Lenkeflis.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import {LinkPanel} from "@navikt/ds-react";
import styles from "./Lenkeflis.module.scss";
import React from "react";
import {sendNavigereEvent} from "../amplitude/events";
import {sendIaTjenesteMetrikk} from "../integrasjoner/ia-tjenestemetrikker-api";
import {sendDigitalIaTjenesteMetrikk} from "../integrasjoner/ia-tjenestemetrikker-api";
import {useOrgnr} from "../hooks/useOrgnr";
import {navigerEtterCallbacks} from "../utils/navigasjon";
import {MetrikkKilde} from "@navikt/ia-metrikker-client";
Expand All @@ -18,7 +18,7 @@ export const Lenkeflis = ({overskrift, ikon, href, brødtekst}: LenkeflisProps)
const orgnr = useOrgnr();
const destinasjon = href ?? "#";

const metrikkutsendelse = () => sendIaTjenesteMetrikk(MetrikkKilde.FOREBYGGE_FRAVÆR, orgnr);
const metrikkutsendelse = () => sendDigitalIaTjenesteMetrikk(MetrikkKilde.FOREBYGGE_FRAVÆR, orgnr);
const eventutsendelse = () => sendNavigereEvent(destinasjon, overskrift);

return (
Expand Down
8 changes: 4 additions & 4 deletions src/hooks/useSendIaTjenesteMetrikkOnEvent.test.tsx
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { render, screen } from "@testing-library/react";
import userEvent from "@testing-library/user-event";
import { sendIaTjenesteMetrikk } from "../../src/integrasjoner/ia-tjenestemetrikker-api";
import { sendDigitalIaTjenesteMetrikk } from "../integrasjoner/ia-tjenestemetrikker-api";

import { useSendIaTjenesteMetrikkOnEvent } from "./useSendIaTjenesteMetrikkOnEvent";
import { FunctionComponent } from "react";
Expand All @@ -13,7 +13,7 @@ jest.mock("../../src/integrasjoner/ia-tjenestemetrikker-api", () => {
return {
__esModule: true,
...jest.requireActual("../../src/integrasjoner/ia-tjenestemetrikker-api"),
sendIaTjenesteMetrikk: jest.fn(),
sendDigitalIaTjenesteMetrikk: jest.fn(),
};
});

Expand Down Expand Up @@ -42,11 +42,11 @@ it("sendLevertInnloggetIaTjeneste kalles når event blir trigget", async () => {

const dummyButton = screen.getByTestId("dummy-button");

expect(sendIaTjenesteMetrikk).not.toHaveBeenCalled();
expect(sendDigitalIaTjenesteMetrikk).not.toHaveBeenCalled();

await userEvent.click(dummyButton);

expect(sendIaTjenesteMetrikk).toHaveBeenCalled();
expect(sendDigitalIaTjenesteMetrikk).toHaveBeenCalled();
});

const UseSendIaTjenesteMetrikkerOnEventExample: FunctionComponent = () => {
Expand Down
4 changes: 2 additions & 2 deletions src/hooks/useSendIaTjenesteMetrikkOnEvent.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { useOrgnr } from "./useOrgnr";
import { useEffect, useRef } from "react";
import { sendIaTjenesteMetrikk } from "../integrasjoner/ia-tjenestemetrikker-api";
import { sendDigitalIaTjenesteMetrikk } from "../integrasjoner/ia-tjenestemetrikker-api";
import { MetrikkKilde } from "@navikt/ia-metrikker-client";

export function useSendIaTjenesteMetrikkOnEvent(
Expand All @@ -13,7 +13,7 @@ export function useSendIaTjenesteMetrikkOnEvent(
useEffect(() => {
if (orgnr && !eventListenerHarBlittSattOpp.current) {
const sendMetrikk = async () => {
await sendIaTjenesteMetrikk(fraIaTjeneste, orgnr);
await sendDigitalIaTjenesteMetrikk(fraIaTjeneste, orgnr);
};
document.addEventListener(event, sendMetrikk, {
once: true,
Expand Down
22 changes: 5 additions & 17 deletions src/integrasjoner/ia-tjenestemetrikker-api.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,8 @@ import {
sendIaMetrikk,
} from "@navikt/ia-metrikker-client";

export const sendIaTjenesteMetrikk = async (
kilde: MetrikkKilde,
export const sendDigitalIaTjenesteMetrikk = async (
tjeneste: MetrikkKilde,
orgnr?: string
) => {
if (!orgnr) {
Expand All @@ -15,31 +15,19 @@ export const sendIaTjenesteMetrikk = async (
return sendIaMetrikk(
orgnr,
MetrikkType.DIGITAL_IA_TJENESTE,
kilde,
tjeneste,
METRIKKER_URL
);
};

export const sendSykefraværsstatistikkIaMetrikk = async (orgnr?: string) => {
if (!orgnr) {
return Promise.reject("orgnr er udefinert");
}
return sendIaMetrikk(
orgnr,
MetrikkType.DIGITAL_IA_TJENESTE,
MetrikkKilde.SYKEFRAVÆRSSTATISTIKK,
METRIKKER_URL
);
};

export const sendIaMetrikkInteraksjonstjeneste = async (orgnr?: string) => {
export const sendIaMetrikkInteraksjonstjeneste = async (tjeneste: MetrikkKilde, orgnr?: string) => {
if (!orgnr) {
return Promise.reject("orgnr er udefinert");
}
return sendIaMetrikk(
orgnr,
MetrikkType.INTERAKSJONSTJENESTE,
MetrikkKilde.SYKEFRAVÆRSSTATISTIKK,
tjeneste,
METRIKKER_URL
);
};
8 changes: 4 additions & 4 deletions src/komponenter/Kalkulator/Kalkulator.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import { render, screen } from "@testing-library/react";
import userEvent from "@testing-library/user-event";
import { Fraværskalulator } from "./Kalkulator";
import { axe } from "jest-axe";
import { sendIaTjenesteMetrikk } from "../../integrasjoner/ia-tjenestemetrikker-api";
import { sendDigitalIaTjenesteMetrikk } from "../../integrasjoner/ia-tjenestemetrikker-api";
jest.mock("next/router", () => ({
useRouter() {
return {
Expand All @@ -23,7 +23,7 @@ jest.mock("next/router", () => ({
jest.mock("../../integrasjoner/ia-tjenestemetrikker-api", () => ({
__esModule: true,
...jest.requireActual("../../integrasjoner/ia-tjenestemetrikker-api"),
sendIaTjenesteMetrikk: jest.fn(),
sendDigitalIaTjenesteMetrikk: jest.fn(),
}));
jest.mock("../../hooks/useOrgnr", () => ({
useOrgnr: () => "999999999",
Expand All @@ -40,11 +40,11 @@ it("Kaller sendIaTjenesteMetrikk ved endring av modus", async () => {

const dagsverkLenke = screen.getByText("Dagsverk");

expect(sendIaTjenesteMetrikk).toHaveBeenCalledTimes(0);
expect(sendDigitalIaTjenesteMetrikk).toHaveBeenCalledTimes(0);

await user.click(dagsverkLenke);

expect(sendIaTjenesteMetrikk).toHaveBeenCalledTimes(1);
expect(sendDigitalIaTjenesteMetrikk).toHaveBeenCalledTimes(1);
});

it("Ingen uu-feil fra axe", async () => {
Expand Down
19 changes: 12 additions & 7 deletions src/metrikker.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import { Fraværskalulator } from "./komponenter/Kalkulator/Kalkulator";
import Home from "./pages";
import {
sendIaMetrikkInteraksjonstjeneste,
sendIaTjenesteMetrikk,
sendDigitalIaTjenesteMetrikk,
} from "./integrasjoner/ia-tjenestemetrikker-api";
import { RestStatus } from "./integrasjoner/rest-status";

Expand All @@ -28,7 +28,7 @@ jest.mock("next/router", () => ({
jest.mock("./integrasjoner/ia-tjenestemetrikker-api", () => ({
__esModule: true,
...jest.requireActual("./integrasjoner/ia-tjenestemetrikker-api"),
sendIaTjenesteMetrikk: jest.fn(),
sendDigitalIaTjenesteMetrikk: jest.fn(),
sendIaMetrikkInteraksjonstjeneste: jest.fn(),
}));

Expand Down Expand Up @@ -76,16 +76,16 @@ describe("Metrikktester av hele siden", () => {
tapteDagsverk="7800"
muligeDagsverk="52000"
nedlastingPågår={false}
/>
/>,
);

const dagsverkLenke = screen.getByText("Dagsverk");

expect(sendIaTjenesteMetrikk).toHaveBeenCalledTimes(0);
expect(sendDigitalIaTjenesteMetrikk).toHaveBeenCalledTimes(0);

await user.click(dagsverkLenke);

expect(sendIaTjenesteMetrikk).toHaveBeenCalledTimes(1);
expect(sendDigitalIaTjenesteMetrikk).toHaveBeenCalledTimes(1);
});
});

Expand All @@ -94,7 +94,7 @@ describe("Metrikktester av hele siden", () => {
renderPage();
const aktivitetHeader = await waitFor(() => {
const lenke = screen.getByText(
"Bli gode på å tilrettelegge for ansatte"
"Bli gode på å tilrettelegge for ansatte",
);

expect(lenke).toBeInTheDocument();
Expand All @@ -114,6 +114,11 @@ describe("Metrikktester av hele siden", () => {
await user.click(startKnapp);

expect(sendIaMetrikkInteraksjonstjeneste).toHaveBeenCalledTimes(1);
expect(sendIaMetrikkInteraksjonstjeneste).toHaveBeenNthCalledWith(
1,
"FOREBYGGINGSPLAN",
"999999999",
);
});

function renderPage() {
Expand All @@ -133,7 +138,7 @@ describe("Metrikktester av hele siden", () => {
minSideArbeidsgiverUrl="minSideArbeidsgiverUrl"
kjørerMockApp={kjørerMockApp}
grafanaAgentUrl="grafanaAgentUrl"
/>
/>,
);
}
});
Expand Down
5 changes: 3 additions & 2 deletions src/sykefravarsstatistikk/Forside/LastNedKnapp/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,8 @@ import React from "react";
import { DownloadIcon } from "@navikt/aksel-icons";
import styles from "./LastNedKnapp.module.css";
import { sendKnappEvent } from "../../../amplitude/events";
import { sendSykefraværsstatistikkIaMetrikk } from "../../../integrasjoner/ia-tjenestemetrikker-api";
import { sendDigitalIaTjenesteMetrikk } from "../../../integrasjoner/ia-tjenestemetrikker-api";
import { MetrikkKilde } from "@navikt/ia-metrikker-client";

export default function LastNedKnapp({
innholdRef,
Expand All @@ -18,7 +19,7 @@ export default function LastNedKnapp({
<ReactToPrint
onBeforePrint={() => {
sendKnappEvent("skriv ut");
sendSykefraværsstatistikkIaMetrikk(orgnr);
sendDigitalIaTjenesteMetrikk(MetrikkKilde.SYKEFRAVÆRSSTATISTIKK, orgnr);
}}
onAfterPrint={() => {
if (lastNedKnappRef.current) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,8 @@ import { getBransjeEllerNæringKategori } from "./GetBransjeEllerNæringKategori
import { Skeleton } from "@navikt/ds-react";
import { RestStatus } from "../../../integrasjoner/rest-status";
import { RestAggregertStatistikk } from "../../hooks/useSykefraværAppData";
import { sendSykefraværsstatistikkIaMetrikk } from "../../../integrasjoner/ia-tjenestemetrikker-api";
import { sendDigitalIaTjenesteMetrikk } from "../../../integrasjoner/ia-tjenestemetrikker-api";
import { MetrikkKilde } from "@navikt/ia-metrikker-client";

interface Props {
aggregertStatistikk: RestAggregertStatistikk;
Expand All @@ -22,7 +23,7 @@ export const Sammenligningspaneler: FunctionComponent<Props> = ({
useEffect(() => {
const timer = setTimeout(() => {
if (skalSendeMetrikkerAutomatisk) {
sendSykefraværsstatistikkIaMetrikk(orgnr);
sendDigitalIaTjenesteMetrikk(MetrikkKilde.SYKEFRAVÆRSSTATISTIKK, orgnr);
}
}, 5000);
return () => clearTimeout(timer);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ jest.mock("../../../../../integrasjoner/ia-tjenestemetrikker-api", () => ({
...jest.requireActual(
"../../../../../integrasjoner/ia-tjenestemetrikker-api",
),
sendSykefraværsstatistikkIaMetrikk: jest.fn(),
sendDigitalIaTjenesteMetrikk: jest.fn(),
}));

jest.mock("../../../../../amplitude/events", () => ({
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,8 @@ import {
sendCheckboxLagtTil,
} from "../../../../../amplitude/events";
import { useOrgnr } from "../../../../../hooks/useOrgnr";
import { sendSykefraværsstatistikkIaMetrikk } from "../../../../../integrasjoner/ia-tjenestemetrikker-api";
import { sendDigitalIaTjenesteMetrikk } from "../../../../../integrasjoner/ia-tjenestemetrikker-api";
import { MetrikkKilde } from "@navikt/ia-metrikker-client";

interface Props {
labels: HistorikkLabels;
Expand Down Expand Up @@ -41,7 +42,7 @@ export const LegendMedToggles: FunctionComponent<Props> = ({
legend="Velg linjer som skal vises i grafen"
value={linjerSomSkalVises}
onChange={(value) => {
sendSykefraværsstatistikkIaMetrikk(orgnr);
sendDigitalIaTjenesteMetrikk(MetrikkKilde.SYKEFRAVÆRSSTATISTIKK, orgnr);
if (value.length > linjerSomSkalVises.length) {
// Brukeren har lagt til noe
const verdiLagtTil = value.find(
Expand Down
9 changes: 5 additions & 4 deletions src/sykefravarsstatistikk/Historikk/Historikk.tsx
Original file line number Diff line number Diff line change
@@ -1,17 +1,18 @@
import React, { FunctionComponent, useState } from "react";
import styles from "./Historikk.module.css";
import {
Alert,
BodyShort,
ToggleGroup,
Heading,
Alert,
Skeleton,
ToggleGroup,
} from "@navikt/ds-react";
import GrafEllerTabell from "./GrafEllerTabell/GrafEllerTabell";
import { RestRessurs, RestStatus } from "../../integrasjoner/rest-status";
import { KvartalsvisSykefraværshistorikk } from "../hooks/useSykefraværAppData";
import { sendToogleEvent } from "../../amplitude/events";
import { sendSykefraværsstatistikkIaMetrikk } from "../../integrasjoner/ia-tjenestemetrikker-api";
import { sendDigitalIaTjenesteMetrikk } from "../../integrasjoner/ia-tjenestemetrikker-api";
import { MetrikkKilde } from "@navikt/ia-metrikker-client";

interface Props {
restSykefraværsstatistikk: RestRessurs<KvartalsvisSykefraværshistorikk[]>;
Expand Down Expand Up @@ -46,7 +47,7 @@ const Historikk: FunctionComponent<Props> = (props) => {
const grafEllerTabell = value as "graf" | "tabell";
setGrafEllerTabell(grafEllerTabell);
sendToogleEvent(grafEllerTabell);
sendSykefraværsstatistikkIaMetrikk(orgnr);
sendDigitalIaTjenesteMetrikk(MetrikkKilde.SYKEFRAVÆRSSTATISTIKK, orgnr)
}}
>
<ToggleGroup.Item value="graf">Graf</ToggleGroup.Item>
Expand Down
7 changes: 5 additions & 2 deletions src/sykefravarsstatistikk/Metrikker.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jest.mock("../hooks/useOrgnr", () => ({
jest.mock("../integrasjoner/ia-tjenestemetrikker-api", () => ({
__esModule: true,
...jest.requireActual("../integrasjoner/ia-tjenestemetrikker-api"),
sendSykefraværsstatistikkIaMetrikk: jest.fn(),
sendDigitalIaTjenesteMetrikk: jest.fn(),
}));

describe("Metrikkutsendelser", () => {
Expand All @@ -36,7 +36,7 @@ describe("Metrikkutsendelser", () => {
MockObserver.startmock();
sykefravarsSpy = jest.spyOn(
metrikker,
"sendSykefraværsstatistikkIaMetrikk"
"sendDigitalIaTjenesteMetrikk"
);
useOrgnrSpy = jest.spyOn(hooks, "useOrgnr");
useOrgnrSpy.mockReturnValue(
Expand Down Expand Up @@ -92,6 +92,8 @@ describe("Metrikkutsendelser", () => {
});

expect(sykefravarsSpy).toHaveBeenCalled();
expect(sykefravarsSpy).toHaveBeenNthCalledWith(1, "SYKEFRAVÆRSSTATISTIKK", "777777777")

jest.useRealTimers();
});

Expand All @@ -107,6 +109,7 @@ describe("Metrikkutsendelser", () => {
await waitFor(() => {
expect(sykefravarsSpy).toHaveBeenCalled();
});
expect(sykefravarsSpy).toHaveBeenNthCalledWith(1, "SYKEFRAVÆRSSTATISTIKK", "777777777")
});

it("Sender it-metrikk når feltere i historikkgrafen toggles", async () => {
Expand Down

0 comments on commit 0a89da3

Please sign in to comment.