Retrieve PEPITE with its id

server/api/pepite/index.js

@@ -7,6 +7,6 @@ module.exports = (options) => {
   var pepiteController = new Controller(options)
   router.get('/ping', pepiteController.ping)
   router.get('/', pepiteController.getAll)
-  router.get('/:id', pepiteController.getPepite)
+  router.get('/:id(\\d+)', pepiteController.getPepite)
   return router
 }

server/api/pepite/pepite.controller.js

@@ -1,7 +1,6 @@
 'use strict'
 
 const Pepite = require('./pepite.model')
-const mongoose = require('mongoose')
 
 class PepiteController {
   ping(req, res) {
@@ -20,9 +19,6 @@ class PepiteController {
   }
 
   getPepite(req, res) {
-    if (!mongoose.Types.ObjectId.isValid(req.params.id)) {
-      return res.sendStatus(404)
-    }
     return Pepite
       .findById(req.params.id).exec()
       .then((pepite) => {

server/api/pepite/test/pepite.api.test.js

@@ -31,4 +31,37 @@ describe('api: pepite', () => {
         })
     })
   })
+
+  describe('When requesting /api/pepite/:id', () => {
+    describe('When the id is valid and exists', () => {
+      it('should return a specific pepite', (done) => {
+        supertest(app)
+          .get('/api/pepite/1')
+          .expect(200)
+          .end((err, res) => {
+            if (err) {
+              return done(err)
+            }
+            expect(res.body.name).toEqual('ETENA')
+            done()
+          })
+      })
+    })
+
+    describe('When the id is valid but does not exist', () => {
+      it('should return a not found error', () => {
+        supertest(app)
+          .get('/api/pepite/42')
+          .expect(404)
+      })
+    })
+
+    describe('When the id is invalid', () => {
+      it('should return a bad request', () => {
+        supertest(app)
+          .get('/api/pepite/invalid_id')
+          .expect(400)
+      })
+    })
+  })
 })