Use ifBreak instead of shouldBreak (#2063)

shouldBreak is only a heuristic to enforce a break, if it breaks "naturally", then it won't be applied.

Fixes #2055

src/printer.js

@@ -840,7 +840,7 @@ function genericPrintNoParens(path, options, print, args) {
         );
       const separator = n.type === "TSInterfaceBody" ||
         n.type === "TSTypeLiteral"
-        ? shouldBreak ? semi : ";"
+        ? ifBreak(semi, ";")
         : ",";
       const fields = [];
       const leftBrace = n.exact ? "{|" : "{";

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

@@ -36,7 +36,17 @@ interface InlineMultiple { x: string; y: string }
 
 exports[`type.js 1`] = `
 type A = {disabled?: boolean, error?: string}
+
+const foo: {aasdf?: string; asdf?: number; foob?: string; zzz?: string; yyy: string}  = {}
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 type A = { disabled?: boolean; error?: string }
 
+const foo: {
+  aasdf?: string
+  asdf?: number
+  foob?: string
+  zzz?: string
+  yyy: string
+} = {}
+
 `;

tests/typescript_nosemi/type.js

@@ -1 +1,3 @@
 type A = {disabled?: boolean, error?: string}
+
+const foo: {aasdf?: string; asdf?: number; foob?: string; zzz?: string; yyy: string}  = {}