ajgenesisnode-entity - npm Package Compare versions

Comparing version 0.0.5 to 0.0.6

add.js

@@ -8,5 +8,5 @@
     
-    ajgenesis.createDirectory('ajgenesis', 'models');
+    ajgenesis.createModelDirectory();
     var template = path.join(__dirname, 'templates', 'entity.json.tpl');
-    var filename = path.join('ajgenesis', 'models', entityname + '.json');
+    var filename = path.join(ajgenesis.getModelDirectory(), entityname + '.json');
     
@@ -13,0 +13,0 @@ var entitymodel = {

addproperty.js

@@ -24,3 +24,3 @@ var path = require('path'),
     
-    var filename = path.join('ajgenesis', 'models', entityname + '.json');
+    var filename = path.join(ajgenesis.getModelDirectory(), entityname + '.json');
     var model = require(path.resolve(filename));
@@ -27,0 +27,0 @@

lib/names.js

@@ -45,3 +45,3 @@
     if (position < 0)
-        return { name: parval, value: null };
+        return { name: parval, value: true };
         
@@ -57,2 +57,4 @@ var name = parval.substring(0, position);
         value = false;
+    else if (value == "null")
+        value = null;
         
@@ -59,0 +61,0 @@ return { name: name, value: value };

package.json

 { "name": "ajgenesisnode-entity"
 , "description": "AjGenesis for Node, Entity tasks and templates"
 , "keywords": [ "codegeneration", "nodejs", "ajgenesis" ]
-, "version": "0.0.5"
+, "version": "0.0.6"
 , "author": "Angel 'Java' Lopez <[e-mail]> (http://www.ajlopez.com)"
 , "repository": { "type": "git", "url": "git://github.com/ajlopez/AjGenesisNode-Entity.git" }
 , "main": "./index.js"
-, "engines": { "node": ">= 0.6.0 && < 0.11.0" }
+, "engines": { "node": ">= 0.6.0 && < 0.13.0" }
 , "scripts": {
@@ -15,4 +15,4 @@ "test": "simpleunit ./test"
 , "devDependencies": {
-    "simpleunit": "0.0.2"
+    "simpleunit": "0.0.4"
   }
 }

README.md

@@ -45,6 +45,7 @@ # AjGenesisNode Entity
 - 0.0.1: Published
-- 0.0.2: Published. It uses `models` directory instead of `ajgenesis/models`.
-- 0.0.3: Published. add and addproperty process parameters.
-- 0.0.4: Published. Fixing add property bug, using a hack.
-- 0.0.5: Using `ajgenesis/models` for model files.
+- 0.0.2: Published. It uses `models` directory instead of `ajgenesis/models`
+- 0.0.3: Published. add and addproperty process parameters
+- 0.0.4: Published. Fixing add property bug, using a hack
+- 0.0.5: Using `ajgenesis/models` for model files
+- 0.0.6: Use createModelDirectory, getModelDirectory
 
@@ -51,0 +52,0 @@ ## Contribution

test/add.js

@@ -5,2 +5,3 @@
     fs = require('fs'),
+    fsutils = require('./lib/fsutils'),
     ajgenesis = require('ajgenesis');
@@ -32,4 +33,4 @@
         test.equal(entity.setdescriptor, "Customers");
-        
-        removeDirSync(path.join(__dirname, 'ajgenesis'));
+
+        fsutils.removeDirSync(path.join(__dirname, 'ajgenesis'));
             
@@ -68,3 +69,3 @@ test.done();
         
-        removeDirSync(path.join(__dirname, 'ajgenesis'));
+        fsutils.removeDirSync(path.join(__dirname, 'ajgenesis'));
             
@@ -76,32 +77,1 @@ test.done();
 }
-
-function removeDirSync(dirname) {
-    var filenames = fs.readdirSync(dirname);
-    
-    filenames.forEach(function (filename) {
-        filename = path.join(dirname, filename);
-        
-        if (isDirectory(filename))
-            removeDirSync(filename);
-        else
-            removeFileSync(filename);
-    });
-    
-    fs.rmdirSync(dirname);
-}
-
-function removeFileSync(filename) {
-    fs.unlinkSync(filename);
-}
-
-function isDirectory(filename)
-{
-    try {
-        var stats = fs.lstatSync(filename);
-        return stats.isDirectory();
-    }
-    catch (err)
-    {
-        return false;
-    }
-}

test/addproperty.js

@@ -5,2 +5,3 @@ var addpropertytask = require('../addproperty'),
     fs = require('fs'),
+    fsutils = require('./lib/fsutils'),
     ajgenesis = require('ajgenesis');
@@ -43,3 +44,3 @@
             
-            removeDirSync(path.join(__dirname, 'ajgenesis'));
+            fsutils.removeDirSync(path.join(__dirname, 'ajgenesis'));
             test.done();
@@ -101,3 +102,3 @@ });
                 
-                removeDirSync(path.join(__dirname, 'ajgenesis'));
+                fsutils.removeDirSync(path.join(__dirname, 'ajgenesis'));
                     
@@ -111,32 +112,1 @@ test.done();
 }
-
-function removeDirSync(dirname) {
-    var filenames = fs.readdirSync(dirname);
-    
-    filenames.forEach(function (filename) {
-        filename = path.join(dirname, filename);
-        
-        if (isDirectory(filename))
-            removeDirSync(filename);
-        else
-            removeFileSync(filename);
-    });
-    
-    fs.rmdirSync(dirname);
-}
-
-function removeFileSync(filename) {
-    fs.unlinkSync(filename);
-}
-
-function isDirectory(filename)
-{
-    try {
-        var stats = fs.lstatSync(filename);
-        return stats.isDirectory();
-    }
-    catch (err)
-    {
-        return false;
-    }
-}

test/names.js

@@ -70,9 +70,18 @@
 
-exports['Get parameter without value'] = function (test) {
-    var result = names.getParameterValue("name");
+exports['Get parameter without value as boolean flag'] = function (test) {
+    var result = names.getParameterValue("required");
     
     test.ok(result);
-    test.equal(result.name, "name");
-    test.equal(result.value, null);
+    test.equal(result.name, "required");
+    test.strictEqual(result.value, true);
 };
 
+exports['Get parameter with null value'] = function (test) {
+    var result = names.getParameterValue("email=null");
+    
+    test.ok(result);
+    test.equal(result.name, "email");
+    test.strictEqual(result.value, null);
+};
+
+

No alert changes

Improved metrics

Total package byte prevSize

17243

increased by1.73%

Number of package files

12

increased by9.09%

Lines of code

351

increased by1.74%

Number of lines in readme file

59

increased by1.72%

Worsened metrics

Number of low supply chain risk alerts

5

increased by25%

No dependency changes