when having multiple base contracts we split the Base contracts and keep the is keyword in the first line. (#134)

Author: Janther

src/nodes/ContractDefinition.js

@@ -1,33 +1,53 @@
+/* eslint-disable implicit-arrow-linebreak */
 const {
   doc: {
-    builders: { concat, indent, join, line }
+    builders: { concat, group, indent, join, line }
   }
 } = require('prettier');
 const printPreservingEmptyLines = require('./print-preserving-empty-lines');
 
-const ContractDefinition = {
-  print: ({ node, options, path, print }) => {
-    let parts = [node.kind, ' ', node.name];
-
-    if (node.baseContracts.length > 0) {
-      parts = parts.concat([
-        ' is ',
-        join(', ', path.map(print, 'baseContracts'))
-      ]);
-    }
-
-    parts.push(' {');
-    if (node.subNodes.length > 0) {
-      parts = parts.concat([
-        indent(line),
-        indent(printPreservingEmptyLines(path, 'subNodes', options, print)),
-        line
-      ]);
-    }
-    parts.push('}');
+const inheritance = (node, path, print) => {
+  if (node.baseContracts.length > 0) {
+    return concat([
+      ' is',
+      indent(
+        concat([
+          line,
+          join(concat([',', line]), path.map(print, 'baseContracts'))
+        ])
+      )
+    ]);
+  }
+  return '';
+};
 
-    return concat(parts);
+const body = (node, path, options, print) => {
+  if (node.subNodes.length > 0) {
+    return concat([
+      indent(line),
+      indent(printPreservingEmptyLines(path, 'subNodes', options, print)),
+      line
+    ]);
   }
+  return '';
+};
+
+const ContractDefinition = {
+  print: ({ node, options, path, print }) =>
+    concat([
+      group(
+        concat([
+          node.kind,
+          ' ',
+          node.name,
+          inheritance(node, path, print),
+          line,
+          '{'
+        ])
+      ),
+      body(node, path, options, print),
+      '}'
+    ])
 };
 
 module.exports = ContractDefinition;

tests/ContractDefinitions/ContractDefinitions.sol

@@ -0,0 +1,2 @@
+contract ContractDefinition is Contract1, Contract2, Contract3, Contract4, Contract5 {
+}

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

@@ -0,0 +1,15 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`ContractDefinitions.sol 1`] = `
+contract ContractDefinition is Contract1, Contract2, Contract3, Contract4, Contract5 {
+}
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+contract ContractDefinition is
+    Contract1,
+    Contract2,
+    Contract3,
+    Contract4,
+    Contract5
+{}
+
+`;

tests/ContractDefinitions/jsfmt.spec.js

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