Merge branch 'master' into searchFilter

server/api.test.js

@@ -36,3 +36,35 @@ describe('/api/users', () => {
     )
   })
 })
+
+
+describe('/api/products', () => {
+    it('GET /', () =>
+      request(app)
+        .get(`/api/products/1`)
+        .expect(200)
+    )
+
+    it('POST creates a user', () =>
+      request(app)
+        .post('/api/users')
+        .send({
+          email: '[email protected]',
+          password: '12345'
+        })
+        .expect(201)
+    )
+
+    it('POST redirects to the user it just made', () =>
+      request(app)
+        .post('/api/users')
+        .send({
+          email: '[email protected]',
+          password: '23456',
+        })
+        .redirects(1)
+        .then(res => expect(res.body).to.contain({
+          email: '[email protected]'
+        }))
+    )
+})

server/products.js

@@ -26,7 +26,7 @@ customProductsRoutes.get('/:productId', function(request,response,next){
     }
   })
   .then((product)=>{
-    response.json(product)
+    response.status(200).json(product)
   })
   .catch(next);
 })

server/users.js

@@ -19,14 +19,10 @@ customUserRoutes.post('/', function (request, response, next) {
         email: request.body.email,
         password: request.body.password
     }
-
- User.create(newUserToCreate)
- .then((result) => {
-   response.sendStatus(201);
- }).catch(function(err){
-   console.log(err)
-   next();
- });
+   User.create(newUserToCreate)
+     .then((user) => {
+       response.sendStatus(201);
+     }).catch(next);
 });
 
 customUserRoutes.delete('/:userId', function (request, response, next) {