not-node 4.0.4 → 4.0.5

package/index.js

@@ -29,6 +29,8 @@ module.exports.Routine = require('./src/model/routine');
 module.exports.Common = require('./src/common');
 /** Fields library manager */
 module.exports.Fields = require('./src/fields');
+/** Form validation template **/
+module.exports.Form = require('./src/form');
 /** Application initialization procedures */
 module.exports.Init = require('./src/init').Init;
 /** Application object */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "not-node",
-  "version": "4.0.4",
+  "version": "4.0.5",
   "description": "node complimentary part for client side notFramework.",
   "main": "index.js",
   "scripts": {

package/src/domain.js

@@ -133,6 +133,17 @@ class notDomain extends EventEmitter {
     return null;
   }
 
+  /**
+   *  Returns form
+   *  @param   {string}  name   'formName' or 'moduleName//formName'
+   *                  ('login', 'not-user//login')
+   *  @return {object}        form
+   **/
+  getForm(name) {
+    const type = 'form';
+    return this.getByPath(name, type);
+  }
+
   /**
    *  Returns model
    *  @param   {string}  name   'modelName' or 'moduleName//modelName'
@@ -141,11 +152,7 @@ class notDomain extends EventEmitter {
    **/
   getModel(name) {
     const type = 'model';
-    if (name.indexOf('//') > 0) {
-      return this.getByFullPath(name, type);
-    } else {
-      return this.getByShortPath(name, type);
-    }
+    return this.getByPath(name, type);
   }
 
   getByFullPath(name, type){
@@ -169,31 +176,23 @@ class notDomain extends EventEmitter {
 
   /**
    *  Returns file with model declarations
-   *  @param {string}   modelName  'modelName' or 'moduleName//modelName'
+   *  @param {string}   name  'modelName' or 'moduleName//modelName'
    *  @return  {object}        CommonJS module object
    **/
-  getModelFile(modelName) {
+  getModelFile(name) {
     const type = 'modelFile';
-    if (modelName.indexOf('//') > 0) {
-      return this.getByFullPath(modelName, type);
-    } else {
-      return this.getByShortPath(modelName, type);
-    }
+    return this.getByPath(name, type);
   }
 
   /**
    *  Returns specified by name or 'moduleName//modelName' model Schema
-   *  @param {string} modelName  'modelName' or 'moduleName//modelName'
+   *  @param {string} name  'modelName' or 'moduleName//modelName'
    *  @return {object}       model schema
    **/
 
-  getModelSchema(modelName) {
+  getModelSchema(name) {
     const type = 'modelSchema';
-    if (modelName.indexOf('//') > 0) {
-      return this.getByFullPath(modelName, type);
-    } else {
-      return this.getByShortPath(modelName, type);
-    }
+    return this.getByPath(name, type);
   }
 
   /**
@@ -204,25 +203,25 @@ class notDomain extends EventEmitter {
    **/
   getLogic(name) {
     const type = 'logic';
-    if (name.indexOf('//') > 0) {
-      return this.getByFullPath(name, type);
-    } else {
-      return this.getByShortPath(name, type);
-    }
+    return this.getByPath(name, type);
   }
 
 
   /**
    *  Returns file with logic declarations
-   *  @param {string}   logicName  'logicName' or 'moduleName//logicName'
+   *  @param {string}   name  'logicName' or 'moduleName//logicName'
    *  @return  {object}        CommonJS module object
    **/
-  getLogicFile(logicName) {
+  getLogicFile(name) {
     const type = 'logicFile';
-    if (logicName.indexOf('//') > 0) {
-      return this.getByFullPath(logicName, type);
+    return this.getByPath(name, type);
+  }
+
+  getByPath(name, type){
+    if (name.indexOf('//') > 0) {
+      return this.getByFullPath(name, type);
     } else {
-      return this.getByShortPath(logicName, type);
+      return this.getByShortPath(name, type);
     }
   }
 
@@ -403,6 +402,10 @@ class notDomain extends EventEmitter {
         count: 0,
         list: []
       },
+      forms: {
+        count: 0,
+        list: []
+      },
       actions: {
         count: 0,
         list: []
@@ -418,8 +421,9 @@ class notDomain extends EventEmitter {
       stats.modules.content[modName] = modStatus;
       stats.routes.count += modStatus.routes.count;
       stats.models.count += modStatus.models.count;
+      stats.forms.count += modStatus.forms.count;
       stats.actions.count += modStatus.actions.count;
-      for (let t of ['routes', 'models', 'actions']) {
+      for (let t of ['routes', 'models', 'actions', 'forms']) {
         stats[t].list.push(...(modStatus[t].list.map(itmName => `${modName}//${itmName}`)));
       }
     }

package/src/form/index.js

@@ -0,0 +1,78 @@
+//DB related validation tools
+const mongoose = require('mongoose');
+const Schema = mongoose.Schema;
+//not-node
+const initFields = require('../fields').initFields;
+const {
+  byFieldsValidators
+} = require('../model/enrich');
+const {
+  notValidationError,
+  notError
+} = require('not-error');
+
+/**
+ * Generic form validation class
+ **/
+module.exports = class Form {
+  constructor({
+    FIELDS,
+    FORM_NAME
+  }) {
+    this.FORM_NAME = FORM_NAME;
+    this.FIELDS = FIELDS;
+    this.SCHEMA = byFieldsValidators(initFields(FIELDS, 'model'));
+    this.MODEL = mongoose.model(FORM_NAME, Schema(this.SCHEMA));
+  }
+
+  /**
+   * Extract data from ExpressRequest object and validates it
+   * returns it or throws
+   * @param {ExpressRequest} req expressjs request object
+   * @return {Promise<Object>} form data
+   * @throws {notValidationError}
+   **/
+  async run(req) {
+    let data = await this.extract(req);
+    await this.validate(data);
+    return data;
+  }
+
+  /**
+   * Extracts data, should be overriden
+   * @param {ExpressRequest} req expressjs request object
+   * @return {Object}        forma data
+   **/
+  async extract( /*req*/ ) {
+    return {};
+  }
+
+  /**
+   * Validates form data or throws
+   * @param {Object} data    form data
+   * @return {Object}
+   * @throws {notValidationError}
+   **/
+  async validate(data) {
+    try {
+      await this.MODEL.validate(data, this.FIELDS);
+    } catch (e) {
+      let fields = {};
+      if (e instanceof mongoose.Error.ValidationError) {
+        Object.keys(e.errors).forEach(name => {
+          fields[name] = [e.errors[name].message];
+        });
+        throw new notValidationError(e.message, fields, e, data);
+      } else {
+        throw new notError(
+          'core:form_validation_error', {
+            FORM_NAME: this.FORM_NAME,
+            FIELDS: this.FIELDS,
+            data
+          },
+          e
+        );
+      }
+    }
+  }
+};

package/src/init/db/mongoose.js

@@ -28,6 +28,7 @@ module.exports = class InitDBMongoose{
   }
 
   async run({config, options, master, conf, alias}){
+    log.info(`db.${alias}.pre`);
     await ADDS.run(`db.${alias}.pre`, {config, options, master, conf, alias});
     await InitDBMongoose.initMongoose({conf, master, alias});
     await ADDS.run(`db.${alias}.post`, {config, options, master, conf, alias});

package/src/init/index.js

@@ -1,4 +1,5 @@
 //
+const os = require('os');
 const path = require('path');
 const logger = require('not-log');
 const log = logger(module, 'not-node:Init');
@@ -126,7 +127,7 @@ class Init {
     additional
   }) {
     try {
-      log.info('Kick start app...');
+      log.info('Kick start app...'+ os.platform()+os.arch());
       ADDS.init(additional);
       const initSequence = new InitSequence(STANDART_INIT_SEQUENCE);
       await ADDS.run('pre', {

package/src/init/rateLimiter.js

@@ -29,7 +29,7 @@ module.exports = class InitRateLimiter{
     return new RateLimiterRedis({
       storeClient:  master.getEnv('db.redis'),
       keyPrefix:    'middleware',
-      points:       10,     // 10 requests
+      points:       100,     // 10 requests
       duration:     1,    // per 1 second by IP
     });
   }

package/src/init/routes.js

@@ -13,7 +13,6 @@ module.exports = class InitRoutes {
   static finalError({
     master
   }) {
-    // eslint-disable-next-line no-unused-vars
     return (err, req, res, next) => {
       //reportable errors from known cases
       if (err instanceof notError) {
@@ -53,6 +52,7 @@ module.exports = class InitRoutes {
           status: 'error'
         });
       }
+      next();
     };
   }
 

package/src/manifest/module.js

@@ -47,6 +47,7 @@ class notModule {
     };
     this.models = {};
     this.logics = {};
+    this.forms = {};
     this.manifests = {};
     this.faulty = false;
     this.paths = {
@@ -220,10 +221,15 @@ class notModule {
   }
 
   getStatus() {
+    const formsList = Object.keys(this.forms);
     const modelsList = Object.keys(this.models);
     const routesList = Object.keys(this.routes);
     const actionsList = this.getActionsList();
     return {
+      forms: {
+        count: formsList.length,
+        list: formsList
+      },
       models: {
         count: modelsList.length,
         list: modelsList,
@@ -271,11 +277,18 @@ class notModule {
     return content;
   }
 
-
   setManifest(key, val){
     this.manifests[key] = val;
   }
 
+  getForm(key) {
+    return this.forms[key];
+  }
+
+  setForm(key, val) {
+    this.forms[key] = val;
+  }
+
   setModel(key, val){
     this.models[key] = val;
   }

package/src/manifest/registrator/forms.js

@@ -0,0 +1,48 @@
+const path = require('path');
+const fs = require('fs');
+const {tryFile} = require('../../common');
+
+module.exports = class notModuleRegistratorForms{
+
+  static openFile = require;
+  
+  constructor({nModule}){
+    this.run({nModule});
+  }
+
+  run({nModule}){
+    const srcDir = notModuleRegistratorForms.getPath(nModule);
+    if (!srcDir) { return false; }
+    this.findAll(
+      {
+        nModule,
+        srcDir
+      }
+    );
+    return true;
+  }
+
+  static getPath(nModule){
+    return nModule.module.paths.forms;
+  }
+
+  /**
+  * Searching forms in directory
+  * @param {Object}     input
+  * @param {string}     input.srcDir
+  **/
+  findAll({nModule, srcDir}){
+    fs.readdirSync(srcDir).forEach((file) => {
+      let fromPath = path.join(srcDir, file);
+      if (!tryFile(fromPath)) { return; }
+      this.register({nModule, fromPath});
+    });
+  }
+
+  register({nModule, fromPath}){
+    const Form = notModuleRegistratorForms.openFile(fromPath);
+    const parts = path.parse(fromPath);
+    nModule.setForm(parts.name, new Form());
+  }
+
+};

package/src/manifest/registrator/index.js

@@ -5,6 +5,7 @@
 
 const
   notModuleRegistratorFields = require('./fields'),
+  notModuleRegistratorForms = require('./forms'),
   notModuleRegistratorModels = require('./models'),
   notModuleRegistratorLogics = require('./logics'),
   notModuleRegistratorRoutes = require('./routes'),
@@ -13,6 +14,7 @@ const
 
 const DEFAULT_REGISTRATORS = [
   notModuleRegistratorFields,
+  notModuleRegistratorForms,
   notModuleRegistratorModels,
   notModuleRegistratorLogics,
   notModuleRegistratorRoutes,

package/src/manifest/route.js

@@ -148,7 +148,7 @@ class notRoute{
   * if presented
   * @param {ExpressRequest} req request object
   * @param {object} result result returned by main action processor
-  **/
+  */
   filterResultByReturnRule(req, result){
     const returnList = this.extractReturn(req.notRouteData);
     if(result && (typeof result === 'object') && returnList && Array.isArray(returnList)){

package/test/module/index.js

@@ -22,7 +22,7 @@ module.exports = (input)=>{
       notModuleRegistrator.setRegistrators(regs);
       expect(notModuleRegistrator.registrators.length).to.be.equal(0);
       notModuleRegistrator.resetRegistrators();
-      expect(notModuleRegistrator.registrators.length).to.be.equal(5);
+      expect(notModuleRegistrator.registrators.length).to.be.equal(6);
     });
 
     it('with paths', ()=>{

package/test/notDomain.js

@@ -184,7 +184,7 @@ describe('notDomain', function() {
     it('model name short', function() {
       const route = 'Jungle';
       const ctx = {
-        getByShortPath(){return null;}
+        getByPath(){return null;}
       };
       const res = notDomain.prototype.getModel.call(ctx, route);
       expect(res).to.be.null;
@@ -194,7 +194,7 @@ describe('notDomain', function() {
       const id = Math.random();
       const route = 'module//Jungle';
       const ctx = {
-        getByFullPath(){return id;}
+        getByPath(){return id;}
       };
       const res = notDomain.prototype.getModel.call(ctx, route);
       expect(res).to.be.equal(id);
@@ -206,7 +206,7 @@ describe('notDomain', function() {
       const id = Math.random();
       const route = 'Jungle';
       const ctx = {
-        getByShortPath(){return id;}
+        getByPath(){return id;}
       };
       const res = notDomain.prototype.getModelFile.call(ctx, route);
       expect(res).to.be.equal(id);
@@ -216,7 +216,7 @@ describe('notDomain', function() {
       const id = Math.random();
       const route = 'module//Jungle';
       const ctx = {
-        getByFullPath(){return id;}
+        getByPath(){return id;}
       };
       const res = notDomain.prototype.getModelFile.call(ctx, route);
       expect(res).to.be.equal(id);
@@ -229,7 +229,7 @@ describe('notDomain', function() {
       const id = Math.random();
       const route = 'Jungle';
       const ctx = {
-        getByShortPath(){return id;}
+        getByPath(){return id;}
       };
       const res = notDomain.prototype.getModelSchema.call(ctx, route);
       expect(res).to.be.equal(id);
@@ -239,7 +239,7 @@ describe('notDomain', function() {
       const id = Math.random();
       const route = 'module//Jungle';
       const ctx = {
-        getByFullPath(){return id;}
+        getByPath(){return id;}
       };
       const res = notDomain.prototype.getModelSchema.call(ctx, route);
       expect(res).to.be.equal(id);
@@ -252,7 +252,7 @@ describe('notDomain', function() {
       const id = Math.random();
       const route = 'Jungle';
       const ctx = {
-        getByShortPath(){return id;}
+        getByPath(){return id;}
       };
       const res = notDomain.prototype.getLogic.call(ctx, route);
       expect(res).to.be.equal(id);
@@ -262,7 +262,7 @@ describe('notDomain', function() {
       const id = Math.random();
       const route = 'module//Jungle';
       const ctx = {
-        getByFullPath(){return id;}
+        getByPath(){return id;}
       };
       const res = notDomain.prototype.getLogic.call(ctx, route);
       expect(res).to.be.equal(id);
@@ -274,7 +274,7 @@ describe('notDomain', function() {
       const id = Math.random();
       const route = 'Jungle';
       const ctx = {
-        getByShortPath(){return id;}
+        getByPath(){return id;}
       };
       const res = notDomain.prototype.getLogicFile.call(ctx, route);
       expect(res).to.be.equal(id);
@@ -284,7 +284,7 @@ describe('notDomain', function() {
       const id = Math.random();
       const route = 'module//Jungle';
       const ctx = {
-        getByFullPath(){return id;}
+        getByPath(){return id;}
       };
       const res = notDomain.prototype.getLogicFile.call(ctx, route);
       expect(res).to.be.equal(id);
@@ -711,6 +711,10 @@ describe('notDomain', function() {
                 actions: {
                   count: 2,
                   list: ['user//list', 'role//list']
+                },
+                forms: {
+                  count: 3,
+                  list: ['user//listAll', 'user//list', 'role//list']
                 }
               };
             }
@@ -737,10 +741,18 @@ describe('notDomain', function() {
               actions: {
                 count: 2,
                 list: ['user//list', 'role//list']
+              },
+              forms: {
+                count: 3,
+                list: ['user//listAll', 'user//list', 'role//list']
               }
             }
           }
         },
+        forms: {
+          count: 3,
+          list: ['not-user//user//listAll', 'not-user//user//list', 'not-user//role//list']
+        },
         routes: {
           count: 2,
           list: ["not-user//user","not-user//role"]

package/test/notModule.js

@@ -468,6 +468,9 @@ describe('notModule', function() {
         routes: {
           userRoute: {}
         },
+        forms: {
+          'route//action':{}
+        },
         getModelsStatuses() {
           return {
             user: {
@@ -505,6 +508,10 @@ describe('notModule', function() {
         actions: {
           count: 1,
           list: ['route//action']
+        },
+        forms: {
+          count: 1,
+          list: ['route//action']
         }
       });
     });