From 01cde2f86cec5edfe182735ba88d8da40b29d635 Mon Sep 17 00:00:00 2001 From: Josh Dassinger Date: Thu, 7 Mar 2024 11:29:26 -0600 Subject: [PATCH] SSR-600 Updated unit tests for duplicate changes --- .../duplicate-check/duplicate-check.spec.js | 16 ++++++++++------ src/layouts/duplicate-check/duplicate-check.vue | 14 +++++++------- 2 files changed, 17 insertions(+), 13 deletions(-) diff --git a/src/layouts/duplicate-check/duplicate-check.spec.js b/src/layouts/duplicate-check/duplicate-check.spec.js index 13d00c9f..22c60c38 100644 --- a/src/layouts/duplicate-check/duplicate-check.spec.js +++ b/src/layouts/duplicate-check/duplicate-check.spec.js @@ -136,7 +136,8 @@ describe('duplicateCheck.vue', () => { expect(wrapper.vm.duplicateOrders[0]).toStrictEqual({ Text: duplicateOrderText, Name: referralNumber, - SubText: expectedSubtext + SubText: expectedSubtext, + value: useMainStore().applicationUser.duplicateOrders[0] }); }); test('duplicateOrder in store with null vehicle make => returns order with only date subtext', () => { @@ -173,7 +174,8 @@ describe('duplicateCheck.vue', () => { expect(wrapper.vm.duplicateOrders[0]).toStrictEqual({ Text: duplicateOrderText, Name: referralNumber, - SubText: expectedSubtext + SubText: expectedSubtext, + value: useMainStore().applicationUser.duplicateOrders[0] }); }); test('duplicateOrder in store with null vehicle model => returns order with only date subtext', () => { @@ -210,7 +212,8 @@ describe('duplicateCheck.vue', () => { expect(wrapper.vm.duplicateOrders[0]).toStrictEqual({ Text: duplicateOrderText, Name: referralNumber, - SubText: expectedSubtext + SubText: expectedSubtext, + value: useMainStore().applicationUser.duplicateOrders[0] }); }); test('duplicateOrder in store with all vehicle info => returns order with year, make, model and date in subtext', () => { @@ -248,7 +251,8 @@ describe('duplicateCheck.vue', () => { expect(wrapper.vm.duplicateOrders[0]).toStrictEqual({ Text: duplicateOrderText, Name: referralNumber, - SubText: `${expectedVehicle}, ${expectedDate}` + SubText: `${expectedVehicle}, ${expectedDate}`, + value: useMainStore().applicationUser.duplicateOrders[0] }); }); }); @@ -352,7 +356,7 @@ describe('duplicateCheck.vue', () => { test('Selected duplicate => load session called', async () => { // Arrange const newOrderSelectionName = getRandomString(6, 6); - const selectedAnswer = getRandomString(6, 6); + const selectedAnswer = {}; const mountOptions = getMountOptions({ router: { navigate: jest.fn() } @@ -405,7 +409,7 @@ describe('duplicateCheck.vue', () => { test('Load session throws error => still navigate forward', async () => { // Arrange const newOrderSelectionName = getRandomString(6, 6); - const selectedAnswer = getRandomString(6, 6); + const selectedAnswer = {}; const mountOptions = getMountOptions({ router: { navigate: jest.fn() } diff --git a/src/layouts/duplicate-check/duplicate-check.vue b/src/layouts/duplicate-check/duplicate-check.vue index cceb4fb4..4a842ba1 100644 --- a/src/layouts/duplicate-check/duplicate-check.vue +++ b/src/layouts/duplicate-check/duplicate-check.vue @@ -127,16 +127,16 @@ export default { * @summary Steps to perform when forward button clicked. */ async forwardButtonAction() { - if (typeof this.selectedAnswer === 'string' && this.selectedAnswer === this.getNewOrderSelectionName) { - this.navigateForward(); + if (this.selectedAnswer !== null && typeof this.selectedAnswer === 'object') { + await useMainStore().loadSession(this.selectedAnswer) + .catch(() => {}) + .finally(() => { + this.navigateForward(); + }); return; } - await useMainStore().loadSession(this.selectedAnswer) - .catch(() => {}) - .finally(() => { - this.navigateForward(); - }); + this.navigateForward(); }, navigateForward() { if (!this.mainStore.order.policy.policyLookupSuccessful) {