From 5c53b1de76a9ad803a63d052c2b581fbb19a6653 Mon Sep 17 00:00:00 2001
From: Chris Brody <chris.brody@gmail.com>
Date: Tue, 22 Jan 2019 12:21:44 -0500
Subject: [PATCH] [prettierx] cover standard ternary inconsistency (#42)

in tests/standard & docs

as reported in brodybits/prettierx#40

(this should have been part of the proposal in prettier/prettier#5723)

Note that this inconsistency is resolved by brodybits/prettierx#41
which is to be included in an upcoming merge.

Co-authored-by: Christopher J. Brody <chris.brody@gmail.com>
Co-authored-by: Mohit Singh <mohit@mohitsingh.in>
Co-authored-by: Adam Stankiewicz <sheerun@sher.pl>
---
 README.md                                     |  2 +
 .../standard/__snapshots__/jsfmt.spec.js.snap | 56 +++++++++++++++++++
 tests/standard/incorrect.js                   | 27 +++++++++
 3 files changed, 85 insertions(+)

diff --git a/README.md b/README.md
index 1ca44d1271..9416ff5b3b 100644
--- a/README.md
+++ b/README.md
@@ -38,6 +38,8 @@ The following options should be used to _format_ the code according to [standard
 - `--no-semi` (`semi: false`)
 - `--yield-star-spacing` (`yieldStarSpacing: true`)
 
+Known conflict with `standard` in ternary returning objects ([brodybits/prettierx#40](https://github.com/brodybits/prettierx/issues/40))
+
 Note that this tool does _not_ follow any of the other [standard js](https://standardjs.com/) rules. It is recommended to use this tool together with eslint, in some form, to achive correct formatting according to [standard js](https://standardjs.com/).
 
 <!-- - FUTURE TBD prettierx vs prettier (???):
diff --git a/tests/standard/__snapshots__/jsfmt.spec.js.snap b/tests/standard/__snapshots__/jsfmt.spec.js.snap
index 4b8804f202..0c802eb7a9 100644
--- a/tests/standard/__snapshots__/jsfmt.spec.js.snap
+++ b/tests/standard/__snapshots__/jsfmt.spec.js.snap
@@ -1069,6 +1069,33 @@ function*generator() {
 // "jsx-quotes": ["error", "prefer-single"],
 ;() => <div foo="bar" />
 
+// "indent": [
+//   "error",
+//   2,
+//   {
+//     "SwitchCase": 1,
+//     "VariableDeclarator": 1,
+//     "outerIIFEBody": 1,
+//     "MemberExpression": 1,
+//     "FunctionDeclaration": { "parameters": 1, "body": 1 },
+//     "FunctionExpression": { "parameters": 1, "body": 1 },
+//     "CallExpression": { "arguments": 1 },
+//     "ArrayExpression": 1,
+//     "ObjectExpression": 1,
+//     "ImportDeclaration": 1,
+//     "flatTernaryExpressions": false,
+//     "ignoreComments": false
+//   }
+// ],
+let isSpace = false
+const dress = isSpace ? {
+      spaceSuit: 3,
+      oxygenCylinders: 6
+    } : {
+      shirts: 3,
+      paints: 3
+    }
+
 =====================================output=====================================
 // "arrow-spacing": ["error", { "before": true, "after": true }]
 
@@ -1421,5 +1448,34 @@ function * generator () {
 // "jsx-quotes": ["error", "prefer-single"],
 ;() => <div foo='bar' />
 
+// "indent": [
+//   "error",
+//   2,
+//   {
+//     "SwitchCase": 1,
+//     "VariableDeclarator": 1,
+//     "outerIIFEBody": 1,
+//     "MemberExpression": 1,
+//     "FunctionDeclaration": { "parameters": 1, "body": 1 },
+//     "FunctionExpression": { "parameters": 1, "body": 1 },
+//     "CallExpression": { "arguments": 1 },
+//     "ArrayExpression": 1,
+//     "ObjectExpression": 1,
+//     "ImportDeclaration": 1,
+//     "flatTernaryExpressions": false,
+//     "ignoreComments": false
+//   }
+// ],
+let isSpace = false
+const dress = isSpace
+  ? {
+      spaceSuit: 3,
+      oxygenCylinders: 6
+    }
+  : {
+      shirts: 3,
+      paints: 3
+    }
+
 ================================================================================
 `;
diff --git a/tests/standard/incorrect.js b/tests/standard/incorrect.js
index ddafb8efb3..52e16fdd60 100644
--- a/tests/standard/incorrect.js
+++ b/tests/standard/incorrect.js
@@ -352,3 +352,30 @@ function*generator() {
 
 // "jsx-quotes": ["error", "prefer-single"],
 ;() => <div foo="bar" />
+
+// "indent": [
+//   "error",
+//   2,
+//   {
+//     "SwitchCase": 1,
+//     "VariableDeclarator": 1,
+//     "outerIIFEBody": 1,
+//     "MemberExpression": 1,
+//     "FunctionDeclaration": { "parameters": 1, "body": 1 },
+//     "FunctionExpression": { "parameters": 1, "body": 1 },
+//     "CallExpression": { "arguments": 1 },
+//     "ArrayExpression": 1,
+//     "ObjectExpression": 1,
+//     "ImportDeclaration": 1,
+//     "flatTernaryExpressions": false,
+//     "ignoreComments": false
+//   }
+// ],
+let isSpace = false
+const dress = isSpace ? {
+      spaceSuit: 3,
+      oxygenCylinders: 6
+    } : {
+      shirts: 3,
+      paints: 3
+    }