From b734e481c1df8906c29b039e30813e00a8e5ff2b Mon Sep 17 00:00:00 2001 From: FrankRua Date: Tue, 1 Mar 2022 15:08:00 -0500 Subject: [PATCH 1/3] Fix defect for routing --- src/router/index.js | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/router/index.js b/src/router/index.js index 0d98f6698..aafc5e130 100644 --- a/src/router/index.js +++ b/src/router/index.js @@ -57,17 +57,20 @@ const routes = [ try { // If we already have our route, go to it. if (router.hasRoute(to.query.fmgPage)) { + // Since our route is already in scope, we can grab the component from it and call the arePagePrerequisitesValid function. - const component = router - .getRoutes() - .filter((x) => x.name === to.query.fmgPage)[0].components; + let component = router.getRoutes().filter((x) => x.name === to.query.fmgPage)[0].components; + + // If the component hasn't been loaded fully, load it before we check prerequisites. + if (component.default.methods == undefined) { + component = await component.default(); + } if (!arePagePrerequisitesValid(component)) { await GoToFunnelStartOn404(next); } - - return next({ name: to.query.fmgPage, query: to.query, params: to.params }); + return next({ name: to.query.fmgPage, query: to.query, params: to.params }); } // Get route info for the given url. Names will have a 1:1 relationship with names in the Cms. @@ -82,10 +85,7 @@ const routes = [ // Call the next components arePagePrerequisitesValid method before load. // If it returns false, use the 404 logic. - const nextComponent = await router - .getRoutes() - .filter((x) => x.name === routeData[0].name)[0] - .components.default(); + const nextComponent = await router.getRoutes().filter((x) => x.name === routeData[0].name)[0].components.default(); if (!arePagePrerequisitesValid(nextComponent)) { await GoToFunnelStartOn404(next); From 6596239d38ebf97bd2cddbbe84e9070de9884a17 Mon Sep 17 00:00:00 2001 From: FrankRua Date: Tue, 1 Mar 2022 15:09:18 -0500 Subject: [PATCH 2/3] one more = --- src/router/index.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/router/index.js b/src/router/index.js index aafc5e130..be434e9d8 100644 --- a/src/router/index.js +++ b/src/router/index.js @@ -62,7 +62,7 @@ const routes = [ let component = router.getRoutes().filter((x) => x.name === to.query.fmgPage)[0].components; // If the component hasn't been loaded fully, load it before we check prerequisites. - if (component.default.methods == undefined) { + if (component.default.methods === undefined) { component = await component.default(); } From 352c3c69b9a44486dc3ab5fa7375e59bbd932f5c Mon Sep 17 00:00:00 2001 From: Max Date: Wed, 2 Mar 2022 13:12:14 -0500 Subject: [PATCH 3/3] Radio button fix and radio button unit tests added --- src/ux-components/radio/radio.spec.js | 38 +++++++++++++++++++++++++++ src/ux-components/radio/radio.vue | 32 +++++++++++----------- 2 files changed, 55 insertions(+), 15 deletions(-) diff --git a/src/ux-components/radio/radio.spec.js b/src/ux-components/radio/radio.spec.js index a7c3318ed..113d534c6 100644 --- a/src/ux-components/radio/radio.spec.js +++ b/src/ux-components/radio/radio.spec.js @@ -60,4 +60,42 @@ describe("radio.vue", () => { expect(paragraph.text()).toEqual("screenreader text"); }); + + it("Should emit button value on click", async () => { + // Act + const wrapper = shallowMount(radio, { + propsData: { + buttonLabel: "Windshield", + value: "List Card Checkbox", + buttonID: "List Card Checkbox", + groupID: "radio-demo-1", + groupName: "radio 1", + isRequired: true, + isWide: false, + modelValue: ["List Card Checkbox"], + }, + }); + wrapper.vm.handleCheckChange(); + // Assert + expect(wrapper.emitted()["isCheckedChanged"][0]).toEqual([{"buttonID": "List Card Checkbox", value: "List Card Checkbox", checkValue: Boolean}]); + }); + + it("Should set checkValue data if selectedButtonIDs has value(s)", async () => { + // Act + const wrapper = shallowMount(radio, { + propsData: { + buttonLabel: "Windshield", + buttonID: "List Card Checkbox", + groupID: "radio-demo-1", + groupName: "radio 1", + buttonImage: "windshield-damage.svg", + isRequired: true, + modelValue: ["List Card Checkbox"], + value: "Car-Front", + selectedValues: ["Car-Front"] + }, + }); + // Assert + expect(wrapper.componentVM.checkValue).toEqual(true); + }); }); diff --git a/src/ux-components/radio/radio.vue b/src/ux-components/radio/radio.vue index 99629a39b..17fb7a880 100644 --- a/src/ux-components/radio/radio.vue +++ b/src/ux-components/radio/radio.vue @@ -10,7 +10,8 @@ :aria-required="isRequired" :value="value" :v-model="checkValue" - @change="handleCheckChanged()" + @change="handleCheckChange()" + :checked="checkValue" />