Fix empty file format (#7921)

prettier · Apr 1, 2020 · 22e3004 · 22e3004
1 parent e199e68
commit 22e3004
Show file tree

Hide file tree

Showing 5 changed files with 34 additions and 3 deletions.
diff --git a/src/language-js/printer-estree.js b/src/language-js/printer-estree.js
@@ -494,7 +494,10 @@ function printPathNoParens(path, options, print, args) {
       );
 
       // Only force a trailing newline if there were any contents.
-      if (n.body.length || n.comments) {
+      if (
+        !n.body.every(({ type }) => type === "EmptyStatement") ||
+        n.comments
+      ) {
         parts.push(hardline);
       }
 

diff --git a/tests/js_empty/__snapshots__/jsfmt.spec.js.snap b/tests/js_empty/__snapshots__/jsfmt.spec.js.snap
@@ -1,15 +1,41 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`semicolon.js 1`] = `
+exports[`empty.js 1`] = `
+====================================options=====================================
+parsers: ["babel", "flow", "typescript"]
+printWidth: 80
+                                                                                | printWidth
+=====================================input======================================
+
+
+=====================================output=====================================
+
+================================================================================
+`;
+
+exports[`many-semicolons.js 1`] = `
 ====================================options=====================================
 parsers: ["babel", "flow", "typescript"]
 printWidth: 80
                                                                                 | printWidth
 =====================================input======================================
+;;
 ;
 
 =====================================output=====================================
 
+================================================================================
+`;
+
+exports[`semicolon.js 1`] = `
+====================================options=====================================
+parsers: ["babel", "flow", "typescript"]
+printWidth: 80
+                                                                                | printWidth
+=====================================input======================================
+;
+
+=====================================output=====================================
 
 ================================================================================
 `;
diff --git a/tests/js_empty/empty.js b/tests/js_empty/empty.js
@@ -0,0 +1 @@
+
diff --git a/tests/js_empty/many-semicolons.js b/tests/js_empty/many-semicolons.js
@@ -0,0 +1,2 @@
+;;
+;
diff --git a/tests_config/run_spec.js b/tests_config/run_spec.js
@@ -34,7 +34,6 @@ const unstableTests = new Map(
       "html_angular/attributes.component.html",
       (options) => options.printWidth === 1,
     ],
-    "js_empty/semicolon.js",
     "markdown_footnoteDefinition/multiline.md",
     "markdown_spec/example-234.md",
     "markdown_spec/example-235.md",