From 415200ea5ab50ba89684f5be227c1dbe24ad7b5a Mon Sep 17 00:00:00 2001 From: Scott Kiener Date: Wed, 29 Jun 2022 17:11:00 -0400 Subject: [PATCH] CSR-700 | Update Unit Tests --- .../navigation-helper.spec.js | 10 ++++-- .../heritage-integration/order-helper.spec.js | 31 ++++++++++++------- src/helpers/unit-test-helper.js | 6 ++-- src/store/store.spec.js | 15 +++++++-- 4 files changed, 44 insertions(+), 18 deletions(-) diff --git a/src/helpers/heritage-integration/navigation-helper.spec.js b/src/helpers/heritage-integration/navigation-helper.spec.js index e212284b8..e78782d7c 100644 --- a/src/helpers/heritage-integration/navigation-helper.spec.js +++ b/src/helpers/heritage-integration/navigation-helper.spec.js @@ -308,8 +308,11 @@ describe("navigateToHeritageFunnel", () => { const mockReferralNumber = "2"; const mockCorrelationId = "55"; const mockReferralDate = "2022"; + const mockAccountNumber = "167132"; + const mockSaveQuoteId = "xxx-xxx-xxx"; + const mockCrmCustomerId = "xxx-xxx-xxx" - const mockOrderInfo = getMockOrderInfo(mockReferralNumber, mockCorrelationId, mockReferralDate); + const mockOrderInfo = getMockOrderInfo(mockReferralNumber, mockCorrelationId, mockReferralDate, mockAccountNumber, mockSaveQuoteId, mockCrmCustomerId); const mockData = { actionList: [{ @@ -340,8 +343,11 @@ describe("navigateToHeritageFunnel", () => { const mockReferralNumber = "2"; const mockCorrelationId = "55"; const mockReferralDate = "2022"; + const mockAccountNumber = "167132"; + const mockSaveQuoteId = "xxx-xxx-xxx"; + const mockCrmCustomerId = "xxx-xxx-xxx" - const mockOrderInfo = getMockOrderInfo(mockReferralNumber, mockCorrelationId, mockReferralDate); + const mockOrderInfo = getMockOrderInfo(mockReferralNumber, mockCorrelationId, mockReferralDate, mockAccountNumber, mockSaveQuoteId, mockCrmCustomerId); const mockData = { actionList: [{ diff --git a/src/helpers/heritage-integration/order-helper.spec.js b/src/helpers/heritage-integration/order-helper.spec.js index 7d769a8d8..c3a581e41 100644 --- a/src/helpers/heritage-integration/order-helper.spec.js +++ b/src/helpers/heritage-integration/order-helper.spec.js @@ -111,9 +111,11 @@ describe("saveOrder", () => { const mockReferralNumber = "2"; const mockCorrelationId = "55"; const mockReferralDate = "2022"; - const mockAccountNumber = "5"; + const mockAccountNumber = "167132"; + const mockSaveQuoteId = "xxx-xxx-xxx"; + const mockCrmCustomerId = "xxx-xxx-xxx" - const mockOrderInfo = getMockOrderInfo(mockReferralNumber, mockCorrelationId, mockReferralDate, mockAccountNumber); + const mockOrderInfo = getMockOrderInfo(mockReferralNumber, mockCorrelationId, mockReferralDate, mockAccountNumber, mockSaveQuoteId, mockCrmCustomerId); const mockData = { actionList: [ @@ -122,7 +124,7 @@ describe("saveOrder", () => { data: mockOrderInfo, }, { - actionName: storeActions.SET_REFERRAL_INFORMATION + actionName: storeActions.UPDATE_STORE_WITH_SAVE_ORDER_RESPONSE } ] } @@ -134,21 +136,26 @@ describe("saveOrder", () => { // Assert expect(mocks.baseMixin.methods.dispatchStoreAction).toHaveBeenCalledWith(storeActions.SAVE_ORDER); - expect(mocks.baseMixin.methods.dispatchStoreAction).toHaveBeenCalledWith(storeActions.SET_REFERRAL_INFORMATION, { + expect(mocks.baseMixin.methods.dispatchStoreAction).toHaveBeenCalledWith(storeActions.UPDATE_STORE_WITH_SAVE_ORDER_RESPONSE, { referralNumber: mockReferralNumber, referralDate: mockReferralDate, referralCorrelationId: mockCorrelationId, - accountNumber: mockAccountNumber + accountNumber: mockAccountNumber, + saveQuoteId: mockSaveQuoteId, + crmCustomerId: mockCrmCustomerId, }, false); }); test("saveOrder => should update DidHeritageFunnelUpdateLast cookie value to false", async () => { // Arrange - const testReferralNumber = 1566818; - const testReferralDate = "2022-03-15T10:56:24.597"; - const testReferralCorrelationId = "404d2b04-f86e-45c3-b373-127b6217b060"; + const mockReferralNumber = 1566818; + const mockReferralDate = "2022-03-15T10:56:24.597"; + const mockReferralCorrelationId = "404d2b04-f86e-45c3-b373-127b6217b060"; + const mockAccountNumber = "167132"; + const mockSaveQuoteId = "xxx-xxx-xxx"; + const mockCrmCustomerId = "xxx-xxx-xxx" - const mockOrderInfo = getMockOrderInfo(testReferralNumber, testReferralCorrelationId, testReferralDate); + const mockOrderInfo = getMockOrderInfo(mockReferralNumber, mockReferralCorrelationId, mockReferralDate, mockAccountNumber, mockSaveQuoteId, mockCrmCustomerId); const mockData = { actionList: [{ @@ -161,9 +168,9 @@ describe("saveOrder", () => { setupMocksForJsFiles(mockData); const testCookieValue = { - ReferralNumber: testReferralNumber, - ReferralDate: testReferralDate, - ReferralCorrelationId: testReferralCorrelationId, + ReferralNumber: mockReferralNumber, + ReferralDate: mockReferralDate, + ReferralCorrelationId: mockReferralCorrelationId, ShouldResetState: false, DidHeritageFunnelUpdateLast: true } diff --git a/src/helpers/unit-test-helper.js b/src/helpers/unit-test-helper.js index 592af9a5c..d4f38a64e 100644 --- a/src/helpers/unit-test-helper.js +++ b/src/helpers/unit-test-helper.js @@ -90,12 +90,14 @@ export function removeAllTestCookies() { }); } -export function getMockOrderInfo(mockReferralNumber, mockCorrelationId, mockReferralDate, accountNumber = "0") { +export function getMockOrderInfo(mockReferralNumber, mockCorrelationId, mockReferralDate, accountNumber = "0", saveQuoteId, crmCustomerId) { return { referralNumber: mockReferralNumber, referralCorrelationId: mockCorrelationId, referralDate: mockReferralDate, - accountNumber: accountNumber + accountNumber: accountNumber, + saveQuoteId: saveQuoteId, + crmCustomerId: crmCustomerId, } } diff --git a/src/store/store.spec.js b/src/store/store.spec.js index c70629296..78a6a8d1d 100644 --- a/src/store/store.spec.js +++ b/src/store/store.spec.js @@ -611,7 +611,7 @@ describe("Actions", () => { expect(commit).toBeCalledWith(storeMutations.UPDATE_STATE_WITH_ORDER_INFORMATION, {"referralNumber": 123}); }); - it("setReferralInformation, should call commit three times", () => { + it("updateStoreWithSaveOrderResponse, should call commit six times", () => { // Arrange const context = state; const commit = jest.fn(); @@ -619,12 +619,23 @@ describe("Actions", () => { context.commit = commit; // Act - actions.setReferralInformation(context, {referralNumber: "123", referralDate: new Date().toUTCString(), referralCorrelationId: "xxx-xxx-xxx"}); + actions.updateStoreWithSaveOrderResponse(context, + { + referralNumber: "123", + referralDate: new Date().toUTCString(), + referralCorrelationId: "xxx-xxx-xxx", + accountNumber: "167132", + saveQuoteId: "xxx-xxx-xxx", + crmCustomerId: "xxx-xxx-xxx", + }); // Assert expect(commit).toBeCalledWith(storeMutations.UPDATE_REFERRAL_NUMBER, "123"); expect(commit).toBeCalledWith(storeMutations.UPDATE_REFERRAL_DATE, new Date().toUTCString()); expect(commit).toBeCalledWith(storeMutations.UPDATE_REFERRAL_CORRELATION_ID, "xxx-xxx-xxx"); + expect(commit).toBeCalledWith(storeMutations.UPDATE_PARENT_ACCT_NUMBER, "167132"); + expect(commit).toBeCalledWith(storeMutations.UPDATE_SAVE_QUOTE_ID, "xxx-xxx-xxx"); + expect(commit).toBeCalledWith(storeMutations.UPDATE_CRM_CUSTOMER_ID, "xxx-xxx-xxx"); }); it("logPageView action, should return nothing", async () => {