From a15b7fb0deaa4dd9d528b40f0583fa0b29bad9bc Mon Sep 17 00:00:00 2001
From: Jason Wheeler <jason.wheeler@safelite.com>
Date: Thu, 8 Dec 2022 15:40:50 -0500
Subject: [PATCH] fixed some tests

---
 src/helpers/unit-test-helper.js                 |  6 +++++-
 src/layouts/vehicle-style/vehicle-style.spec.js | 11 ++++++++++-
 src/ux-components/alert/alert.spec.js           | 14 +++++++++-----
 3 files changed, 24 insertions(+), 7 deletions(-)

diff --git a/src/helpers/unit-test-helper.js b/src/helpers/unit-test-helper.js
index e9ffeceb..4cee713a 100644
--- a/src/helpers/unit-test-helper.js
+++ b/src/helpers/unit-test-helper.js
@@ -1,4 +1,5 @@
 import { navigationScenarios } from "@/router/router-constants/navigation-scenarios.js";
+import { RouterLinkStub } from "@vue/test-utils";
 import { vehicleCategories } from "@/constants/vehicle-categories.js";
 import { issPageValues } from "@/router/router-constants/issPage-values";
 import { cookieNames } from "@/constants/cookie-names";
@@ -52,7 +53,10 @@ export function getMountOptions(mockData) {
       mocks: mocks,
       mixins: [mockMixin],
       plugins: [pinia],
-      stubs: { Form }
+      stubs: { 
+        Form, 
+        RouterLink: RouterLinkStub, 
+        }
     };
   
     return { global };
diff --git a/src/layouts/vehicle-style/vehicle-style.spec.js b/src/layouts/vehicle-style/vehicle-style.spec.js
index 5ce29bbb..6a383572 100644
--- a/src/layouts/vehicle-style/vehicle-style.spec.js
+++ b/src/layouts/vehicle-style/vehicle-style.spec.js
@@ -6,11 +6,15 @@ import baseMixin from "@/mixins/base-mixin.js";
 import { getMountOptions } from "@/helpers/unit-test-helper.js";
 import { nextTick } from "vue";
 import { useMainStore } from "@/store";
+import router from "@/router";
 
 // Components
 import vehicleStyle from "@/layouts/vehicle-style/vehicle-style.vue";
 import styleQuestion from "@/layouts/vehicle-style/style-question/style-question";
 
+jest.mock("@/router", () => ({
+    overrideNavigation: jest.fn(),
+}));
 
 // Mock our module for promises.
 jest.mock("@/helpers/layout-helper.js", () => ({
@@ -86,7 +90,8 @@ describe("vehicle-style.vue", () => {
 
     test("Model set, arePagePrerequisitesValid should be true ", () => {
         //Arrange
-        const { wrapper } = setupMocks({});
+        const { wrapper } = setupMocks({ });
+
         useMainStore().order.vehicle = { year: 2011, make: "ford", model: "mustang", style: null }
 
         //Act
@@ -122,8 +127,10 @@ describe("vehicle-style.vue", () => {
 
         // Assert
         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 () => {
         //Arrange
         const { wrapper } = setupMocks({
@@ -151,6 +158,7 @@ describe("vehicle-style.vue", () => {
 
         // Assert
         expect(useMainStore().updateVehicleStyle).not.toHaveBeenCalledWith("2 door sedan");
+        expect(router.overrideNavigation).toHaveBeenCalled();
     });
 });
 
@@ -188,6 +196,7 @@ function setupMocks({
         initializeComponent: jest.fn(),
     };
 
+    
     const mountOptions = getMountOptions(mountOptionsMockData);
     const wrapper = shallowMount(vehicleStyle, mountOptions);
 
diff --git a/src/ux-components/alert/alert.spec.js b/src/ux-components/alert/alert.spec.js
index fcccbf4f..a4596d19 100644
--- a/src/ux-components/alert/alert.spec.js
+++ b/src/ux-components/alert/alert.spec.js
@@ -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 alert from "./alert";
 
 describe("alert.vue", () => {
@@ -60,12 +61,11 @@ describe("alert.vue", () => {
                     manualHeadline: "testHeader",
                     manualCopy: "testCopy with a {routerLink: testName, testLink} inside of it",
                     cmsWidgetName: "alert",
-                },
-                stubs: ["router-link"],
+                }
             })
         );
         // 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", () => {
@@ -79,7 +79,6 @@ describe("alert.vue", () => {
                         "<p>testCopy with a {routerLink: testName, testLink} inside of it</p><p>and two paragraphs</p>",
                     cmsWidgetName: "alert",
                 },
-                stubs: ["router-link"],
             })
         );
         // Assert
@@ -150,6 +149,7 @@ describe("alert.vue", () => {
                     shouldScrollToOnMount: false,
                     manualHeadline: "testHeader",
                     manualCopy: "testCopy",
+                    cmsWidgetName: "alert",
                 },
             })
         );
@@ -193,6 +193,9 @@ const mockMixin = {
         dynamicStrings: jest.fn(() => {
             return { ROUTER_LINK: "routerLink:" };
         }),
+        cssClassNameForCmsWidget(){
+            return "widget-name-";
+        },
     },
 };
 
@@ -215,6 +218,7 @@ function setupMocks(mountOptionsMockData = {}) {
         propsData: {
             manualHeadline: "testHeader",
             manualCopy: "testCopy",
+            cmsWidgetName: "alert",
         },
         mixins: [mockMixin],
     };