avoid indenting body of arrow functions twice. fixes #357

lib/hooks/ArrowFunctionExpression.js

@@ -21,9 +21,10 @@ exports.format = function ArrowFunctionExpression(node) {
 };
 
 exports.getIndentEdges = function(node, opts) {
-  var edges = [
-    node.body
-  ];
+  var edges = [];
+  if (shouldIndentBody(node, opts)) {
+    edges.push(node.body);
+  }
   if (shouldHandleParams(node)) {
     edges.push(_params.getIndentEdges(node, opts));
   }
@@ -35,3 +36,9 @@ function shouldHandleParams(node) {
   // we don't check based on `node.params` because of `node.defaults`
   return tk.findPrevNonEmpty(arrow).value === ')';
 }
+
+function shouldIndentBody(node, opts) {
+  // we don't want to indent the body twice if ObjectExpression or
+  // ArrayExpression or CallExpression
+  return node.body.type === 'BlockStatement' || !opts[node.body.type];
+}

test/compare/default/arrow_function_expression-in.js

@@ -21,3 +21,22 @@ a = () => {
     return 1
   })
 }
+
+// issue #357
+const object = x => ({
+  x
+});
+
+const retObject = x => {
+  return {
+    x
+  };
+}
+
+const array = x => [
+  x
+];
+
+const callWithObject = x => f({
+  x
+});

test/compare/default/arrow_function_expression-out.js

@@ -24,3 +24,22 @@ a = () => {
     return 1
   })
 }
+
+// issue #357
+const object = x => ({
+  x
+});
+
+const retObject = x => {
+  return {
+    x
+  };
+}
+
+const array = x => [
+  x
+];
+
+const callWithObject = x => f({
+  x
+});