Align empty module declaration with TS (#19568)

changelog_unreleased/flow/19568.md

@@ -0,0 +1,14 @@
+#### Stop enforcing empty module declaration to break (#19568 by @fisker)
+
+<!-- prettier-ignore -->
+```flow
+// Input
+declare module "foo" {}
+
+// Prettier stable
+declare module "foo" {
+}
+
+// Prettier main
+declare module "foo" {}
+```

src/language-js/print/block.js

@@ -48,6 +48,7 @@ function printBlock(path, options, print) {
       parent.type === "ModuleExpression" ||
       (parent.type === "CatchClause" && !parentParent.finalizer) ||
       parent.type === "TSModuleDeclaration" ||
+      parent.type === "DeclareModule" ||
       parent.type === "MatchStatementCase" ||
       node.type === "StaticBlock"
     )) {

tests/format/flow/flow-repo/coverage/__snapshots__/format.test.js.snap

@@ -22,11 +22,9 @@ declare module bar {
 // that we implicitly assume in type-at-pos and coverage implementations. In
 // particular, when unchecked it causes a crash with coverage --color.
 
-declare module foo {
+declare module foo {}
-}
 
-declare module bar {
+declare module bar {}
-}
 
 // TODO
 
@@ -46,8 +44,7 @@ declare module foo {
 =====================================output=====================================
 // check coverage of declare module
 
-declare module foo {
+declare module foo {}
-}
 
 ================================================================================
 `;
@@ -92,11 +89,9 @@ declare class qux {
 // that we implicitly assume in type-at-pos and coverage implementations. In
 // particular, when unchecked it causes non-termination with coverage --color.
 
-declare module foo {
+declare module foo {}
-}
 
-declare module bar {
+declare module bar {}
-}
 
 // TODO
 

tests/format/flow/type-declarations/__snapshots__/format.test.js.snap

@@ -44,8 +44,7 @@ declare module "foo" {
   declare var bar: string
 }
 
-declare module baz {
+declare module baz {}
-}
 
 ================================================================================
 `;
@@ -66,8 +65,7 @@ declare module "foo" {
   declare var bar: string;
 }
 
-declare module baz {
+declare module baz {}
-}
 
 ================================================================================
 `;

tests/format/typescript/declare/declare-module-empty/__snapshots__/format.test.js.snap

@@ -0,0 +1,14 @@
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
+
+exports[`empty.ts format 1`] = `
+====================================options=====================================
+parsers: ["typescript", "flow"]
+                                                      printWidth: 80 (default) |
+=====================================input======================================
+declare module "foo" {}
+
+=====================================output=====================================
+declare module "foo" {}
+
+================================================================================
+`;

tests/format/typescript/declare/declare-module-empty/empty.ts

@@ -0,0 +1 @@
+declare module "foo" {}

tests/format/typescript/declare/declare-module-empty/format.test.js

@@ -0,0 +1 @@
+runFormatTest(import.meta, ["typescript", "flow"]);