Improve

.autod.conf

@@ -0,0 +1,17 @@
+'ues strict';
+
+module.exports = {
+  write: true,
+  prefix: '^',
+  devprefix: '^',
+  exclude: [
+    'test/fixtures',
+  ],
+  devdep: [
+    'autod',
+  ],
+  keep: [
+  ],
+  semver: [
+  ],
+};

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,24 @@
+<!--
+Thank you for your pull request. Please review below requirements.
+Bug fixes and new features should include tests and possibly benchmarks.
+Contributors guide: https://github.com/eggjs/egg/blob/master/CONTRIBUTING.md
+
+感谢您贡献代码。请确认下列 checklist 的完成情况。
+Bug 修复和新功能必须包含测试，必要时请附上性能测试。
+Contributors guide: https://github.com/eggjs/egg/blob/master/CONTRIBUTING.md
+-->
+
+##### Checklist
+<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->
+
+- [ ] `npm test` passes
+- [ ] tests and/or benchmarks are included
+- [ ] documentation is changed or added
+- [ ] commit message follows commit guidelines
+
+##### Affected core subsystem(s)
+<!-- Provide affected core subsystem(s). -->
+
+
+##### Description of change
+<!-- Provide a description of the change below this comment. -->

README.md

@@ -214,7 +214,8 @@ this.loadExtend('application', app);
 - {String|Array} directory - directories to load
 - {Object} target: attach object from loaded files,
 - {String} ignore - ignore the files when load
-- {Function} initializer - custom file exports
+- {Function} initializer - custom file exports, receive two parameters, the first one is the inject object,
+  second one is the path of the loaded file.
 - {Boolean} lowercaseFirst - determine whether the fist letter is lowercase
 - {Boolean} override: determine whether override the property when get the same name
 - {Boolean} call - determine whether invoke when exports is function

lib/loader/egg_loader.js

@@ -265,6 +265,14 @@ class EggLoader {
     new ContextLoader(opt).load();
   }
 
+  get FileLoader() {
+    return FileLoader;
+  }
+
+  get ContextLoader() {
+    return ContextLoader;
+  }
+
 }
 
 /**

lib/loader/file_loader.js

@@ -119,7 +119,7 @@ function getExports(fullpath, initializer, isCall, inject) {
 
   // process exports as you like
   if (initializer) {
-    exports = initializer(exports);
+    exports = initializer(exports, fullpath);
   }
 
   // return exports when it's a class or generator

package.json

@@ -4,6 +4,7 @@
   "description": "A core Plugable framework based on koa",
   "main": "index.js",
   "scripts": {
+    "autod": "autod",
     "lint": "eslint lib test *.js",
     "test": "npm run lint && egg-bin test",
     "cov": "egg-bin cov",
@@ -30,29 +31,30 @@
     "version": "4, 6"
   },
   "devDependencies": {
+    "autod": "^2.7.0",
     "egg-bin": "1",
     "egg-ci": "1",
     "eslint": "~3.1.0",
     "eslint-config-egg": "3",
     "koa": "1",
     "koa-router": "4",
-    "mm": "1",
+    "mm": "^2.0.0",
     "pedding": "^1.0.0",
-    "should": "9",
-    "supertest": "1"
+    "should": "^11.1.0",
+    "supertest": "^2.0.0"
   },
   "dependencies": {
     "debug": "^2.2.0",
     "depd": "^1.1.0",
-    "egg-logger": "^1.1.0",
+    "egg-logger": "^1.2.0",
     "extend": "^3.0.0",
     "globby": "^6.0.0",
     "inflection": "^1.10.0",
     "interop-require": "^1.0.0",
     "is-type-of": "^1.0.0",
-    "koa": "^1.2.0",
+    "koa": "^1.2.1",
     "koa-router": "^5.4.0",
     "ready-callback": "^1.0.0",
     "utility": "^1.8.0"
   }
-}
+}

test/fixtures/load_dirs/dao/TestClass.js

@@ -1,9 +1,11 @@
 'use strict';
 
 module.exports = class TestClass {
-  constructor() {
+  constructor(app, fullpath) {
     this.user = {
       name: 'kai.fangk',
     };
+    this.app = app;
+    this.path = fullpath;
   }
 }

test/loader/egg_loader.test.js

@@ -0,0 +1,18 @@
+'use strict';
+
+const should = require('should');
+const utils = require('../utils');
+
+describe('test/loader/egg_loader.test.js', () => {
+
+  let app;
+  before(() => {
+    app = utils.createApp('nothing');
+  });
+
+  it('should container FileLoader and ContextLoader', () => {
+    should.exists(app.loader.FileLoader);
+    should.exists(app.loader.ContextLoader);
+  });
+
+});

test/loader/file_loader.test.js

@@ -155,12 +155,14 @@ describe('test/file_loader.test.js', () => {
       directory: path.join(dirBase, 'dao'),
       target: app.dao,
       ignore: 'util/**',
-      initializer(exports) {
-        return new exports(app);
+      initializer(exports, fullpath) {
+        return new exports(app, fullpath);
       },
     }).load();
     app.dao.should.have.property('TestClass');
     app.dao.TestClass.user.should.eql({ name: 'kai.fangk' });
+    app.dao.TestClass.app.should.equal(app);
+    app.dao.TestClass.path.should.equal(path.join(dirBase, 'dao/TestClass.js'));
     app.dao.should.have.property('testFunction', { user: { name: 'kai.fangk' } });
     app.dao.should.have.property('testReturnFunction', { user: { name: 'kai.fangk' } });
   });