fix the code according to TL comments

Relates #26

server/controllers/postFeedback.js

@@ -4,14 +4,14 @@ const feedbackSchema = require('../validation/feedback');
 module.exports = (req, res, next) => {
   feedbackSchema
     .validateAsync(req.body)
-    .then((data) => postFeedback(data))
+    .then(postFeedback)
     .then(() => {
-      res.status(200).send({ statusCode: 200, message: 'Your feedback has sent!' });
+      res.status(201).send({ statusCode: 201, message: 'Your feedback has sent!' });
     })
     .catch((err) => {
       if (err.details) {
         if (err.details[0].message) {
-          res.status(400).send({ statusCode: 400, message: err.details[0].message });
+          res.status(400).send({ statusCode: 400, error: err.details[0].message });
         }
       } else {
         next(err);

server/database/queries/postFeedback.js

@@ -3,7 +3,7 @@ const connection = require('../config/connection');
 module.exports = (data) => {
   const { orderId, email, feedback } = data;
   const sql = {
-    text: 'INSERT INTO feedback (order_id, email, feedback) values ($1, $2, $3);',
+    text: 'INSERT INTO feedback (order_id, email, feedback) VALUES ($1, $2, $3);',
     values: [orderId, email, feedback],
   };
   return connection.query(sql);

test/feedback.test.js

@@ -8,13 +8,13 @@ tape('testing post feedback route', (t) => {
     .post('/api/v1/post-feedback')
     .send({ orderId: 2, email: '[email protected]', feedback: 'Every thing is good' })
     .expect('Content-Type', /json/)
-    .expect(200)
+    .expect(201)
     .end((err, res) => {
       if (err) {
         t.error(err);
         t.end();
       } else {
-        t.deepEqual(res.body, { statusCode: 200, message: 'Your feedback has sent!' }, 'should be the same');
+        t.deepEqual(res.body, { statusCode: 201, message: 'Your feedback has sent!' }, 'should be the same');
         t.end();
       }
     });