fix unused controller throwing error in route handling

src/models/route_info.ts

@@ -34,9 +34,7 @@ export class RouteInfo implements IRouteInfo {
 
  set path(value) {
  this.path_ = value;
- if (value) {
- this.pathSplitted = value.split("/");
- }
+ this.pathSplitted = value ? value.split("/") : [];
  }
 
  get path() {

tests/general/controllers/user_profile_controller.ts

@@ -17,6 +17,22 @@ export class UserProfileController extends Controller {
 @shields(UserProfileNestedShield)
 export class UserProfileNestedController extends Controller {
 
+ @worker()
+ @route('/me')
+ async getMe(@singleton(UserService) userService: UserService) {
+ const userId = await this.session.get('userId');
+ const user = userService.getUser(userId);
+ return jsonResult({
+ nestedProfile: user
+ });
+ }
+}
+
+// do not remove this - 
+// this to make sure that controller which is not used but created are not creating problem
+@shields(UserProfileNestedShield)
+export class NotUsedController extends Controller {
+
  @worker()
  @route('/me')
  async getMe(@singleton(UserService) userService: UserService) {

tests/general/crons/counter.spec.ts

@@ -1,10 +1,7 @@
 import { initServer } from "..";
-import { viewResult, Fort, textResult } from "fortjs";
+import { Fort } from "fortjs";
 import { CounterScheduler } from "./counter";
-// import { } from "jest;
 
-// jest.
-// jest.sp
 describe("counter scheduler", () => {
  beforeAll(async () => {
  await initServer();

tests/general/package.json

@@ -16,7 +16,7 @@
  "build:test:prod": "NODE_ENV=production npm run build:test",
  "install:build:test": "npm run install:lib && npm run build:test",
  "webpack:watch": "webpack --config webpack.config.js --watch",
- "start": "export NODE_OPTIONS=--openssl-legacy-provider; npm run webpack:watch",
+ "start": "export NODE_OPTIONS=\"--openssl-legacy-provider --enable-source-maps\" && npm run webpack:watch",
  "start:dev": "NODE_ENV=development npm run start",
  "start:prod": "NODE_ENV=production npm run start"
  },
@@ -51,4 +51,4 @@
  "ts-loader": "^8.0.1",
  "typescript": "^3.9.7"
  }
-}
+}

tests/general/routes.test.ts

@@ -11,8 +11,9 @@ describe('DefaultController', () => {
  const router = new Router();
  const routesAsArray = router.routesAsArray;
  expect(routesAsArray).toHaveLength(
- routes.length + 2
+ routes.length + 2 + 1
  // +2 due to nested child
+ // +1 unused controller
  );
  const controllerName = "HomeController";
  const controller = routesAsArray.find(q => q.controllerName === controllerName);

tests/general/test/home.js

@@ -201,13 +201,13 @@ describe("/home", () => {
  request.get('/setSingletonValue?value=' + value).end((err, res) => {
  expect(err).to.be.null;
  expect(res).to.have.status(200);
- })
 
- request.get('/getSingletonValue').end((err, res) => {
- expect(err).to.be.null;
- expect(res).to.have.status(200);
- expect(res.text).to.be.equal(value);
- done();
+ request.get('/getSingletonValue').end((err, res) => {
+ expect(err).to.be.null;
+ expect(res).to.have.status(200);
+ expect(res.text).to.be.equal(value);
+ done();
+ })
  })
  })
  })