Merge pull request #887 from Safelite/feature/CSR-803-refactoring-2

CSR-938: fix vehicle-parts back button

src/common-components/button-question/button-question.spec.js

@@ -2,14 +2,6 @@ import { shallowMount, mount } from "@vue/test-utils";
 import buttonQuestion from "@/common-components/button-question/button-question";
 import { getMountOptions } from "@/helpers/unit-test-helper.js";
 
-jest.mock(
-    "@/store",
-    () => {
-        return {};
-    },
-    { virtual: true }
-);
-
 describe("buttonQuestion.vue", () => {
     it("Should show overflow classes on fieldset if isOverflowScrollable is true", () => {
         // Act

src/common-components/dropdown-question/dropdown-question.spec.js

@@ -23,8 +23,6 @@ describe("dropdownQuestion.vue", () => {
             mixins: [mockMixin],
         });
 
-        wrapper.getCmsContent = jest.fn();
-
         // Act
         const select = wrapper.find("select");
 

src/layouts/vehicle-parts/vehicle-parts.vue

@@ -35,7 +35,7 @@
                     cmsWidgetName="FunnelFooterWidget"
                     ref="funnelFooter"
                     :isForwardActionDisabled="isForwardActionDisabled"
-                    @back-click="navigateBack"
+                    @back-clicked="navigateBack"
                     @ForwardClicked="forwardButtonAction" />
             </div>
         </div>

src/ux-components/button-main/button-main.spec.js

@@ -4,89 +4,77 @@ import { getMountOptions } from "@/helpers/unit-test-helper.js";
 import { nextTick } from "vue";
 
 describe("buttonMain.vue", () => {
-    it("Should return btn-primary class", async () => {
+    it("Should return btn-primary class", () => {
-        // Act
+        // Arrange/Act
         const wrapper = shallowMount(
             buttonMain,
             setupMocks({
-                propsData: {
+                props: {
                     isPrimary: true,
                 },
             })
         );
-
-        // Assert
         const button = wrapper.find("button");
 
-        // Expect
+        // Assert
         expect(button.attributes("class")).toContain("btn-primary");
     });
 
-    it("Should return aria-disabled state", async () => {
+    it("Should return aria-disabled state", () => {
-        // Act
+        // Arrange/Act
         const wrapper = shallowMount(
             buttonMain,
             setupMocks({
-                propsData: {
+                props: {
                     isDisabled: true,
                 },
             })
         );
-
-        // Assert
         const button = wrapper.find("button");
 
-        // Expect
+        // Assert
         expect(button.attributes()["aria-disabled"]).toEqual("true");
     });
 
     it("Should return loader color", async () => {
-        // Act
+        // Arrange
         const wrapper = shallowMount(
             buttonMain,
             setupMocks({
-                propsData: {
+                props: {
                     loaderColor: "blue",
                     loaderEnabled: true,
                 },
             })
         );
 
-        // Assert
+        // Act
-
-        const label = wrapper.find("label");
-
         wrapper.vm.clicked();
-
         await nextTick();
 
+        // Assert
         const loader = wrapper.find("loader-stub");
-
         expect(loader.attributes("class")).toContain("blue");
     });
 
     it("Should return loader position", async () => {
-        // Act
+        // Arrange
         const wrapper = shallowMount(
             buttonMain,
             setupMocks({
-                propsData: {
+                props: {
                     loaderPosition: "right",
                     loaderEnabled: true,
                 },
             })
         );
 
-        // Assert
+        // Act
-
-        const label = wrapper.find("label");
-
         wrapper.vm.clicked();
-
         await nextTick();
 
+        // Assert
         const loader = wrapper.find("loader-stub");
-
         expect(loader.attributes("class")).toContain("right");
     });
 });

src/ux-components/loader/loader.spec.js

@@ -1 +1,31 @@
-test.todo("some test to be written in the future");
+import { shallowMount, mount } from "@vue/test-utils";
+import loader from "./loader";
+
+describe("loader.vue", () => {
+    test("if it rendered the HTML element with class", () => {
+        // Arrange/Act
+        const wrapper = shallowMount(loader, {
+            props: {},
+        });
+
+        // Assert
+        expect(wrapper.find("div").exists()).toBeTruthy();
+        expect(wrapper.find(".loader").exists()).toBeTruthy();
+    });
+
+    test("if it correctly passed props", () => {
+        // Arrange/Act
+        const wrapper = shallowMount(loader, {
+            props: {
+                loaderColor: "blue",
+                loaderPosition: "left",
+            },
+        });
+
+        // Assert
+        expect(wrapper.props()).toMatchObject({
+            loaderColor: "blue",
+            loaderPosition: "left",
+        });
+    });
+});