fix(rome_js_formatter): Union in mapped type (#4100)

rome · Dec 23, 2022 · 84812f0 · 84812f0
1 parent 846ac95
commit 84812f0
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 9 deletions.
diff --git a/crates/rome_js_formatter/src/ts/types/union_type.rs b/crates/rome_js_formatter/src/ts/types/union_type.rs
@@ -53,6 +53,7 @@ impl FormatNodeRule<TsUnionType> for FormatTsUnionType {
                                 | JsSyntaxKind::TS_TUPLE_TYPE_ELEMENT_LIST
                                 | JsSyntaxKind::TS_TYPE_ASSERTION_ASSIGNMENT
                                 | JsSyntaxKind::TS_TYPE_ARGUMENT_LIST
+                                | JsSyntaxKind::TS_MAPPED_TYPE
                         )
                     )
                 }

diff --git a/crates/rome_js_formatter/tests/specs/ts/type/union_type.ts b/crates/rome_js_formatter/tests/specs/ts/type/union_type.ts
@@ -12,7 +12,7 @@ type Comments =
 trailing type */
 
 type A = [
-    /*leading comment with new line*/ 
+    /*leading comment with new line*/
     A | B,
     ];
 
@@ -243,4 +243,4 @@ type GetChatsSagaEffects =
 type SuperLongTypeNameLoremIpsumLoremIpsumBlaBlaBlaBlaBlaBlaBlaBlaBlaBlaBlaBla =
 | Fooo1000
 | Baz2000
-| BarLoooooooooooooooooooooooooooooooooooooooooooooooooLong;
+| BarLoooooooooooooooooooooooooooooooooooooooooooooooooLong;
diff --git a/crates/rome_js_formatter/tests/specs/ts/type/union_type.ts.snap b/crates/rome_js_formatter/tests/specs/ts/type/union_type.ts.snap
@@ -1,7 +1,6 @@
 ---
 source: crates/rome_formatter_test/src/snapshot_builder.rs
-info:
-  test_file: ts/type/union_type.ts
+info: "ts\\type\\union_type.ts"
 ---
 
 # Input
@@ -21,7 +20,7 @@ type Comments =
 trailing type */
 
 type A = [
-    /*leading comment with new line*/ 
+    /*leading comment with new line*/
     A | B,
     ];
 
@@ -253,6 +252,7 @@ type SuperLongTypeNameLoremIpsumLoremIpsumBlaBlaBlaBlaBlaBlaBlaBlaBlaBlaBlaBla =
 | Fooo1000
 | Baz2000
 | BarLoooooooooooooooooooooooooooooooooooooooooooooooooLong;
+
 ```
 
 
@@ -377,10 +377,10 @@ const fooo: SomeThingWithShortMappedType<{
 
 const fooo: SomeThingWithLongMappedType<{
 	[P in
-			| AAAAAAAAAAAAAAAAA
-			| BBBBBBBBBBBB
-			| CCCCCCCCCCCCCCCCCCCCC
-			| DDDDDDDDDDDDDDDDDDDDDDDDDDDDD]: number;
+		| AAAAAAAAAAAAAAAAA
+		| BBBBBBBBBBBB
+		| CCCCCCCCCCCCCCCCCCCCC
+		| DDDDDDDDDDDDDDDDDDDDDDDDDDDDD]: number;
 }> = {};
 
 export type A =