Implement feedback

server/controllers/DocumentController.js

@@ -110,7 +110,7 @@ class DocumentController {
         });
       })
       .catch((error) => {
-        res.status(400).json(error);
+        res.status(500).json({ message: 'An Error Ocurred', error });
       });
   }
 
@@ -129,7 +129,7 @@ class DocumentController {
         });
       }
       return res.status(200).json(document);
-    }).catch(error => res.status(400).json(error));
+    }).catch(error => res.status(500).json({ message: 'An Error Ocurred', error }));
   }
 
   /**
@@ -140,11 +140,16 @@ class DocumentController {
    */
   static update(req, res) {
     return Document.findById(req.params.documentId).then((document) => {
+      if (!document) {
+        return res.status(404).json({
+          message: 'Document Not Found'
+        });
+      }
       return document
         .update(req.body)
         .then(() => res.status(200).json(document))
         .catch(error => res.status(400).json(error));
-    });
+    }).catch(error => res.status(500).json({ message: 'An Error Ocurred', error }));
   }
 
   /**
@@ -155,10 +160,15 @@ class DocumentController {
    */
   static delete(req, res) {
     return Document.findById(req.params.documentId).then((document) => {
+      if (!document) {
+        return res.status(404).json({
+          message: 'Document Not Found'
+        });
+      }
       return document.destroy()
       .then(() => res.send(200))
-      .catch(error => res.status(400).json(error));
-    });
+      .catch(error => res.status(500).json({ message: 'An Error Ocurred', error }));
+    }).catch(error => res.status(500).json({ message: 'An Error Ocurred', error }));
   }
 }
 export default DocumentController;

server/controllers/RoleController.js

