fixed #1

package.json

@@ -31,7 +31,7 @@
     "test:coverage": "jest --config jestconfig.json ---forceExit --coverage",
     "test:watch": "jest --config jestconfig.json --watch --forceExit",
     "prepare": "run-s build",
-    "prepublishOnlyx": "run-s format lint test build",
+    "prepublishOnly": "run-s format lint test build",
     "publish:patch": "npm version patch --force && npm publish",
     "example": "chokidar \"**/*.ts\" -c  \" ts-node example/tester/index.ts\""
   },

src/express-extension/use-express-server.ts

@@ -41,9 +41,10 @@ function addRouterToExpress(app: express.Application, combinedRoutes: CombineRou
   combinedRoutes.forEach((route: any) => {
     if (!route.isClass) {
       const requestMethod: RequestMethod = route.requestMethod;
-      const routePath = prefix + route.path;
+      const routePath = combineRouterPath(prefix, route.path);
 
       if (route.middlewares.length > 0) {
+        // Combine multiple middlewares
         const middleware = combineMiddlewares(...route.middlewares);
         app[requestMethod](routePath, middleware, callInstance(controllerInstance, route));
       } else {
@@ -53,6 +54,21 @@ function addRouterToExpress(app: express.Application, combinedRoutes: CombineRou
   });
 }
 
+export const combineRouterPath = (prefix: string, path: string) => {
+  let result = '';
+  if (prefix !== '') {
+    if (prefix.charAt(0) === '/') prefix = prefix.substring(1);
+    result += prefix;
+  }
+  result += '/';
+  if (path !== '') {
+    if (path.charAt(0) === '/') path = path.substring(1);
+    result += path;
+  }
+  if (result.charAt(0) !== '/') return '/' + result;
+  return result;
+};
+
 const callInstance = (instance: any, route: RouteMetadataArgs) =>
   asyncHelper(async (req: Request, res: Response, next: NextFunction) => {
     await instance[route.methodName](req, res, next);

test/express-extension/utils/combine-router-path.test.ts

@@ -0,0 +1,54 @@
+import { combineRouterPath } from '../../../src/express-extension/use-express-server';
+
+describe('combineRouterPath', () => {
+
+    it(`path: '' & '' path should be '/'`, () => {
+        const result = combineRouterPath('','');
+        const expected = '/';
+        expect(expected).toBe(result);
+    });
+
+    it(`path: '/' & '' should be '/'`, () => {
+        const result = combineRouterPath('/','');
+        const expected = '/';
+        expect(expected).toBe(result);
+    });
+
+    it(`path: '' & '/' should be '/'`, () => {
+        const result = combineRouterPath('','/');
+        const expected = '/';
+        expect(expected).toBe(result);
+    });
+
+    it(`path: '/' & '/' should be '/'`, () => {
+        const result = combineRouterPath('/','/');
+        const expected = '/';
+        expect(expected).toBe(result);
+    });
+
+    it(`path: 'a' & 'b' should be '/a/b'`, () => {
+        const result = combineRouterPath('a','b');
+        const expected = '/a/b';
+        expect(expected).toBe(result);
+    });
+
+    it(`path: '/a' & 'b' should be '/a/b'`, () => {
+        const result = combineRouterPath('/a','b');
+        const expected = '/a/b';
+        expect(expected).toBe(result);
+    });
+
+    it(`path: 'a' & '/b' should be '/a/b'`, () => {
+        const result = combineRouterPath('a','/b');
+        const expected = '/a/b';
+        expect(expected).toBe(result);
+    });
+
+    it(`path: '/a' & '/b' should be '/a/b'`, () => {
+        const result = combineRouterPath('/a','/b');
+        const expected = '/a/b';
+        expect(expected).toBe(result);
+    });
+
+
+});