feat: dump application router json

Store on `run/router.json`

lib/application.js

@@ -1,12 +1,13 @@
 'use strict';
 
 const path = require('path');
+const fs = require('fs');
 const graceful = require('graceful');
 const http = require('http');
-const EggApplication = require('./egg');
-const AppWorkerLoader = require('./loader').AppWorkerLoader;
 const cluster = require('cluster-client');
 const { assign } = require('utility');
+const EggApplication = require('./egg');
+const AppWorkerLoader = require('./loader').AppWorkerLoader;
 
 const KEYS = Symbol('Application#keys');
 const HELPER = Symbol('Application#Helper');
@@ -115,6 +116,35 @@ class Application extends EggApplication {
     return context;
   }
 
+  /**
+   * save routers to `run/router.json`
+   * @private
+   */
+  dumpConfig() {
+    super.dumpConfig();
+
+    // dump routers to router.json
+    const rundir = this.config.rundir;
+    const FULLPATH = this.loader.FileLoader.FULLPATH;
+    try {
+      const dumpRouterFile = path.join(rundir, 'router.json');
+      const routers = [];
+      for (const layer of this.router.stack) {
+        routers.push({
+          name: layer.name,
+          methods: layer.methods,
+          paramNames: layer.paramNames,
+          path: layer.path,
+          regexp: layer.regexp.toString(),
+          stack: layer.stack.map(stack => stack[FULLPATH] || stack._name || stack.name || 'anonymous'),
+        });
+      }
+      fs.writeFileSync(dumpRouterFile, JSON.stringify(routers, null, 2));
+    } catch (err) {
+      this.coreLogger.warn(`dumpConfig router.json error: ${err.message}`);
+    }
+  }
+
   /**
    * Create an anonymous context, the context isn't request level, so the request is mocked.
    * then you can use context level API like `ctx.service`

package.json

@@ -24,7 +24,7 @@
     "delegates": "^1.0.0",
     "egg-cluster": "^1.12.4",
     "egg-cookies": "^2.2.1",
-    "egg-core": "^3.18.0",
+    "egg-core": "^3.19.1",
     "egg-development": "^1.3.2",
     "egg-i18n": "^1.2.0",
     "egg-jsonp": "^1.2.1",

test/fixtures/apps/base-context-class/config/config.unittest.js

@@ -0,0 +1,5 @@
+'use strict';
+
+exports.logger = {
+  consoleLevel: 'NONE',
+};

test/fixtures/apps/demo/app.js

@@ -3,5 +3,7 @@
 module.exports = app => {
   app.ready(() => {
     app.config.tips = 'hello egg started';
+    // dynamic router
+    app.all('/all', app.controller.home);
   });
 };

test/fixtures/apps/demo/app/controller/obj.js

@@ -0,0 +1,19 @@
+'use strict';
+
+module.exports = app => {
+  return {
+    * bar() {
+      this.ctx.body = 'this is obj bar!';
+    },
+
+    * error() {
+      aaa;
+    },
+
+    subObj: {
+      * hello() {
+        this.ctx.body = 'this is subObj hello!';
+      },
+    },
+  };
+};

test/fixtures/apps/demo/app/controller/obj2.js

@@ -0,0 +1,19 @@
+'use strict';
+
+module.exports = {
+  * bar() {
+    this.ctx.body = 'this is obj bar!';
+  },
+
+  subObj: {
+    * hello() {
+      this.ctx.body = 'this is subObj hello!';
+    },
+
+    subSubObj: {
+      * hello() {
+        this.ctx.body = 'this is subSubObj hello!';
+      },
+    },
+  },
+};

test/fixtures/apps/demo/app/router.js

@@ -12,4 +12,12 @@ module.exports = app => {
   app.get('/ip', app.controller.ip);
 
   app.get('/class-controller', 'foo.bar');
+
+  app.get('/obj-controller', 'obj.bar');
+  app.get('/obj-error', 'obj.error');
+  app.get('/subobj-controller', 'obj.subObj.hello');
+
+  app.get('/obj2-controller', app.controller.obj2.bar);
+  app.get('/subobj2-controller', app.controller.obj2.subObj.hello);
+  app.get('/subSubObj-hello', app.controller.obj2.subObj.subSubObj.hello);
 };

test/lib/core/httpclient.test.js

@@ -327,18 +327,21 @@ describe('test/lib/core/httpclient.test.js', () => {
 
       let res = yield httpclient.request(url, {
         method: 'GET',
+        timeout: 20000,
       });
       assert(res.status === 200);
 
 
       res = yield httpclient.request('https://github.com', {
         method: 'GET',
+        timeout: 20000,
       });
 
       assert(res.status === 200);
 
       res = yield httpclient.request('https://www.npmjs.com', {
         method: 'GET',
+        timeout: 20000,
       });
       assert(res.status === 200);
 
@@ -361,17 +364,20 @@ describe('test/lib/core/httpclient.test.js', () => {
 
       res = yield httpclient.request(url, {
         method: 'GET',
+        timeout: 20000,
       });
       assert(res.status === 200);
 
 
       res = yield httpclient.request('https://github.com', {
         method: 'GET',
+        timeout: 20000,
       });
       assert(res.status === 200);
 
       res = yield httpclient.request('https://www.npmjs.com', {
         method: 'GET',
+        timeout: 20000,
       });
       assert(res.status === 200);
 

test/lib/egg.test.js

@@ -28,9 +28,26 @@ describe('test/lib/egg.test.js', () => {
       assert(json.config.tips === 'hello egg');
       json = require(path.join(baseDir, 'run/application_config.json'));
       assert(/\d+\.\d+\.\d+/.test(json.plugins.onerror.version));
+      // should dump dynamic config
       assert(json.config.tips === 'hello egg started');
     });
 
+    it('should dump router json', () => {
+      const routers = require(path.join(baseDir, 'run/router.json'));
+      // 13 static routers on app/router.js and 1 dynamic router on app.js
+      assert(routers.length === 14);
+      for (const router of routers) {
+        assert.deepEqual(Object.keys(router), [
+          'name',
+          'methods',
+          'paramNames',
+          'path',
+          'regexp',
+          'stack',
+        ]);
+      }
+    });
+
     it('should dump config meta', () => {
       let json = require(path.join(baseDir, 'run/agent_config_meta.json'));
       assert(json.name === path.join(__dirname, '../../config/config.default.js'));
@@ -219,7 +236,6 @@ describe('test/lib/egg.test.js', () => {
     after(() => app.close());
 
     it('should access base context properties success', function* () {
-      mm(app.config.logger, 'level', 'DEBUG');
       yield app.httpRequest()
         .get('/')
         .expect('hello')