yudarity - npm Package Compare versions

Comparing version 0.1.2 to 0.1.3

arity.js

@@ -6,3 +6,9 @@ module.exports = function (func) {
     }
-    return result[1].replace(/\{[^\}]*\}/g, 'param') .split(/\s*,\s*/).length; // eslint-disable-line
+
+    var argumentStr = result[1].trim();
+    if (argumentStr.substring(argumentStr.length - 1, argumentStr.length) === ',') {
+        argumentStr = argumentStr.substring(0, argumentStr.length - 1);
+    }
+
+    return argumentStr.replace(/\{[^\}]*\}/g, 'param') .split(/\s*,\s*/).length; // eslint-disable-line
 }

package.json

 {
   "name": "yudarity",
   "description": "Returns function arity (number of possible arguments) supports all ES6 features including argument destructor and default.",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "author": "Roee Yudilevtich ([e-mail])",
@@ -14,3 +14,4 @@ "devDependencies": {
     "verbose": true,
-    "testEnvironment": "node"
+    "testEnvironment": "node",
+    "transform": {}
   },
@@ -17,0 +18,0 @@ "repository": "https://github.com/roeeyud/arity",

test/arity.test.js

@@ -37,2 +37,33 @@ const arity = require('../arity');
     });
+
+    it('"," at the end', function () {
+        function testFn(
+            id,
+            options,
+            a,
+            c,
+            ) {
+            return dispatch({
+                resource: 'a',
+                method: 'b',
+                path: 'c',
+                payload: id,
+                options,
+            });
+        }
+
+        expect(arity(testFn)).toBe(4);
+    });
+
+    it('arrow', function () {
+        expect(arity((id, options, a, c) => {
+            return dispatch({
+                resource: 'a',
+                method: 'b',
+                path: 'c',
+                payload: id,
+                options,
+            });
+        })).toBe(4);
+    });
 });

No alert changes

Improved metrics

Total package byte prevSize

122733

increased by5.96%

Lines of code

70

increased by84.21%

Worsened metrics

Number of package files

13

decreased by-7.14%

No dependency changes