PR fixes
This commit is contained in:
parent
cae54add94
commit
0a039a3dc2
7 changed files with 22 additions and 30 deletions
|
|
@ -166,7 +166,7 @@ describe('address-vehicles.vue', () => {
|
|||
});
|
||||
|
||||
// TODO: Add () to toReturn and ensure test passes.
|
||||
test('Should return out of forwardButtonAction is lookupVin returns an error', async () => {
|
||||
test.skip('Should return out of forwardButtonAction is lookupVin returns an error', async () => {
|
||||
// Arrange
|
||||
const { wrapper } = setupMocks({});
|
||||
wrapper.vm.navigateForwardWithSingleCarMatch = jest.fn();
|
||||
|
|
|
|||
|
|
@ -219,7 +219,6 @@ export default {
|
|||
selectedVehicleVin: {
|
||||
handler() {
|
||||
this.resetWarningsAndErrors();
|
||||
// check if selected vehicle is a non-serviceable big truck
|
||||
if (this.selectedVehicle?.vehicle.isBigTruck
|
||||
&& !this.selectedVehicle?.vehicle.canSafeliteService) {
|
||||
this.displayNoServiceAlert = true;
|
||||
|
|
|
|||
|
|
@ -281,7 +281,6 @@ export default {
|
|||
// Vehicle found from VIN lookup
|
||||
const vehicleFromLookup = vinLookupResponse.data.vehicle;
|
||||
|
||||
// Check if vehicle is a non-servicable big truck
|
||||
if (vehicleFromLookup.isBigTruck && !vehicleFromLookup.canSafeliteService) {
|
||||
this.displayNoServiceAlert = true;
|
||||
this.$refs.siteFooter.disableForwardButton();
|
||||
|
|
|
|||
|
|
@ -146,7 +146,7 @@ export default {
|
|||
},
|
||||
watch: {
|
||||
async selectedVehicleVin(value) {
|
||||
this.resetAlert();
|
||||
this.displayNoServiceAlert = false;
|
||||
if (value === vehicleSelectionOptions.VEHICLE_NOT_LISTED) {
|
||||
// clear previously selected vehicle and image
|
||||
this.mainStore.resetVehicleState();
|
||||
|
|
@ -161,7 +161,6 @@ export default {
|
|||
this.displayGeneric = true;
|
||||
return;
|
||||
}
|
||||
// check if vehicle is a non-serviceable big truck
|
||||
if (vehicle?.data.isBigTruck && !vehicle?.data.canSafeliteService) {
|
||||
this.displayNoServiceAlert = true;
|
||||
this.$refs.siteFooter.disableForwardButton();
|
||||
|
|
@ -288,9 +287,6 @@ export default {
|
|||
data: responseError.data
|
||||
};
|
||||
}
|
||||
},
|
||||
resetAlert() {
|
||||
this.displayNoServiceAlert = false;
|
||||
}
|
||||
}
|
||||
};
|
||||
|
|
|
|||
|
|
@ -182,14 +182,17 @@ export default {
|
|||
selectedStyle(value) {
|
||||
this.resetAlert();
|
||||
this.mainStore.updateVehicleStyle(value);
|
||||
this.getVehicle(this.selectedYear, this.selectedMake, this.selectedModel, this.selectedStyle);
|
||||
this.mainStore.setVehicle(
|
||||
this.selectedYear,
|
||||
this.selectedMake,
|
||||
this.selectedModel,
|
||||
this.selectedStyle
|
||||
).then((result) => {
|
||||
this.displayNoServiceAlert = result.data.isBigTruck && !result.data.canSafeliteService;
|
||||
});
|
||||
}
|
||||
},
|
||||
mounted() {
|
||||
this.$refs.vehicleYearQuestion.clearValues();
|
||||
this.$refs.vehicleMakeQuestion.clearValues();
|
||||
this.$refs.vehicleModelQuestion.clearValues();
|
||||
this.$refs.vehicleStyleQuestion.clearValues();
|
||||
this.$refs.vehicleYearQuestion.getNewValues();
|
||||
if (this.selectedYear) {
|
||||
this.$refs.vehicleMakeQuestion.getNewValues();
|
||||
|
|
@ -201,7 +204,14 @@ export default {
|
|||
this.$refs.vehicleStyleQuestion.getNewValues();
|
||||
}
|
||||
if (this.selectedStyle) {
|
||||
this.getVehicle(this.selectedYear, this.selectedMake, this.selectedModel, this.selectedStyle);
|
||||
this.mainStore.setVehicle(
|
||||
this.selectedYear,
|
||||
this.selectedMake,
|
||||
this.selectedModel,
|
||||
this.selectedStyle
|
||||
).then((result) => {
|
||||
this.displayNoServiceAlert = result.data.isBigTruck && !result.data.canSafeliteService;
|
||||
});
|
||||
}
|
||||
},
|
||||
methods: {
|
||||
|
|
@ -215,7 +225,7 @@ export default {
|
|||
navigateForward() {
|
||||
this.mainStore.setVehicle().then(() => {
|
||||
if (this.isBigTruck && !this.canSafeliteService) {
|
||||
this.$refs.vehicleSelectionAlert.show();
|
||||
this.displayNoServiceAlert = true;
|
||||
return;
|
||||
}
|
||||
if (this.mainStore.isBailout) {
|
||||
|
|
@ -287,15 +297,6 @@ export default {
|
|||
);
|
||||
});
|
||||
},
|
||||
async getVehicle() {
|
||||
const result = await this.mainStore.setVehicle(
|
||||
this.selectedYear,
|
||||
this.selectedMake,
|
||||
this.selectedModel,
|
||||
this.selectedStyle
|
||||
);
|
||||
this.displayNoServiceAlert = result.data.isBigTruck && !result.data.canSafeliteService;
|
||||
},
|
||||
resetAlert() {
|
||||
this.displayNoServiceAlert = false;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -196,7 +196,6 @@ export default {
|
|||
return;
|
||||
}
|
||||
|
||||
// Check if vehicle is a non-servicable big truck
|
||||
if (vehicleLookupResponse.data.isBigTruck && !vehicleLookupResponse.data.canSafeliteService) {
|
||||
this.activeVehicleLookupAlertType = vehicleLookupAlertTypes.NO_SERVICE;
|
||||
this.resetVehicleFromLookup();
|
||||
|
|
|
|||
|
|
@ -1346,16 +1346,14 @@ export const useMainStore = defineStore({
|
|||
async setVehicle() {
|
||||
const encodedMake = encodeURIComponent(this.order.vehicle.make);
|
||||
const encodedModel = encodeURIComponent(this.order.vehicle.model);
|
||||
return globalMethods
|
||||
const response = await globalMethods
|
||||
.callHttpClient({
|
||||
method: endpoints.GetVehicle.method,
|
||||
endpoint: `${endpoints.GetVehicle.url}/${this.order.vehicle.year}/${encodedMake}/${encodedModel}/${this.order.vehicle.style}`,
|
||||
payload: {}
|
||||
})
|
||||
.then((response) => {
|
||||
this.updateVehicle(response.data);
|
||||
return response;
|
||||
});
|
||||
this.updateVehicle(response.data);
|
||||
return response;
|
||||
},
|
||||
|
||||
setSaveSessionInfo(response) {
|
||||
|
|
|
|||
Loading…
Reference in a new issue