Merge pull request #257 from raveljs/bugfix/256

getDependencies within injector not including @autoinjected deps

documentation.yml

@@ -1,6 +1,7 @@
 name: Ravel API
-version: 1.0.0-rc.3
+version: 1.0.0-rc.4
 versions:
+  - 1.0.0-rc.4
   - 1.0.0-rc.3
   - 1.0.0-rc.2
   - 1.0.0-rc.1

jest/integration/ravel-lifecycle.test.js

@@ -146,7 +146,7 @@ describe('Ravel lifeycle test', () => {
     app.set('public directory', '/public');
     app.set('favicon path', '/favicon.ico');
 
-    app.load(Another, Users, UsersResource, TestRoutes);
+    app.load(Users, UsersResource, TestRoutes, Another);
   });
 
   describe('#init()', () => {

lib/core/injector.js

@@ -38,8 +38,9 @@ class Injector {
 
   /**
    * @param {Class} DIClass - A class to retrieve injection metadata from.
-   * @returns {Array<string>} The modules which have been marked as DI dependencies
-   *   for the target class (`@autoinject`ed).
+   * @returns {Object} The modules (keys) which have been marked as DI dependencies
+   *   for the target class (`@autoinject`ed), as well as the name (values) they will be
+   *   assigned to on the current scope.
    * @private
    */
   getAutoinjectedDependencies (DIClass) {
@@ -53,7 +54,7 @@ class Injector {
    * @private
    */
   getDependencies (DIClass) {
-    return this.getInjectedDependencies(DIClass).concat(this.getAutoinjectedDependencies(DIClass));
+    return this.getInjectedDependencies(DIClass).concat(Object.keys(this.getAutoinjectedDependencies(DIClass)));
   }
 
   /**

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ravel",
-  "version": "1.0.0-rc.3",
+  "version": "1.0.0-rc.4",
   "author": "Sean McIntyre <[email protected]>",
   "description": "Ravel Rapid Application Development Framework",
   "engines": {