Skip to content

Commit

Permalink
Merge pull request #24 from akhilome/ft-make-new-order-160239612
Browse files Browse the repository at this point in the history
#160239612 User should be able to make a new order
  • Loading branch information
akhilome committed Sep 7, 2018
2 parents af4ba77 + eea4290 commit 469ab5c
Show file tree
Hide file tree
Showing 6 changed files with 135 additions and 42 deletions.
128 changes: 86 additions & 42 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
},
"homepage": "https://github.com/akhilome/fast-food-fast#readme",
"dependencies": {
"body-parser": "^1.18.3",
"express": "^4.16.3"
},
"devDependencies": {
Expand Down
17 changes: 17 additions & 0 deletions server/controllers/orderController.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import orders from '../db/orders';
import Order from '../models/Order';

class OrderController {
static getAllOrders(req, res) {
Expand All @@ -16,6 +17,22 @@ class OrderController {

return res.status(200).json(order);
}

static newOrder(req, res) {
const { author, title } = req.body;
if (!author || !title) {
return res.status(400).json({
error: 'incomplete data',
});
}

const order = new Order(orders.length + 1, author, title);
orders.push(order);
return res.status(201).json({
message: 'order placed',
order,
});
}
}

export default OrderController;
3 changes: 3 additions & 0 deletions server/index.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import express from 'express';
import bodyParser from 'body-parser';
import router from './routes/routes';

const app = express();
Expand All @@ -9,6 +10,8 @@ app.get('/', (req, res) => {
});
});

app.use(bodyParser.json());
app.use(bodyParser.urlencoded({ extended: true }));
app.use('/api/v1', router);

app.listen(3000);
Expand Down
1 change: 1 addition & 0 deletions server/routes/routes.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,5 +12,6 @@ router.get('/', (req, res) => {

router.get('/orders', OrderController.getAllOrders);
router.get('/orders/:id', findOrder, OrderController.getOrder);
router.post('/orders', OrderController.newOrder);

export default router;
27 changes: 27 additions & 0 deletions tests/routes/routes.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -99,3 +99,30 @@ describe('GET /api/v1/orders/<orderId>', () => {
});
});
});

describe('POST /api/v1/orders', () => {
const completeData = { author: 'Kizito', title: 'Turkey' };
const incompleteData = { author: 'Kizito' };

it('should return a 400 error and a message if data is incomplete', (done) => {
chai.request(app)
.post('/api/v1/orders')
.send(incompleteData)
.end((err, res) => {
res.should.have.status(400);
res.body.should.be.an('object').which.has.a.property('error');
done();
});
});

it('should add the order to the database and respond with 201 if data is complete', (done) => {
chai.request(app)
.post('/api/v1/orders')
.send(completeData)
.end((err, res) => {
res.should.have.status(201);
res.body.should.be.an('object').which.has.a.property('message');
done();
});
});
});

0 comments on commit 469ab5c

Please sign in to comment.