fix: creating a item should return the newly created value.

fixes nodeshift-starters#79

Author: lholmquist

lib/api/fruits.js

@@ -11,7 +11,7 @@ function findAll () {
 }
 
 function create (name, stock) {
-  return db.query('INSERT INTO products (name, stock) VALUES ($1, $2)', [name, stock]);
+  return db.query('INSERT INTO products (name, stock) VALUES ($1, $2) RETURNING *', [name, stock]);
 }
 
 function update (options = {}) {

lib/routes/fruits.js

@@ -29,8 +29,9 @@ router.get('/fruits', (request, response) => {
 
 router.post('/fruits', validations.validateCreate, (request, response) => {
   const {name, stock} = request.body;
-  return fruits.create(name, stock).then(() => {
-    response.sendStatus(201);
+  return fruits.create(name, stock).then((result) => {
+    response.status(201);
+    return response.send(result.rows[0]);
   }).catch((err) => {
     response.status(400);
     response.send(err);

test/fruits-test.js

@@ -145,7 +145,7 @@ test('test POST fruit', (t) => {
     create: (name, stock) => {
       t.equal(name, fruitData.name, `respone.body.name should be ${fruitData.name}`);
       t.equal(stock, fruitData.stock, `respone.body.stock should be ${fruitData.stock}`);
-      return Promise.resolve();
+      return Promise.resolve({rows: []});
     }
   };