Prettified

This commit is contained in:
Leah Schumann 2023-03-09 13:55:42 -05:00
parent 8ee9952538
commit 99faf4d067
5 changed files with 49 additions and 21 deletions

View file

@ -1,7 +1,12 @@
import { storeActions } from "@/constants/store-actions"; import { storeActions } from "@/constants/store-actions";
import baseMixin from "@/mixins/base-mixin.js"; import baseMixin from "@/mixins/base-mixin.js";
export async function getPricedMobileFeePart(serviceZipCode, serviceType, parentAccountNumber, billToAccountNumber) { export async function getPricedMobileFeePart(
serviceZipCode,
serviceType,
parentAccountNumber,
billToAccountNumber
) {
if (!serviceZipCode || !serviceType || !parentAccountNumber) { if (!serviceZipCode || !serviceType || !parentAccountNumber) {
return Promise.resolve(null); return Promise.resolve(null);
} }
@ -14,7 +19,7 @@ export async function getPricedMobileFeePart(serviceZipCode, serviceType, parent
{ {
serviceType: serviceType, serviceType: serviceType,
parentAccountNumber: parentAccountNumber, parentAccountNumber: parentAccountNumber,
billToAccountNumber: billToAccountNumber billToAccountNumber: billToAccountNumber,
}, },
false false
); );

View file

@ -48,13 +48,18 @@ describe("service-location-helper.js", () => {
it("Should return null if no service zip code is passed in", async () => { it("Should return null if no service zip code is passed in", async () => {
// Arrange // Arrange
const serviceZipCode = null; const serviceZipCode = null;
const serviceType = "Replace"; const serviceType = "Replace";
const parentAccountNumber = 167132 const parentAccountNumber = 167132;
const billToAccountNumber = 1234 const billToAccountNumber = 1234;
const expected = null; const expected = null;
// Act // Act
const result = await getPricedMobileFeePart(serviceZipCode, serviceType, parentAccountNumber, billToAccountNumber); const result = await getPricedMobileFeePart(
serviceZipCode,
serviceType,
parentAccountNumber,
billToAccountNumber
);
// Assert // Assert
expect(result).toEqual(expected); expect(result).toEqual(expected);
@ -63,9 +68,9 @@ describe("service-location-helper.js", () => {
it("Should return the priced mobile fee part", async () => { it("Should return the priced mobile fee part", async () => {
// Arrange // Arrange
const serviceZipCode = "43235"; const serviceZipCode = "43235";
const serviceType = "Replace"; const serviceType = "Replace";
const parentAccountNumber = 167132 const parentAccountNumber = 167132;
const billToAccountNumber = 1234 const billToAccountNumber = 1234;
const expected = { const expected = {
partNumber: "MOBILE FEE", partNumber: "MOBILE FEE",
@ -77,7 +82,12 @@ describe("service-location-helper.js", () => {
}; };
// Act // Act
const result = await getPricedMobileFeePart(serviceZipCode, serviceType, parentAccountNumber, billToAccountNumber); const result = await getPricedMobileFeePart(
serviceZipCode,
serviceType,
parentAccountNumber,
billToAccountNumber
);
// Assert // Assert
expect(result).toEqual(expected); expect(result).toEqual(expected);

View file

@ -187,8 +187,8 @@ export default {
this.resetComponent({ this.resetComponent({
state: newValue.state, state: newValue.state,
zipCode: newValue.zipCode zipCode: newValue.zipCode,
}) });
}, },
deep: true, deep: true,
}, },

View file

@ -32,11 +32,14 @@ const mockGetPricedMobileFeePart = (mockServiceZipCode) => {
return Promise.resolve(mobileFeePart); return Promise.resolve(mobileFeePart);
}; };
jest.mock("@/layouts/service-location/helpers/service-location-helper/service-location-helper", () => ({ jest.mock(
getPricedMobileFeePart: jest.fn((mockServiceZipCode) => { "@/layouts/service-location/helpers/service-location-helper/service-location-helper",
return mockGetPricedMobileFeePart(mockServiceZipCode); () => ({
}), getPricedMobileFeePart: jest.fn((mockServiceZipCode) => {
})); return mockGetPricedMobileFeePart(mockServiceZipCode);
}),
})
);
jest.mock("@/store", () => ({ jest.mock("@/store", () => ({
commit: jest.fn(), commit: jest.fn(),
@ -134,7 +137,7 @@ beforeEach(() => {
}, },
}, },
damage: { damage: {
isRepair: false isRepair: false,
}, },
payment: { payment: {
isInsurance: false, isInsurance: false,
@ -224,7 +227,7 @@ describe("service-location.vue", () => {
test("updates the page model after providing the service zip code", () => { test("updates the page model after providing the service zip code", () => {
// Arrange // Arrange
const { wrapper } = setupMocks({ const { wrapper } = setupMocks({
mixins: [mockMixin] mixins: [mockMixin],
}); });
wrapper.vm.$refs.mobileLocationModalQuestions.resetComponent = jest.fn(); wrapper.vm.$refs.mobileLocationModalQuestions.resetComponent = jest.fn();

View file

@ -64,7 +64,12 @@ export default {
const parentAccountNumber = store.getters.payment.parentAccountNumber; const parentAccountNumber = store.getters.payment.parentAccountNumber;
const billToAccountNumber = 0; const billToAccountNumber = 0;
const mobileFeePartPromise = getPricedMobileFeePart(serviceZipCode, serviceType, parentAccountNumber, billToAccountNumber); const mobileFeePartPromise = getPricedMobileFeePart(
serviceZipCode,
serviceType,
parentAccountNumber,
billToAccountNumber
);
// Settle promises and get results // Settle promises and get results
const promiseResultMap = [ const promiseResultMap = [
@ -157,7 +162,12 @@ export default {
const parentAccountNumber = store.getters.payment.parentAccountNumber; const parentAccountNumber = store.getters.payment.parentAccountNumber;
const billToAccountNumber = 0; const billToAccountNumber = 0;
getPricedMobileFeePart(serviceZipCode, serviceType, parentAccountNumber, billToAccountNumber).then((pricedMobileFeePart) => { getPricedMobileFeePart(
serviceZipCode,
serviceType,
parentAccountNumber,
billToAccountNumber
).then((pricedMobileFeePart) => {
this.mobileFeePart = pricedMobileFeePart; this.mobileFeePart = pricedMobileFeePart;
}); });
}, },