Fix for the conditional formating (#132)

Author: Janther

src/nodes/Conditional.js

@@ -1,19 +1,23 @@
 /* eslint-disable implicit-arrow-linebreak */
 const {
   doc: {
-    builders: { join }
+    builders: { concat, group, indent, line }
   }
 } = require('prettier');
 
 const Conditional = {
   print: ({ path, print }) =>
-    join(' ', [
-      path.call(print, 'condition'),
-      '?',
-      path.call(print, 'trueExpression'),
-      ':',
-      path.call(print, 'falseExpression')
-    ])
+    group(
+      concat([
+        path.call(print, 'condition'),
+        indent(line),
+        '? ',
+        path.call(print, 'trueExpression'),
+        indent(line),
+        ': ',
+        path.call(print, 'falseExpression')
+      ])
+    )
 };
 
 module.exports = Conditional;

tests/Conditional/Conditional.sol

@@ -0,0 +1,17 @@
+pragma solidity ^0.4.24;
+
+
+contract Conditional {
+    function foo() {
+        address contextAddress = longAddress_ == address(0) ? msg.sender : currentContextAddress_;
+    }
+
+    // TODO: work with a break in the condition level.
+    // function foo() {
+    //     address contextAddress = veryVeryVeryVeryVeryVeryVeryVeryVeryLongAddress_ == address(0) ? msg.sender : currentContextAddress_;
+    // }
+
+    function bar() {
+        uint a = true ? 0 : 1;
+    }
+}

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

@@ -0,0 +1,41 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Conditional.sol 1`] = `
+pragma solidity ^0.4.24;
+
+
+contract Conditional {
+    function foo() {
+        address contextAddress = longAddress_ == address(0) ? msg.sender : currentContextAddress_;
+    }
+
+    // TODO: work with a break in the condition level.
+    // function foo() {
+    //     address contextAddress = veryVeryVeryVeryVeryVeryVeryVeryVeryLongAddress_ == address(0) ? msg.sender : currentContextAddress_;
+    // }
+
+    function bar() {
+        uint a = true ? 0 : 1;
+    }
+}
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+pragma solidity ^0.4.24;
+
+contract Conditional {
+    function foo() {
+        address contextAddress = longAddress_ == address(0)
+            ? msg.sender
+            : currentContextAddress_;
+    }
+
+    // TODO: work with a break in the condition level.
+    // function foo() {
+    //     address contextAddress = veryVeryVeryVeryVeryVeryVeryVeryVeryLongAddress_ == address(0) ? msg.sender : currentContextAddress_;
+    // }
+
+    function bar() {
+        uint a = true ? 0 : 1;
+    }
+}
+
+`;

tests/Conditional/jsfmt.spec.js

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