From bfa3d2435e5e46c3666d84f04cf77dbc46768841 Mon Sep 17 00:00:00 2001 From: "Christopher J. Brody" Date: Sun, 3 Jan 2021 18:37:42 -0500 Subject: [PATCH] prettierx: test formatting of nested ternary promise expressions check for consistent formatting on prettierx side as discussed in: - https://github.com/eslint/eslint/issues/13971 - https://github.com/standard/standard/issues/1624 - https://github.com/brodybits/prettierx/issues/41 --- .../__snapshots__/jsfmt.spec.js.snap | 168 +++++++++++++++++ .../nested-ternary-promises.js | 14 ++ .../__snapshots__/jsfmt.spec.js.snap | 176 ++++++++++++++++++ 3 files changed, 358 insertions(+) create mode 100644 tests/ternary-object-expressions/nested-ternary-promises.js diff --git a/tests/ternary-object-expressions/__snapshots__/jsfmt.spec.js.snap b/tests/ternary-object-expressions/__snapshots__/jsfmt.spec.js.snap index 7485646822..31bd406db3 100644 --- a/tests/ternary-object-expressions/__snapshots__/jsfmt.spec.js.snap +++ b/tests/ternary-object-expressions/__snapshots__/jsfmt.spec.js.snap @@ -135,3 +135,171 @@ const dress = isSpace ================================================================================ `; + +exports[`nested-ternary-promises.js 1`] = ` +====================================options===================================== +alignTernaryLines: true +parsers: ["babel", "babel-flow", "flow", "typescript"] +printWidth: 80 + | printWidth +=====================================input====================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4) +} + +=====================================output===================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4); +} + +================================================================================ +`; + +exports[`nested-ternary-promises.js 2`] = ` +====================================options===================================== +alignTernaryLines: true +parsers: ["babel", "babel-flow", "flow", "typescript"] +printWidth: 80 +tabWidth: 4 + | printWidth +=====================================input====================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4) +} + +=====================================output===================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4); +} + +================================================================================ +`; + +exports[`nested-ternary-promises.js 3`] = ` +====================================options===================================== +alignTernaryLines: true +parsers: ["babel", "babel-flow", "flow", "typescript"] +printWidth: 80 +useTabs: true + | printWidth +=====================================input====================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4) +} + +=====================================output===================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4); +} + +================================================================================ +`; + +exports[`nested-ternary-promises.js 4`] = ` +====================================options===================================== +alignTernaryLines: true +parsers: ["babel", "babel-flow", "flow", "typescript"] +printWidth: 80 +tabWidth: 4 +useTabs: true + | printWidth +=====================================input====================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4) +} + +=====================================output===================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4); +} + +================================================================================ +`; diff --git a/tests/ternary-object-expressions/nested-ternary-promises.js b/tests/ternary-object-expressions/nested-ternary-promises.js new file mode 100644 index 0000000000..59fd9d27aa --- /dev/null +++ b/tests/ternary-object-expressions/nested-ternary-promises.js @@ -0,0 +1,14 @@ +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4) +} diff --git a/tests/ternary-object-expressions/with-balanced-formatting/__snapshots__/jsfmt.spec.js.snap b/tests/ternary-object-expressions/with-balanced-formatting/__snapshots__/jsfmt.spec.js.snap index aa06446e6a..8ebf96fdd9 100644 --- a/tests/ternary-object-expressions/with-balanced-formatting/__snapshots__/jsfmt.spec.js.snap +++ b/tests/ternary-object-expressions/with-balanced-formatting/__snapshots__/jsfmt.spec.js.snap @@ -143,3 +143,179 @@ const dress = isSpace ================================================================================ `; + +exports[`nested-ternary-promises.js 1`] = ` +====================================options===================================== +alignTernaryLines: false +arrowParens: "avoid" +parsers: ["babel", "babel-flow", "flow", "typescript"] +printWidth: 80 +trailingComma: "none" + | printWidth +=====================================input====================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4) +} + +=====================================output===================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4); +} + +================================================================================ +`; + +exports[`nested-ternary-promises.js 2`] = ` +====================================options===================================== +alignTernaryLines: false +arrowParens: "avoid" +parsers: ["babel", "babel-flow", "flow", "typescript"] +printWidth: 80 +tabWidth: 4 +trailingComma: "none" + | printWidth +=====================================input====================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4) +} + +=====================================output===================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4); +} + +================================================================================ +`; + +exports[`nested-ternary-promises.js 3`] = ` +====================================options===================================== +alignTernaryLines: false +arrowParens: "avoid" +parsers: ["babel", "babel-flow", "flow", "typescript"] +printWidth: 80 +trailingComma: "none" +useTabs: true + | printWidth +=====================================input====================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4) +} + +=====================================output===================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4); +} + +================================================================================ +`; + +exports[`nested-ternary-promises.js 4`] = ` +====================================options===================================== +alignTernaryLines: false +arrowParens: "avoid" +parsers: ["babel", "babel-flow", "flow", "typescript"] +printWidth: 80 +tabWidth: 4 +trailingComma: "none" +useTabs: true + | printWidth +=====================================input====================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4) +} + +=====================================output===================================== +// check for consistent formatting as discussed in: +// - https://github.com/eslint/eslint/issues/13971 +// - https://github.com/standard/standard/issues/1624 +// - https://github.com/brodybits/prettierx/issues/41 + +function test1() { + return condition1 + ? condition2 + ? Promise(1) + : Promise(2) + : condition3 + ? Promise(3) + : Promise(4); +} + +================================================================================ +`;