hotfix: bookings fetch

Author: Víctor Martínez

src/controllers/bookings-controller.js

@@ -5,8 +5,8 @@ async function bookProperty(req, res, next) {
     const { uid } = req.user;
     const { propertyId, contactInfo } = req.body;
     try {
-        const property = await getPropertyById(propertyId);
-
+        const response = await getPropertyById(propertyId);
+        const property = response.data;
         const booking = await db.Bookings.create({
             clientId: uid,
             employeeId: property.employee_id,

src/controllers/properties-controller.js

@@ -4,10 +4,7 @@ async function getProperty(req, res, next) {
     const { propertyId } = req.params;
     try {
         const response = await getPropertyById(propertyId);
-        res.status(200).send({
-            data: response,
-            error: null
-        });
+        res.status(200).send(response);
     } catch (err) {
         next(err);
     }
@@ -16,10 +13,7 @@ async function getProperty(req, res, next) {
 async function searchProperties(req, res, next) {
     try {
         const response = await searchAdminProperties(req.query);
-        res.status(200).send({
-            data: response,
-            error: null
-        });
+        res.status(200).send(response);
     } catch (err) {
         next(err);
     }

src/routes/__tests__/bookings-routes.test.js

@@ -11,7 +11,7 @@ const setupTestDB = require("../../mock/seedTestDB");
 const mockHome = setupTestDB.getHome();
 
 jest.mock("../../utils/properties", () => ({
-    getPropertyById: () => mockHome
+    getPropertyById: () => ({ data: mockHome, error: null })
 }))
 
 const request = supertest(app);
@@ -42,7 +42,7 @@ describe("Private bookings routes", () => {
         expect(booking.status).toBe("pending");
         expect(booking.contactInfo).toMatchObject(postData.contactInfo);
     });
-    
+
     it("can get my bookings", async () => {
         //insert into db test_property with create property
         const res = await request.get("/bookings/all");

src/routes/__tests__/properties-routes.test.js

@@ -51,7 +51,7 @@ describe("Properties routes", () => {
         });
 
         const res = await request.get("/properties?kind=Office");
-        expect(res.body.data).toMatchObject([{ hello: "World" }]);
+        expect(res.body).toMatchObject([{ hello: "World" }]);
     });
 
     it("can get property by Id", async () => {
@@ -63,6 +63,6 @@ describe("Properties routes", () => {
         });
 
         const res = await request.get("/properties/sdflksdgkn");
-        expect(res.body.data).toMatchObject({ hello: "World" });
+        expect(res.body).toMatchObject({ hello: "World" });
     });
 });