fix(): Fix issue #120

Default values are now returned as strings instead of raw values when parameter default value
consists of a list of expressions. This is the expected behavior a the lib. Plus, the fix is now
much cleaner.
TAG: latest

fixes #120

Author: ericmorand-emakina

src/lib/plugins/params.js

@@ -34,16 +34,12 @@ export default (app) => (node, result) => {
     result.args.push(name)
 
     if (param.right && param.right.type === 'SequenceExpression') {
-      let value
       let lastExpression = param.right.expressions.pop()
 
-      if (lastExpression.type === 'NullLiteral') {
-        value = null
-      } else {
-        value = lastExpression.value
-      }
-
-      result.defaults[name] = value
+      result.defaults[name] = result.value.slice(
+        lastExpression.start,
+        lastExpression.end
+      )
     } else {
       result.defaults[name] = param.right
         ? result.value.slice(param.right.start, param.right.end)

test/index.js

@@ -31,6 +31,7 @@ const actuals = {
     'function () {}',
     'function (a = (true, false)) {}',
     'function (a = (true, null)) {}',
+    'function (a = (true, "bar")) {}',
     'function (a, b = (i++, true)) {}',
   ],
   named: [
@@ -41,6 +42,7 @@ const actuals = {
     'function namedFn () {}',
     'function namedFn(a = (true, false)) {}',
     'function namedFn(a = (true, null)) {}',
+    'function namedFn(a = (true, "bar")) {}',
     'function namedFn(a, b = (i++, true)) {}',
   ],
   generators: [
@@ -51,6 +53,7 @@ const actuals = {
     'function * namedFn () {}',
     'function * namedFn(a = (true, false)) {}',
     'function * namedFn(a = (true, null)) {}',
+    'function * namedFn(a = (true, "bar")) {}',
     'function * namedFn(a, b = (i++, true)) {}',
   ],
   arrows: [
@@ -61,6 +64,7 @@ const actuals = {
     '() => {}',
     '(a = (true, false)) => {}',
     '(a = (true, null)) => {}',
+    '(a = (true, "bar")) => {}',
     '(a, b = (i++, true)) => {}',
     '(a) => a * 3 * a',
     'd => d * 355 * d',
@@ -126,21 +130,28 @@ const regulars = [
     params: 'a',
     args: ['a'],
     body: '',
-    defaults: {a: false},
+    defaults: {a: 'false'},
   },
   {
     name: null,
     params: 'a',
     args: ['a'],
     body: '',
-    defaults: {a: null},
+    defaults: {a: 'null'},
+  },
+  {
+    name: null,
+    params: 'a',
+    args: ['a'],
+    body: '',
+    defaults: {a: '"bar"'},
   },
   {
     name: null,
     params: 'a, b',
     args: ['a', 'b'],
     body: '',
-    defaults: {a: undefined, b: true},
+    defaults: {a: undefined, b: 'true'},
   },
 ]