indent return arguments wrapped into parenthesis. fixes #465

Author: millermedeiros

lib/hooks/ReturnStatement.js

@@ -36,16 +36,21 @@ exports.getIndentEdges = function(node, opts) {
     return false;
   }
 
-  var parentheses = expressionParentheses.getParentheses(node.argument);
-  return parentheses ?
-    {
-      startToken: parentheses.opening,
-      endToken: parentheses.closing
-    } :
-    {
-      startToken: node.startToken.next,
-      endToken: _tk.isEmpty(node.endToken) ?
-        _tk.findPrevNonEmpty(node.endToken) :
-        node.endToken
+  // handle agument wrapped in parenthesis
+  var argumentStart = _tk.findNextNonEmpty(node.startToken);
+  if (argumentStart.value === '(') {
+    return {
+      startToken: argumentStart,
+      endToken: node.endToken.value === ')' ?
+        node.endToken :
+        _tk.findPrevNonEmpty(_tk.findPrev(node.endToken, ')'))
     };
+  }
+
+  return {
+    startToken: node.startToken.next,
+    endToken: _tk.isEmpty(node.endToken) ?
+      _tk.findPrevNonEmpty(node.endToken) :
+      node.endToken
+  };
 };

test/compare/default/function_declaration-in.js

@@ -119,3 +119,15 @@ function test(arr) {
 // #458
 function flow(one: OneType<string>, two: TwoTypes<string, string>    ,    three: string) {
 }
+
+// #465
+function indentReturnParentheses() {
+  if (bar) {
+    return (
+      bar
+    )
+  }
+  return (
+    dolorAmet
+  );
+}

test/compare/default/function_declaration-out.js

@@ -130,3 +130,15 @@ function test(arr) {
 // #458
 function flow(one: OneType<string>, two: TwoTypes<string, string>, three: string) {
 }
+
+// #465
+function indentReturnParentheses() {
+  if (bar) {
+    return (
+      bar
+    )
+  }
+  return (
+    dolorAmet
+  );
+}