fixed some tests

This commit is contained in:
Jason Wheeler 2022-12-08 15:40:50 -05:00
parent 619a7619d2
commit a15b7fb0de
3 changed files with 24 additions and 7 deletions

View file

@ -1,4 +1,5 @@
import { navigationScenarios } from "@/router/router-constants/navigation-scenarios.js"; import { navigationScenarios } from "@/router/router-constants/navigation-scenarios.js";
import { RouterLinkStub } from "@vue/test-utils";
import { vehicleCategories } from "@/constants/vehicle-categories.js"; import { vehicleCategories } from "@/constants/vehicle-categories.js";
import { issPageValues } from "@/router/router-constants/issPage-values"; import { issPageValues } from "@/router/router-constants/issPage-values";
import { cookieNames } from "@/constants/cookie-names"; import { cookieNames } from "@/constants/cookie-names";
@ -52,7 +53,10 @@ export function getMountOptions(mockData) {
mocks: mocks, mocks: mocks,
mixins: [mockMixin], mixins: [mockMixin],
plugins: [pinia], plugins: [pinia],
stubs: { Form } stubs: {
Form,
RouterLink: RouterLinkStub,
}
}; };
return { global }; return { global };

View file

@ -6,11 +6,15 @@ import baseMixin from "@/mixins/base-mixin.js";
import { getMountOptions } from "@/helpers/unit-test-helper.js"; import { getMountOptions } from "@/helpers/unit-test-helper.js";
import { nextTick } from "vue"; import { nextTick } from "vue";
import { useMainStore } from "@/store"; import { useMainStore } from "@/store";
import router from "@/router";
// Components // Components
import vehicleStyle from "@/layouts/vehicle-style/vehicle-style.vue"; import vehicleStyle from "@/layouts/vehicle-style/vehicle-style.vue";
import styleQuestion from "@/layouts/vehicle-style/style-question/style-question"; import styleQuestion from "@/layouts/vehicle-style/style-question/style-question";
jest.mock("@/router", () => ({
overrideNavigation: jest.fn(),
}));
// Mock our module for promises. // Mock our module for promises.
jest.mock("@/helpers/layout-helper.js", () => ({ jest.mock("@/helpers/layout-helper.js", () => ({
@ -86,7 +90,8 @@ describe("vehicle-style.vue", () => {
test("Model set, arePagePrerequisitesValid should be true ", () => { test("Model set, arePagePrerequisitesValid should be true ", () => {
//Arrange //Arrange
const { wrapper } = setupMocks({}); const { wrapper } = setupMocks({ });
useMainStore().order.vehicle = { year: 2011, make: "ford", model: "mustang", style: null } useMainStore().order.vehicle = { year: 2011, make: "ford", model: "mustang", style: null }
//Act //Act
@ -122,8 +127,10 @@ describe("vehicle-style.vue", () => {
// Assert // Assert
expect(useMainStore().updateVehicleStyle).toHaveBeenCalledWith("2 door sedan"); expect(useMainStore().updateVehicleStyle).toHaveBeenCalledWith("2 door sedan");
expect(router.overrideNavigation).toHaveBeenCalled();
}); });
test("there is only one vehicle style and vehicle-damage was visited => don't autoselect or move to vehicle damage", async () => { test("there is only one vehicle style and vehicle-damage was visited => don't autoselect or move to vehicle damage", async () => {
//Arrange //Arrange
const { wrapper } = setupMocks({ const { wrapper } = setupMocks({
@ -151,6 +158,7 @@ describe("vehicle-style.vue", () => {
// Assert // Assert
expect(useMainStore().updateVehicleStyle).not.toHaveBeenCalledWith("2 door sedan"); expect(useMainStore().updateVehicleStyle).not.toHaveBeenCalledWith("2 door sedan");
expect(router.overrideNavigation).toHaveBeenCalled();
}); });
}); });
@ -188,6 +196,7 @@ function setupMocks({
initializeComponent: jest.fn(), initializeComponent: jest.fn(),
}; };
const mountOptions = getMountOptions(mountOptionsMockData); const mountOptions = getMountOptions(mountOptionsMockData);
const wrapper = shallowMount(vehicleStyle, mountOptions); const wrapper = shallowMount(vehicleStyle, mountOptions);

View file

@ -1,5 +1,6 @@
import { shallowMount } from "@vue/test-utils"; import { shallowMount, RouterLinkStub } from "@vue/test-utils";RouterLinkStub
import { getMountOptions } from "@/helpers/unit-test-helper.js"; import { getMountOptions } from "@/helpers/unit-test-helper.js";
import alert from "./alert"; import alert from "./alert";
describe("alert.vue", () => { describe("alert.vue", () => {
@ -60,12 +61,11 @@ describe("alert.vue", () => {
manualHeadline: "testHeader", manualHeadline: "testHeader",
manualCopy: "testCopy with a {routerLink: testName, testLink} inside of it", manualCopy: "testCopy with a {routerLink: testName, testLink} inside of it",
cmsWidgetName: "alert", cmsWidgetName: "alert",
}, }
stubs: ["router-link"],
}) })
); );
// Assert // Assert
expect(wrapper.find("router-link").exists()).toBe(true); expect(wrapper.findComponent(RouterLinkStub).exists()).toBe(true);
}); });
it("Should contain 'n+1' <p> tags if the body copy has 'n' <p> tags", () => { it("Should contain 'n+1' <p> tags if the body copy has 'n' <p> tags", () => {
@ -79,7 +79,6 @@ describe("alert.vue", () => {
"<p>testCopy with a {routerLink: testName, testLink} inside of it</p><p>and two paragraphs</p>", "<p>testCopy with a {routerLink: testName, testLink} inside of it</p><p>and two paragraphs</p>",
cmsWidgetName: "alert", cmsWidgetName: "alert",
}, },
stubs: ["router-link"],
}) })
); );
// Assert // Assert
@ -150,6 +149,7 @@ describe("alert.vue", () => {
shouldScrollToOnMount: false, shouldScrollToOnMount: false,
manualHeadline: "testHeader", manualHeadline: "testHeader",
manualCopy: "testCopy", manualCopy: "testCopy",
cmsWidgetName: "alert",
}, },
}) })
); );
@ -193,6 +193,9 @@ const mockMixin = {
dynamicStrings: jest.fn(() => { dynamicStrings: jest.fn(() => {
return { ROUTER_LINK: "routerLink:" }; return { ROUTER_LINK: "routerLink:" };
}), }),
cssClassNameForCmsWidget(){
return "widget-name-";
},
}, },
}; };
@ -215,6 +218,7 @@ function setupMocks(mountOptionsMockData = {}) {
propsData: { propsData: {
manualHeadline: "testHeader", manualHeadline: "testHeader",
manualCopy: "testCopy", manualCopy: "testCopy",
cmsWidgetName: "alert",
}, },
mixins: [mockMixin], mixins: [mockMixin],
}; };