Skip to content

Commit

Permalink
useTitle and test fix
Browse files Browse the repository at this point in the history
  • Loading branch information
IgorCapCoder committed Dec 15, 2023
1 parent 4920bd0 commit 5fb40ea
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 4 deletions.
4 changes: 4 additions & 0 deletions src/components/administration/ProvisioningOptionsPage.unit.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@ import * as routerComposables from "vue-router/composables";

jest.mock("@data-provisioning-options");

jest.mock<typeof import("@/utils/pageTitle")>("@/utils/pageTitle", () => ({
buildPageTitle: (pageTitle) => pageTitle ?? "",
}));

const $theme = {
name: "instance name",
};
Expand Down
11 changes: 7 additions & 4 deletions src/components/administration/ProvisioningOptionsPage.vue
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,8 @@ import {
import { useRouter } from "vue-router/composables";
import DefaultWireframe from "../templates/DefaultWireframe.vue";
import { Breadcrumb } from "../templates/default-wireframe.types";
import { buildPageTitle } from "../../utils/pageTitle";
import { buildPageTitle } from "@/utils/pageTitle";
import { useTitle } from "@vueuse/core";
export default defineComponent({
name: "ProvisioningOptionsPage",
Expand All @@ -112,6 +113,11 @@ export default defineComponent({
} = useProvisioningOptionsState();
const router = useRouter();
const pageTitle = buildPageTitle(
t("components.administration.provisioningOptions.page.title")
);
useTitle(pageTitle);
const schoolSettingsPage: Breadcrumb = {
text: t("pages.administration.school.index.title"),
to: "/administration/school-settings",
Expand All @@ -133,9 +139,6 @@ export default defineComponent({
);
onMounted(async () => {
document.title = buildPageTitle(
t("components.administration.provisioningOptions.page.title")
);
await fetchProvisioningOptionsData(props.systemId);
});
Expand Down

0 comments on commit 5fb40ea

Please sign in to comment.