Merge pull request #5 from BoolJS/develop

Version bump to v0.5.0

Author: pandres95

.editorconfig

@@ -0,0 +1,4 @@
+[*]
+indent_size=4
+[package.json]
+indent_size=2

.eslintrc.js

@@ -0,0 +1,14 @@
+'use strict';
+
+module.exports = {
+    extends: [ 'standard' ],
+    env: {
+        es6: true,
+        node: true
+    },
+    rules: {
+        indent: [ 'error', 4 ],
+        semi: [ 'error', 'always' ],
+        'no-multi-spaces': 0
+    }
+};

.jshintrc

@@ -1,3 +1,3 @@
 language: node_js
 node_js:
-  - 4.0
+  - 8.0

.travis.yml

@@ -0,0 +1,12 @@
+'use strict';
+
+const { Middleware } = require('..');
+
+module.exports = class DependantMiddleware extends Middleware {
+    constructor () {
+        super('dependant-middleware');
+    }
+    action (req, res, next) {
+        next();
+    }
+};

example/dependant-class.js

@@ -1,12 +1,12 @@
 'use strict';
 
-var API = require('..');
+const { Middleware } = require('..');
 
-module.exports = class ExampleMiddleware extends API.Middleware {
-    constructor(){
-        super('example-middleware');
+module.exports = class ExampleMiddleware extends Middleware {
+    constructor () {
+        super('example-middleware', [ require.resolve('./dependant-class') ]);
     }
-    action(req, res, next) {
+    action (req, res, next) {
         next();
     }
 };

example/plugin.js

@@ -1,7 +1,7 @@
 'use strict';
 
-var AppInstance     = require('./instance')
-,   instancePool    = {};
+const AppInstance = require('./instance');
+const instancePool = {};
 
 /**
  * @class App
@@ -17,7 +17,7 @@ module.exports = {
      * @return {String[]} An array with the namespaces of the instances in
      * the pool.
      */
-    listInstances: function(){
+    listInstances: function () {
         return Object.keys(instancePool);
     },
     /**
@@ -29,8 +29,8 @@ module.exports = {
      * otherwise, it won't work after.
      * @return {API.Component.AppInstance} The application object.
      */
-    getInstance: function(namespace, dependencies){
-        if(instancePool[namespace] === undefined){
+    getInstance: function (namespace, dependencies) {
+        if (instancePool[namespace] === undefined) {
             var newInstance = new AppInstance(namespace, dependencies);
             instancePool[namespace] = newInstance;
         }
@@ -41,7 +41,7 @@ module.exports = {
      * @description Removes instance of an application from the pool.
      * @param {String} namespace - The namespace of the application.
      */
-    removeInstance: function(namespace){
+    removeInstance: function (namespace) {
         delete instancePool[namespace];
     }
 

lib/app/index.js

@@ -1,8 +1,9 @@
 'use strict';
 
-var Store           = require('./store')
-,   Namespace       = require('./namespace')
-,   AppComponents   = require('./components');
+const Store = require('./store');
+const Namespace = require('./namespace');
+const AppComponents = require('./components');
+const injector = require('object-injector');
 
 /**
  * @class AppInstance
@@ -13,11 +14,10 @@ var Store           = require('./store')
  * @param {String[]} [dependencies] - A list of packages that instance
  * depends
  */
-module.exports = function(namespace, dependencies){
-
-    if(!Namespace.checker(namespace)) throw new Error(
-        "Specified namespace is invalid"
-    );
+module.exports = function (namespace, dependencies) {
+    if (!Namespace.checker(namespace)) {
+        throw new Error('Specified namespace is invalid');
+    }
 
     var components = new AppComponents();
     injector(this, components);
@@ -47,8 +47,7 @@ module.exports = function(namespace, dependencies){
      * @description Shortcut for long schema skeleton of namespace.
      * @return {Object} The skeleton of the application, including namespaces.
      */
-    this.getSkeleton = function(){
-        return Namespace.generator(namespace, {}, this.getComponents);
+    this.getSkeleton = function () {
+        return Namespace.generator(namespace, this.getComponents);
     };
-
 };

lib/app/instance/index.js

@@ -12,22 +12,16 @@
  * @throws {Error} In case namespace is not according to standard naming
  * conventions.
  */
-var namespaceGenerator = exports.generator = function(namespace, skel, final){
+exports.generator = function namespaceGenerator (namespace, final) {
+    const packages = namespace.split('.');
 
-    var skeleton = {};
-    skel = skel || {}; //Empty object for initial call
+    let current = { [packages.pop()]: final() };
 
-    var ns = namespace.split('.');
-
-    if(ns.length > 0 && ns[0] !== ''){
-        skeleton[ns[0]] = namespaceGenerator(
-            ns.slice(1).join("."), {}, final
-        );
-        return skeleton;
-    } else {
-        return final();
+    while (packages.length > 1) {
+        current = { [packages.pop()]: current };
     }
 
+    return { [packages.pop()]: current };
 };
 
 /**
@@ -39,6 +33,6 @@ var namespaceGenerator = exports.generator = function(namespace, skel, final){
  * @returns {Boolean} A boolean value stating true if the namespace is valid,
  * and false otherwise.
  */
-var namespaceChecker = exports.checker = function(namespace){
+exports.checker = function namespaceChecker (namespace) {
     return /^(@?[a-z_A-Z]\w+(?:\.?[a-z_A-Z]\w+)*)$/.test(namespace);
 };

lib/app/instance/namespace.js

@@ -11,10 +11,10 @@
  * @return {API.Error} An {@link API.Error} instance.
  */
 module.exports = class BoolError extends Error {
-    constructor(status, code, message, uri) {
+    constructor (status, code, message, uri) {
         super();
         // Executed in case first argument is `code`.
-        if(!(typeof status === 'number' && (status % 1) === 0)) {
+        if (!(typeof status === 'number' && (status % 1) === 0)) {
             status = 500;
             code = status || 'server_error';
             message = code || null;
@@ -26,7 +26,6 @@ module.exports = class BoolError extends Error {
             uri = uri || null;
         }
 
-
         /**
          * @value name
          * @description Name of error