@@ -29,7 +29,7 @@ class RoleController {
   static list(req, res) {
     return Role.all()
       .then(roles => res.status(200).json(roles))
-      .catch(error => res.status(400).json(error));
+      .catch(error => res.status(500).json({ message: 'An Error Ocurred', error }));
   }
 
   /**
@@ -48,7 +48,7 @@ class RoleController {
         }
         return res.status(200).json(roles);
       })
-      .catch(error => res.status(400).json(error));
+       .catch(error => res.status(500).json(error));
   }
 
   /**
@@ -72,7 +72,7 @@ class RoleController {
           .then(() => res.status(200).json(roles))
           .catch(error => res.status(400).json(error));
       })
-      .catch(error => res.status(400).json(error));
+      .catch(error => res.status(500).json(error));
   }
 
   /**
@@ -91,9 +91,10 @@ class RoleController {
         }
         return roles
           .destroy()
-          .then(() => res.send(200));
+          .then(() => res.send(200))
+          .catch(error => res.status(500).json({ message: 'An Error Ocurred', error }));
       })
-      .catch(error => res.status(400).json(error));
+      .catch(error => res.status(500).json(error));
   }
 }
 

server/controllers/UserController.js

@@ -136,7 +136,7 @@ class UserController {
           return res.status(401).json({ message: 'Wrong Password' });
         }
       })
-      .catch(error => res.status(400).json(error));
+      .catch(error => res.status(500).json({ message: 'An Error Ocurred', error }));
   }
 
   /**
@@ -168,7 +168,8 @@ class UserController {
         $or: [
           { username: { $iLike: `%${search}%` } },
           { firstname: { $iLike: `%${search}%` } },
-          { lastname: { $iLike: `%${search}%` } }],
+          { lastname: { $iLike: `%${search}%` } }
+        ],
         $not: [{ id: req.decoded.id }]
       },
       include: [{ model: Role }],
@@ -198,7 +199,7 @@ class UserController {
         });
       })
       .catch((error) => {
-        res.status(400).json(error);
+        res.status(500).json({ message: 'An Error Ocurred', error });
       });
   }
 
@@ -212,9 +213,14 @@ class UserController {
    * @memberof UserController
    */
   static find(req, res) {
-    return User.findById(req.params.id).then((user) => {
-      return res.status(200).json(userInfo(user));
-    }).catch(error => res.status(400).json(error));
+    return User.findById(req.params.id)
+      .then((user) => {
+        if (!user) {
+          return res.status(404).json({ message: 'User not found' });
+        }
+        return res.status(200).json(userInfo(user));
+      })
+      .catch(error => res.status(500).json({ message: 'An Error Ocurred', error }));
   }
 
   /**
@@ -229,6 +235,20 @@ class UserController {
    */
   static update(req, res) {
     return User.findById(req.params.id).then((user) => {
+      if (!user) {
+        return res.status(404).json({ message: 'User not found' });
+      }
+      if (req.body.email || req.body.username) {
+        User.findOne({
+          where: {
+            $or: [{ username: req.body.username }, { email: req.body.email }]
+          }
+        }).then((existingUser) => {
+          if (existingUser) {
+            return res.status(409).send({ message: 'Email already Exist' });
+          }
+        });
+      }
       if (req.body.password) {
         req.body.password = user.encryptUpdatePassword(req.body.password);
       }
@@ -237,7 +257,9 @@ class UserController {
         .then((user) => {
           res.status(200).json(userInfo(user));
         })
-        .catch(error => res.status(400).json(error));
+        .catch(error =>
+          res.status(500).json({ message: 'An Error Ocurred', error })
+        );
     });
   }
 
@@ -252,8 +274,15 @@ class UserController {
    */
   static delete(req, res) {
     return User.findById(req.params.id).then((user) => {
-      return user.destroy().then(() => res.send(200))
-      .catch(error => res.status(400).json(error));
+      if (!user) {
+        return res.status(404).json({ message: 'User not found' });
+      }
+      return user
+        .destroy()
+        .then(() => res.send(200))
+        .catch(error =>
+          res.status(500).json({ message: 'An Error Ocurred', error })
+        );
     });
   }
 
@@ -274,12 +303,17 @@ class UserController {
     let query;
     if (req.decoded) {
       query = req.decoded.roleId === 2
-        ? { title: { $iLike: search },
+        ? {
+          title: { $iLike: search },
           $or: [
               { $and: [{ access: 'private' }, { userId: req.decoded.id }] },
-            { userId: req.params.id,
+            {
+              userId: req.params.id,
               $or: [{ access: 'public' }, { access: 'role' }]
-            }] } : { userId: req.params.id, title: { $iLike: search } };
+            }
+          ]
+        }
+        : { userId: req.params.id, title: { $iLike: search } };
     }
     Document.findAndCountAll({
       where: query,
@@ -300,23 +334,25 @@ class UserController {
       limit: req.query.limit || 15,
       offset: req.query.offset || 0,
       order: [['createdAt', 'DESC']]
-    }).then((document) => {
-      const limit = req.query.limit || 15;
-      const offset = req.query.offset || 0;
-      const totalCount = document.count;
-      const pageCount = Math.ceil(totalCount / limit);
-      const currentPage = Math.floor(offset / limit) + 1;
-      return res.status(200).json({
-        document: document.rows,
-        pagination: {
-          totalCount,
-          limit,
-          offset,
-          pageCount,
-          currentPage
-        }
-      });
-    }).catch(error => res.status(400).json(error));
+    })
+      .then((document) => {
+        const limit = req.query.limit || 15;
+        const offset = req.query.offset || 0;
+        const totalCount = document.count;
+        const pageCount = Math.ceil(totalCount / limit);
+        const currentPage = Math.floor(offset / limit) + 1;
+        return res.status(200).json({
+          document: document.rows,
+          pagination: {
+            totalCount,
+            limit,
+            offset,
+            pageCount,
+            currentPage
+          }
+        });
+      })
+      .catch(error => res.status(500).json({ message: 'An Error Ocurred', error }));
   }
 }
 

server/test/api/role.spec.js

@@ -135,7 +135,7 @@ describe('Role', () => {
         .get('/roles/3000000000')
         .set({ authorization: adminToken })
         .end((err, res) => {
-          expect(res.status).to.equal(400);
+          expect(res.status).to.equal(500);
           expect(res.body).to.be.a('object');
           expect(res.body.message).to.eql(
             'value "3000000000" is out of range for type integer'
@@ -212,7 +212,7 @@ describe('Role', () => {
         .set({ authorization: adminToken })
         .send({ name: 'team7' })
         .end((err, res) => {
-          expect(res.status).to.equal(400);
+          expect(res.status).to.equal(500);
           expect(res.body).to.be.a('object');
           expect(res.body.message).to.eql(
             'value "3000000000" is out of range for type integer'
@@ -254,7 +254,7 @@ describe('Role', () => {
         .delete('/roles/3000000000')
         .set({ authorization: adminToken })
         .end((err, res) => {
-          expect(res.status).to.equal(400);
+          expect(res.status).to.equal(500);
           expect(res.body).to.be.a('object');
           expect(res.body.message).to.eql(
             'value "3000000000" is out of range for type integer'

server/test/api/user.spec.js

@@ -308,9 +308,9 @@ describe('User', () => {
         .set({ authorization: regularToken })
         .send({ email: TestUser3.email })
         .end((err, res) => {
-          expect(res.status).to.equal(400);
+          expect(res.status).to.equal(409);
           expect(res.body).to.be.a('object');
-          expect(res.body.message).to.eql('Email already exist');
+          expect(res.body.message).to.eql('Email already Exist');
           done();
         });
     });