From aa69a2c782c46371f4c1569c8310630d96e31d45 Mon Sep 17 00:00:00 2001
From: Sneha <sneha.a@safelite.com>
Date: Fri, 15 Dec 2023 16:05:42 +0530
Subject: [PATCH] CSR-1885

---
 src/layouts/vehicle/vehicle.vue | 18 +++++-----
 src/store/index.js              | 63 +++++++++++----------------------
 src/store/store.spec.js         |  6 ++--
 3 files changed, 33 insertions(+), 54 deletions(-)

diff --git a/src/layouts/vehicle/vehicle.vue b/src/layouts/vehicle/vehicle.vue
index 1b51fb9ca..bba7b89b8 100644
--- a/src/layouts/vehicle/vehicle.vue
+++ b/src/layouts/vehicle/vehicle.vue
@@ -216,9 +216,9 @@ export default {
                 styleQuestionInitialDataPromise
             )
                 vm.initializeMMSComponent(
-                    resultMap.makeQuestionInitialData?.response?.data,
-                    resultMap.modelQuestionInitialData?.response?.data,
-                    resultMap.styleQuestionInitialData?.response?.data
+                    resultMap.makeQuestionInitialData,
+                    resultMap.modelQuestionInitialData,
+                    resultMap.styleQuestionInitialData
                 );
         });
     },
@@ -227,8 +227,8 @@ export default {
         async selectedYear(year) {
             if (year) {
                 const result = await this.getMakeOptions(year);
-                if (result?.year == year) {
-                    this.makeOptions = result?.response?.data;
+                if (this.selectedYear == year) {
+                    this.makeOptions = result?.data;
                     if (this.selectedYearfromStore() !== year) {
                         this.selectedMake = null;
                         this.selectedModel = null;
@@ -247,8 +247,8 @@ export default {
         async selectedMake(make) {
             if (make) {
                 const result = await this.getModelOptions(this.selectedYear, make);
-                if (decodeURIComponent(result?.make) == make) {
-                    this.modelOptions = result?.response?.data;
+                if (this.selectedMake == make) {
+                    this.modelOptions = result?.data;
                     if (this.modelOptions.length === 1) this.selectedModel = this.modelOptions[0];
                     else {
                         this.selectedModel = null;
@@ -270,8 +270,8 @@ export default {
                     this.selectedMake,
                     model
                 );
-                if (decodeURIComponent(result?.model) == model) {
-                    this.styleOptions = result?.response?.data;
+                if (this.selectedModel == model) {
+                    this.styleOptions = result?.data;
                     if (this.styleOptions.length === 1) {
                         const sameStyle = this.selectedStyle === this.styleOptions[0];
                         this.selectedStyle = this.styleOptions[0];
diff --git a/src/store/index.js b/src/store/index.js
index dd5c556b2..673c4dfa1 100644
--- a/src/store/index.js
+++ b/src/store/index.js
@@ -854,54 +854,33 @@ export const actions = {
     },
 
     getVehicleMakes(context, { payload: { year }, pageNameToLog }) {
-        return globalMethods
-            .callHttpClient({
-                method: endpoints.GetVehicleMakes.method,
-                endpoint: `${endpoints.GetVehicleMakes.url}/${year}`,
-                payload: {},
-                logApiCall: true,
-                pageNameToLog: pageNameToLog,
-            })
-            .then((response) => {
-                return {
-                    response,
-                    year,
-                };
-            });
+        return globalMethods.callHttpClient({
+            method: endpoints.GetVehicleMakes.method,
+            endpoint: `${endpoints.GetVehicleMakes.url}/${year}`,
+            payload: {},
+            logApiCall: true,
+            pageNameToLog: pageNameToLog,
+        });
     },
 
     getVehicleModels(context, { payload: { year, make }, pageNameToLog }) {
-        return globalMethods
-            .callHttpClient({
-                method: endpoints.GetVehicleModels.method,
-                endpoint: `${endpoints.GetVehicleModels.url}/${year}/${make}`,
-                payload: {},
-                logApiCall: true,
-                pageNameToLog: pageNameToLog,
-            })
-            .then((response) => {
-                return {
-                    response,
-                    make,
-                };
-            });
+        return globalMethods.callHttpClient({
+            method: endpoints.GetVehicleModels.method,
+            endpoint: `${endpoints.GetVehicleModels.url}/${year}/${make}`,
+            payload: {},
+            logApiCall: true,
+            pageNameToLog: pageNameToLog,
+        });
     },
 
     getVehicleStyles(context, { payload: { year, make, model }, pageNameToLog }) {
-        return globalMethods
-            .callHttpClient({
-                method: endpoints.GetVehicleStyles.method,
-                endpoint: `${endpoints.GetVehicleStyles.url}/${year}/${make}/${model}`,
-                payload: {},
-                logApiCall: true,
-                pageNameToLog: pageNameToLog,
-            })
-            .then((response) => {
-                return {
-                    response,
-                    model,
-                };
-            });
+        return globalMethods.callHttpClient({
+            method: endpoints.GetVehicleStyles.method,
+            endpoint: `${endpoints.GetVehicleStyles.url}/${year}/${make}/${model}`,
+            payload: {},
+            logApiCall: true,
+            pageNameToLog: pageNameToLog,
+        });
     },
 
     getVehicle(context, { payload: { year, make, model, style }, pageNameToLog }) {
diff --git a/src/store/store.spec.js b/src/store/store.spec.js
index f5ce39cf6..8353ec0d8 100644
--- a/src/store/store.spec.js
+++ b/src/store/store.spec.js
@@ -493,7 +493,7 @@ describe("Actions", () => {
             pageNameToLog: "test",
         });
 
-        expect(response.response.data).toEqual(["Acura", "Honda"]);
+        expect(response.data).toEqual(["Acura", "Honda"]);
     });
 
     it("getVehicleModels action, should return models list", async () => {
@@ -511,7 +511,7 @@ describe("Actions", () => {
             pageNameToLog: "test",
         });
 
-        expect(response.response.data).toEqual(["ILX", "RDX"]);
+        expect(response.data).toEqual(["ILX", "RDX"]);
     });
 
     it("getVehicleStyles action, should return models list", async () => {
@@ -529,7 +529,7 @@ describe("Actions", () => {
             pageNameToLog: "test",
         });
 
-        expect(response.response.data).toEqual({ style: "4 DOOR SEDAN" });
+        expect(response.data).toEqual({ style: "4 DOOR SEDAN" });
     });
 
     it("getVehicle action, should get vehicle data ", async () => {