test(rome_js_formatter): Add prettier JSX tests

This PR extracts the JSX tests from prettier and updates existing tests.

There are a few instability issues, most related to comments. I'll look into those as part of the comments refactoring

crates/rome_js_formatter/tests/prettier_tests.rs

@@ -31,7 +31,7 @@ struct TestInfo {
 
 mod check_reformat;
 
-tests_macros::gen_tests! {"tests/specs/prettier/{js,typescript}/**/*.{js,ts,jsx,tsx}", crate::test_snapshot, "script"}
+tests_macros::gen_tests! {"tests/specs/prettier/{js,typescript,jsx}/**/*.{js,ts,jsx,tsx}", crate::test_snapshot, "script"}
 
 const PRETTIER_IGNORE: &str = "prettier-ignore";
 const ROME_IGNORE: &str = "rome-ignore format: prettier ignore";

crates/rome_js_formatter/tests/specs/prettier/js/export/blank-line-between-specifiers.js.prettier-snap

@@ -1,7 +1,6 @@
 export {
   // a
   foo1,
-
   // b
   bar1,
   baz1,
@@ -14,7 +13,6 @@ const baz2 = 1;
 export {
   // a
   foo2,
-
   // b
   bar2,
   baz2,

crates/rome_js_formatter/tests/specs/prettier/js/if/trailing_comment.js

@@ -10,10 +10,3 @@ if (
 ) {
   a;
 }
-
-if (
-	// comment
-	1
-) {
-	b;
-}

crates/rome_js_formatter/tests/specs/prettier/js/if/trailing_comment.js.snap

@@ -1,5 +1,7 @@
 ---
 source: crates/rome_js_formatter/tests/prettier_tests.rs
+info:
+  test_file: js/if/trailing_comment.js
 ---
 
 # Input
@@ -17,13 +19,6 @@ if (
 ) {
   a;
 }
-
-if (
-	// comment
-	1
-) {
-	b;
-}
 ```
 
 
@@ -42,17 +37,6 @@ if (
  }
  if (code === 92) {
    /* '\' */
-@@ -16,3 +15,10 @@
- ) {
-   a;
- }
-+
-+if (
-+  // comment
-+  1
-+) {
-+  b;
-+}
 ```
 
 # Output
@@ -75,13 +59,6 @@ if (
 ) {
   a;
 }
-
-if (
-  // comment
-  1
-) {
-  b;
-}
 ```
 
 

crates/rome_js_formatter/tests/specs/prettier/js/import-assertions/re-export.js

@@ -1,3 +1,3 @@
-export { foo2 } from "foo.json" assert { type: "json" };
+export { default as foo2 } from "foo.json" assert { type: "json" };
 export * from "foo.json" assert { type: "json" };
 export * as foo3 from "foo.json" assert { type: "json" };

crates/rome_js_formatter/tests/specs/prettier/js/import-assertions/re-export.js.prettier-snap

@@ -1,3 +1,3 @@
-export { foo2 } from "foo.json" assert { type: "json" };
+export { default as foo2 } from "foo.json" assert { type: "json" };
 export * from "foo.json" assert { type: "json" };
 export * as foo3 from "foo.json" assert { type: "json" };

crates/rome_js_formatter/tests/specs/prettier/js/record/spread.js

@@ -4,4 +4,3 @@ const taskLater = #{ ...taskNow, status: "DONE" }
 
 // A reminder: The ordering of keys in record literals does not affect equality (and is not retained)
 assert(taskLater === #{ status: "DONE", title: formData.title, id: 42 })
-

crates/rome_js_formatter/tests/specs/prettier/js/record/spread.js.snap

@@ -1,5 +1,7 @@
 ---
 source: crates/rome_js_formatter/tests/prettier_tests.rs
+info:
+  test_file: js/record/spread.js
 ---
 
 # Input
@@ -11,7 +13,6 @@ const taskLater = #{ ...taskNow, status: "DONE" }
 
 // A reminder: The ordering of keys in record literals does not affect equality (and is not retained)
 assert(taskLater === #{ status: "DONE", title: formData.title, id: 42 })
-
 ```
 
 

crates/rome_js_formatter/tests/specs/prettier/js/test-declarations/test_declarations.js.prettier-snap

@@ -85,43 +85,75 @@ skip(`does something really long and complicated so I have to write a very long
 
 skip("does something really long and complicated so I have to write a very long name for the test", () => {});
 
-test.step("does something really long and complicated so I have to write a very long name for the test", () => {});
+test.step(
+  "does something really long and complicated so I have to write a very long name for the test",
+  () => {},
+);
 
-test.step(`does something really long and complicated so I have to write a very long name for the test`, () => {});
+test.step(
+  `does something really long and complicated so I have to write a very long name for the test`,
+  () => {},
+);
 
-test.describe("does something really long and complicated so I have to write a very long name for the test", () => {});
+test.describe(
+  "does something really long and complicated so I have to write a very long name for the test",
+  () => {},
+);
 
-test.describe(`does something really long and complicated so I have to write a very long name for the test`, () => {});
+test.describe(
+  `does something really long and complicated so I have to write a very long name for the test`,
+  () => {},
+);
 
-test.describe
-  .only("does something really long and complicated so I have to write a very long name for the test", () => {});
+test.describe.only(
+  "does something really long and complicated so I have to write a very long name for the test",
+  () => {},
+);
 
-test.describe
-  .only(`does something really long and complicated so I have to write a very long name for the test`, () => {});
+test.describe.only(
+  `does something really long and complicated so I have to write a very long name for the test`,
+  () => {},
+);
 
-test.describe
-  .parallel("does something really long and complicated so I have to write a very long name for the test", () => {});
+test.describe.parallel(
+  "does something really long and complicated so I have to write a very long name for the test",
+  () => {},
+);
 
-test.describe
-  .parallel(`does something really long and complicated so I have to write a very long name for the test`, () => {});
+test.describe.parallel(
+  `does something really long and complicated so I have to write a very long name for the test`,
+  () => {},
+);
 
-test.describe.parallel
-  .only("does something really long and complicated so I have to write a very long name for the testThis is a very", () => {});
+test.describe.parallel.only(
+  "does something really long and complicated so I have to write a very long name for the testThis is a very",
+  () => {},
+);
 
-test.describe.parallel
-  .only(`does something really long and complicated so I have to write a very long name for the testThis is a very`, () => {});
+test.describe.parallel.only(
+  `does something really long and complicated so I have to write a very long name for the testThis is a very`,
+  () => {},
+);
 
-test.describe
-  .serial("does something really long and complicated so I have to write a very long name for the test", () => {});
+test.describe.serial(
+  "does something really long and complicated so I have to write a very long name for the test",
+  () => {},
+);
 
-test.describe
-  .serial(`does something really long and complicated so I have to write a very long name for the test`, () => {});
+test.describe.serial(
+  `does something really long and complicated so I have to write a very long name for the test`,
+  () => {},
+);
 
-test.describe.serial
-  .only("does something really long and complicated so I have to write a very long name for the test", () => {});
+test.describe.serial.only(
+  "does something really long and complicated so I have to write a very long name for the test",
+  () => {},
+);
 
-test.describe.serial
-  .only(`does something really long and complicated so I have to write a very long name for the test`, () => {});
+test.describe.serial.only(
+  `does something really long and complicated so I have to write a very long name for the test`,
+  () => {},
+);
 
 // Should break