From ea0613c9132734e126a9f0b362d9050f9c262643 Mon Sep 17 00:00:00 2001 From: CarlNation Date: Mon, 9 May 2022 15:17:46 -0400 Subject: [PATCH 1/3] CSR-520 remove close modal gold plating --- .../loading-modal/loading-modal.spec.js | 56 +------------------ .../loading-modal/loading-modal.vue | 27 +-------- src/layouts/address-lookup/address-lookup.vue | 2 +- .../license-plate-lookup.vue | 2 +- src/layouts/vin-lookup/vin-lookup.vue | 2 +- 5 files changed, 7 insertions(+), 82 deletions(-) diff --git a/src/common-components/loading-modal/loading-modal.spec.js b/src/common-components/loading-modal/loading-modal.spec.js index ca7717cca..7a500ce73 100644 --- a/src/common-components/loading-modal/loading-modal.spec.js +++ b/src/common-components/loading-modal/loading-modal.spec.js @@ -17,7 +17,7 @@ jest.mock('@/assets/img/windshield.png', () => 'windshield.png') describe("loadingModal", () => { test("showModal sets modal visible", async () => { // Arrange - const { wrapper } = setupMocks({ clickOutCloses: false }); + const { wrapper } = setupMocks(); wrapper.vm.isModalVisible = false; //Act @@ -30,54 +30,7 @@ describe("loadingModal", () => { }); -describe("loadingModal", () => { - test("closeModal sets modal not visible", async () => { - // Arrange - const { wrapper } = setupMocks({ clickOutCloses: false }); - wrapper.vm.isModalVisible = true; - - //Act - wrapper.vm.closeModal(); - - // Assert - expect(wrapper.vm.isModalVisible).toEqual(false); - wrapper.unmount(); - }); -}); - -describe("loadingModal", () => { - test("canClose calls closeModal", async () => { - // Arrange - const { wrapper } = setupMocks({ clickOutCloses: true }); - wrapper.vm.isModalVisible = true; - - //Act - wrapper.vm.canClose(); - - // Assert - expect(wrapper.vm.isModalVisible).toEqual(false); - wrapper.unmount(); - }); -}); - -describe("loadingModal", () => { - test("canClose does not call closeModal", async () => { - // Arrange - const { wrapper } = setupMocks({ clickOutCloses: false }); - wrapper.vm.isModalVisible = true; - - //Act - wrapper.vm.canClose(); - - // Assert - expect(wrapper.vm.isModalVisible).toEqual(true); - wrapper.unmount(); - }); -}); - -function setupMocks({ - clickOutCloses -}) { +function setupMocks() { //Mock store store.dispatch = jest.fn(() => {}); @@ -89,9 +42,6 @@ function setupMocks({ }, }); - //Mock props - mountOptions.propsData = { clickOutCloses: clickOutCloses }; const wrapper = shallowMount(loadingModal, mountOptions); - return { wrapper }; -} +} \ No newline at end of file diff --git a/src/common-components/loading-modal/loading-modal.vue b/src/common-components/loading-modal/loading-modal.vue index 4f1365802..9665c6ea5 100644 --- a/src/common-components/loading-modal/loading-modal.vue +++ b/src/common-components/loading-modal/loading-modal.vue @@ -1,7 +1,6 @@