create middleware to catch non-existent routes

[Finishes #164694145]

server/app.js

@@ -36,14 +36,19 @@ app.use(expressSession({
   saveUninitialized: true
 }));
 
-// Setup a default catch-all route that sends back a welcome message in JSON format.
+// Setup an index route
 app.get('/', (req, res) => res.status(200).send({
   message: 'Authors Haven.'
 }));
 
 // Routes
 app.use('/api', routes);
 
+// Return 404 for nonexistent routes
+app.use((req, res) => {
+  res.status(404).send({ message: 'Route not found' });
+});
+
 // Set Port
 const port = process.env.PORT || 3000;
 

server/middlewares/ValidateBody.js

@@ -2,7 +2,7 @@
 
 import { body, validationResult } from 'express-validator/check';
 
-const highltght = [
+const highlight = [
   body('highlighted')
     .exists()
     .withMessage('highlighted required')
@@ -33,4 +33,4 @@ const validationHandler = (req, res, next) => {
   return next();
 };
 
-export { highltght, validationHandler };
+export { highlight, validationHandler };

server/routes/articles.js

@@ -29,7 +29,7 @@ router.post('/articles/:slug/comment',
 router.post('/articles/:slug/highlight',
   AuthenticateUser.verifyUser,
   AuthenticateArticle.verifyArticle,
-  Validate.highltght,
+  Validate.highlight,
   Validate.validationHandler,
   Comment.highlight);
 

server/test/user.spec.js

@@ -529,3 +529,16 @@ describe('Toggle notification', () => {
       });
   });
 });
+
+describe('Test wild card route', () => {
+  it('Should return an 404 error for unfound routes', (done) => {
+    chai
+      .request(app)
+      .get('/api/reportadmin')
+      .end((req, res) => {
+        expect(res.status).to.equal(404);
+        expect(res.body.message).to.be.a('string').to.equal('Route not found');
+        done();
+      });
+  });
+});