adding commas between arguments and indenting the arguments (#212)

Author: Janther

src/nodes/InheritanceSpecifier.js

@@ -1,21 +1,32 @@
 const {
   doc: {
-    builders: { concat }
+    builders: { concat, group, indent, join, line, softline }
   }
 } = require('prettier/standalone');
 
-const InheritanceSpecifier = {
-  print: ({ node, path, print }) => {
-    let parts = [path.call(print, 'baseName')];
+const printArguments = (node, path, print) => {
+  if (node.arguments && node.arguments.length) {
+    return group(
+      concat([
+        '(',
+        indent(
+          concat([
+            softline,
+            join(concat([',', line]), path.map(print, 'arguments'))
+          ])
+        ),
+        softline,
+        ')'
+      ])
+    );
+  }
 
-    if (node.arguments && node.arguments.length) {
-      parts.push('(');
-      parts = parts.concat(path.map(print, 'arguments'));
-      parts.push(')');
-    }
+  return '';
+};
 
-    return concat(parts);
-  }
+const InheritanceSpecifier = {
+  print: ({ node, path, print }) =>
+    concat([path.call(print, 'baseName'), printArguments(node, path, print)])
 };
 
 module.exports = InheritanceSpecifier;

tests/InheritanceSpecifier/InheritanceSpecifier.sol

@@ -0,0 +1,2 @@
+contract InheritanceSpecifier is SomeOtherContract(1234,false) {}
+contract LongInheritanceSpecifier is SomeOtherContract(123467890,false,0xCA35b7d915458EF540aDe6068dFe2F44E8fa733c) {}

tests/InheritanceSpecifier/__snapshots__/jsfmt.spec.js.snap

@@ -0,0 +1,16 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`InheritanceSpecifier.sol 1`] = `
+contract InheritanceSpecifier is SomeOtherContract(1234,false) {}
+contract LongInheritanceSpecifier is SomeOtherContract(123467890,false,0xCA35b7d915458EF540aDe6068dFe2F44E8fa733c) {}
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+contract InheritanceSpecifier is SomeOtherContract(1234, false) {}
+contract LongInheritanceSpecifier is
+    SomeOtherContract(
+        123467890,
+        false,
+        0xCA35b7d915458EF540aDe6068dFe2F44E8fa733c
+    )
+{}
+
+`;

tests/InheritanceSpecifier/jsfmt.spec.js

@@ -0,0 +1 @@
+run_spec(__dirname);