better typings, unknown request method error handler

dist/src/global-modules-indexer/index.d.ts

@@ -1,2 +1,2 @@
-declare function GlobalModulesIndexer(context: string, dirName: string): Object;
+declare function GlobalModulesIndexer(context: string, dirName: string): object;
 export default GlobalModulesIndexer;

dist/src/route-mapper/map.js

@@ -44,8 +44,13 @@ function default_1(context, dirName) {
         }
     };
     router.all('*', function (req, res, next) {
-        RoutesMapping[req.method](router, req.path);
-        next();
+        if (typeof RoutesMapping[req.method] !== 'function') {
+            res.status(500).json("request not handled, it must be one of GET, POST, PUT, PATCH or DELETE.");
+        }
+        else {
+            RoutesMapping[req.method](router, req.path);
+            next();
+        }
     });
     return router;
 }

src/global-modules-indexer/index.ts

@@ -1,7 +1,7 @@
 import fs = require('fs');
 import path = require('path');
 
-let modulesIndex: Object = {};
+let modulesIndex: object = {};
 
 function startRecursiveCheck(path: string): void {
     fs.readdirSync(path).forEach(e => {
@@ -19,7 +19,7 @@ function nextChecks(name: string, path: string): void {
 };
 
 
-function GlobalModulesIndexer(context: string, dirName: string): Object {
+function GlobalModulesIndexer(context: string, dirName: string): object {
     let absolutePath = path.join(context, dirName);
 
     startRecursiveCheck(absolutePath);

src/route-mapper/map.js

@@ -55,8 +55,12 @@ module.exports = function (context, dirName) {
   }
 
   router.all('*', (req, res, next) => {
-    RoutesMapping[req.method](router, req.path);
-    next();
+    if (typeof RoutesMapping[req.method] !== 'function') {
+      res.status(500).json({ error: 'Request not handled, it must be one of GET, POST, PUT, PATCH, or DELETE' });
+    } else {
+      RoutesMapping[req.method](router, req.path);
+      next();
+    }
   });
 
   return router;

src/route-mapper/map.ts

@@ -9,37 +9,37 @@ export default function (context: string, dirName: string): Router {
     let utils = Utils;
 
     const RoutesMapping = {
-        'GET': function (router: Router, path: string) {
+        'GET': (router: Router, path: string): void => {
             let options = utils.options(path);
             let ctrl = options.ctrl;
 
             utils.hasReqParams(path) ?
                 router.get(options.url, (req, res, next) => { return modulesIndex[ctrl].find(req, res, next); }) :
                 router.get(path, (req, res, next) => { return modulesIndex[ctrl].list(req, res, next); });
         },
-        'POST': function (router, path) {
+        'POST': (router: Router, path: string): void => {
             let ctrl = path.replace('/', '');
             router.post(path, (req, res, next) => {
                 return modulesIndex[ctrl].create(req, res, next);
             })
         },
-        'PUT': function (router, path) {
+        'PUT': (router: Router, path: string): void => {
             let options = utils.options(path);
             let ctrl = options.ctrl;
 
             router.put(options.url, (req, res, next) => {
                 return modulesIndex[ctrl].update(req, res, next);
             })
         },
-        'PATCH': function (router, path) {
+        'PATCH': (router: Router, path: string): void => {
             let options = utils.options(path);
             let ctrl = options.ctrl;
 
             router.patch(options.url, (req, res, next) => {
                 return modulesIndex[ctrl].update(req, res, next);
             })
         },
-        'DELETE': function (router, path) {
+        'DELETE': (router: Router, path: string): void => {
             let options = utils.options(path);
             let ctrl = options.ctrl;
 
@@ -50,8 +50,12 @@ export default function (context: string, dirName: string): Router {
     }
 
     router.all('*', (req, res, next) => {
-        RoutesMapping[req.method](router, req.path);
-        next();
+        if (typeof RoutesMapping[req.method] !== 'function') {
+            res.status(500).json(`request not handled, it must be one of GET, POST, PUT, PATCH or DELETE.`)
+        } else {
+            RoutesMapping[req.method](router, req.path);
+            next();
+        }
     });
 
     return router;

src/utils/utils.ts

@@ -1,8 +1,8 @@
 export default {
-    hasReqParams: function (path: string): boolean {
+    hasReqParams: (path: string): boolean => {
         return path.split('/').filter(e => e.length > 0).length > 1;
     },
-    options: function (path: string): any {
+    options: (path: string): any => {
         let config = {
             ctrl: '',
             url: '/'