buggy grouping

vitri-ent · Oct 21, 2023 · 6eac6f1 · 6eac6f1
1 parent 8604116
commit 6eac6f1
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/ts-rs/src/lib.rs b/ts-rs/src/lib.rs
@@ -440,11 +440,11 @@ impl<T: TS> TS for Option<T> {
             "called Option::name_with_type_args with {} args",
             args.len()
         );
-        format!("{} | null", args[0])
+        format!("({} | null)", args[0])
     }
 
     fn inline() -> String {
-        format!("{} | null", T::inline())
+        format!("({} | null)", T::inline())
     }
 
     fn dependencies() -> Vec<Dependency>
@@ -471,11 +471,11 @@ impl<T: TS, E: TS> TS for Result<T, E> {
             "called Result::name_with_type_args with {} args",
             args.len()
         );
-        format!("{{ ok: {} }} | {{ err: {} }}", args[0], args[1])
+        format!("({{ ok: {} }} | {{ err: {} }})", args[0], args[1])
     }
 
     fn inline() -> String {
-        format!("{{ ok: {} }} | {{ err: {} }}", T::inline(), E::inline())
+        format!("({{ ok: {} }} | {{ err: {} }})", T::inline(), E::inline())
     }
 
     fn dependencies() -> Vec<Dependency>

diff --git a/ts-rs/tests/optional_field.rs b/ts-rs/tests/optional_field.rs
@@ -14,7 +14,7 @@ struct Optional {
 #[test]
 fn test() {
     #[cfg(not(feature = "serde-compat"))]
-    assert_eq!(Optional::inline(), "{ a?: number, b: string | null, }");
+    assert_eq!(Optional::inline(), "({ a?: number, b: string | null, })");
     #[cfg(feature = "serde-compat")]
-    assert_eq!(Optional::inline(), "{ a?: number, b?: string, }")
+    assert_eq!(Optional::inline(), "({ a?: number, b?: string, })")
 }
diff --git a/ts-rs/tests/result.rs b/ts-rs/tests/result.rs
@@ -11,6 +11,6 @@ struct Fallible {
 fn test() {
     assert_eq!(
         Fallible::inline(),
-        "{ a: { ok: number } | { err: string }, }"
+        "({ a: { ok: number } | { err: string }, })"
     )
 }