futoin-invoker - npm Package Compare versions

Comparing version 1.4.1 to 1.4.2

.eslintrc.js

@@ -28,8 +28,16 @@ module.exports = {
         "no-multi-spaces": ["error"],
-        "array-bracket-newline": ["error", { "multiline": true }],
+        "array-bracket-newline": ["error", { "multiline": true, "minItems": 3 }],
         "array-bracket-spacing": ["error", "always"],
-        "array-element-newline": ["error", { "multiline": true }],
+        "array-element-newline": ["error", { "multiline": true, "minItems": 3 }],
         "block-spacing": ["error"],
         "brace-style": ["error", "allman"],
         "no-trailing-spaces": "error",
+        "no-unused-vars": [
+            "error",
+            {
+                "vars" : "all",
+                "args" : "all",
+                "argsIgnorePattern" : "^(as|err|_.*)$"
+            }
+        ],
         "no-useless-concat": "error",
@@ -36,0 +44,0 @@ "no-useless-return": "error",

CacheFace.js

@@ -17,6 +17,6 @@ 'use strict';
  * @augments NativeIface
- * @param {SimpleCCM} ccm - CCM instance
+ * @param {SimpleCCM} _ccm - CCM instance
  * @param {object} info - internal info
  */
-function CacheFace( ccm, info )
+function CacheFace( _ccm, info )
 {
@@ -23,0 +23,0 @@ NativeIface.apply( this, arguments );

CHANGELOG.txt

 
+=== 1.4.2 (2017-08-24) ===
+FIXED: improved error handling on spec loading
+
 === 1.4.1 (2017-08-19) ===
@@ -3,0 +6,0 @@ FIXED: check failure when "boolean" is used in type variant

futoin.json

 {
   "name": "futoin-invoker",
-  "version": "1.4.1",
+  "version": "1.4.2",
   "vcs": "git",
@@ -5,0 +5,0 @@ "rms": "npm",

Gruntfile.js

@@ -71,7 +71,16 @@ 'use strict';
     grunt.registerTask( 'build-browser', [ 'webpack' ] );
-    grunt.registerTask( 'test-browser', [ 'connect', 'external_daemon:unittest', 'mocha_phantomjs' ] );
+    grunt.registerTask( 'test-browser', [
+        'connect',
+        'external_daemon:unittest',
+        'mocha_phantomjs',
+    ] );
 
     grunt.registerTask( 'node', [ 'connect', 'mocha_istanbul' ] );
     grunt.registerTask( 'browser', [ 'build-browser', 'test-browser' ] );
-    grunt.registerTask( 'test', [ 'check', 'node', 'browser', 'doc' ] );
+    grunt.registerTask( 'test', [
+        'check',
+        'node',
+        'browser',
+        'doc',
+    ] );
 
@@ -78,0 +87,0 @@ grunt.loadNpmTasks( 'grunt-jsdoc-to-markdown' );

lib/node/comms.js

@@ -400,3 +400,3 @@ "use strict";
 
-        const cleanup = ( arg1, arg2 ) =>
+        const cleanup = ( _arg1, arg2 ) =>
         {
@@ -403,0 +403,0 @@ opts.disconnectSniffer( info );

lib/SimpleCCMImpl.js

@@ -131,3 +131,3 @@ "use strict";
 
-    onMessageResponse : function( as, ctx, rsp )
+    onMessageResponse : function( as, _ctx, rsp )
     {
@@ -144,3 +144,3 @@ if ( 'e' in rsp )
 
-    onDataResponse : function( as, ctx, rsp )
+    onDataResponse : function( as, _ctx, rsp )
     {
@@ -147,0 +147,0 @@ as.success( rsp );

package.json

 {
   "name": "futoin-invoker",
-  "version": "1.4.1",
+  "version": "1.4.2",
   "description": "FutoIn Invoker - Reference Implementation. Transparently and efficiently invoke remote or local service methods with strict API definition for Node and Browser",
@@ -41,3 +41,3 @@ "main": "lib/invoker.js",
     "faye-websocket": "^0.11.1",
-    "futoin-asyncsteps": "^1.6.2",
+    "futoin-asyncsteps": "^1.6.3",
     "lodash": "^4.17.4",
@@ -49,3 +49,3 @@ "performance-now": "^2.1.0",
     "chai": "^4.1.1",
-    "eslint": "^4.4.1",
+    "eslint": "^4.5.0",
     "grunt": "^1.0.1",
@@ -52,0 +52,0 @@ "grunt-contrib-connect": "^1.0.2",

SpecTools.js

@@ -101,3 +101,3 @@ "use strict";
 
-        as.forEach( specdirs, function( as, k, v )
+        as.forEach( specdirs, function( as, _k, v )
         {
@@ -115,3 +115,3 @@ // Check Node.js fs
 
-                var on_read = function( data )
+                var on_read = function( data, err )
                 {
@@ -123,13 +123,16 @@ if ( !read_as )
 
-                    try
+                    if ( !err )
                     {
-                        v = JSON.parse( data );
-                        v._just_loaded = true;
-                        read_as.success();
-                        return;
+                        try
+                        {
+                            v = JSON.parse( data );
+                            v._just_loaded = true;
+                            read_as.success();
+                            return;
+                        }
+                        catch ( e )
+                        {
+                            // ignore
+                        }
                     }
-                    catch ( e )
-                    {
-                        // ignore
-                    }
 
@@ -148,5 +151,5 @@ try
                 {
-                    request( uri, function( error, response, body )
+                    request( uri, function( error, _response, body )
                     {
-                        on_read( body );
+                        on_read( body, error );
                     } );
@@ -161,3 +164,3 @@ }
                         {
-                            on_read( data );
+                            on_read( data, err );
                         }
@@ -342,3 +345,3 @@ );
             {
-                spectools._parseImportInherit( as, info, specdirs, raw_spec, sup_info );
+                spectools._parseImportInherit( as, info, raw_spec, sup_info );
 
@@ -359,3 +362,3 @@ info.inherits.push( raw_spec.inherit );
 
-            as.forEach( raw_spec.imports, function( as, k, v )
+            as.forEach( raw_spec.imports, function( as, _k, v )
             {
@@ -438,3 +441,3 @@ var m = v.match( iface_pattern );
                         {
-                            spectools._parseImportInherit( as, info, specdirs, raw_spec, imp_info );
+                            spectools._parseImportInherit( as, info, raw_spec, imp_info );
                         } );
@@ -890,7 +893,6 @@ } );
      * @param {object} info - _
-     * @param {array} specdirs - _
      * @param {object} raw_spec - _
      * @param {object} sup_info - _
      */
-    _parseImportInherit : function( as, info, specdirs, raw_spec, sup_info )
+    _parseImportInherit : function( as, info, raw_spec, sup_info )
     {
@@ -897,0 +899,0 @@ var i;

No alert changes

Improved metrics

Total package byte prevSize

589606

increased by0.16%

Lines of code

8357

increased by0.23%

Dependency changes

• Updatedfutoin-asyncsteps@^1.6.3