check if the third test call is appropriate

biomejs · Nov 18, 2023 · 90b8c62 · 90b8c62
1 parent 0149819
commit 90b8c62
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 19 deletions.
diff --git a/crates/biome_js_formatter/src/utils/test_call.rs b/crates/biome_js_formatter/src/utils/test_call.rs
@@ -366,7 +366,7 @@ fn contains_a_test_pattern(callee: &AnyJsExpression) -> SyntaxResult<bool> {
             Some("only" | "skip" | "step") => third.is_none(),
             Some("describe") => match third {
                 None => true,
-                Some("only") => true,
+                Some("only") => fourth.is_none(),
                 Some("parallel" | "serial") => match fourth {
                     None => true,
                     Some("only") => fifth.is_none(),

diff --git a/...es/biome_js_formatter/tests/specs/prettier/js/test-declarations/test_declarations.js.snap b/...es/biome_js_formatter/tests/specs/prettier/js/test-declarations/test_declarations.js.snap
@@ -223,18 +223,6 @@ it(
 
  // Should break
 
-@@ -152,10 +142,7 @@
-   () => {},
- );
-
--test.describe.only.parallel(
--  "does something really long and complicated so I have to write a very long name for the test",
--  () => {},
--);
-+test.describe.only.parallel("does something really long and complicated so I have to write a very long name for the test", () => {});
-
- test.describe.parallel.serial(
-   "does something really long and complicated so I have to write a very long name for the testThis is a very",
 ```
 
 # Output
@@ -384,7 +372,10 @@ xskip(
   () => {},
 );
 
-test.describe.only.parallel("does something really long and complicated so I have to write a very long name for the test", () => {});
+test.describe.only.parallel(
+  "does something really long and complicated so I have to write a very long name for the test",
+  () => {},
+);
 
 test.describe.parallel.serial(
   "does something really long and complicated so I have to write a very long name for the testThis is a very",
@@ -470,11 +461,11 @@ it(
   127:   "does something really long and complicated so I have to write a very long name for the test",
   134:   "does something really long and complicated so I have to write a very long name for the test",
   141:   "does something really long and complicated so I have to write a very long name for the test",
-  145: test.describe.only.parallel("does something really long and complicated so I have to write a very long name for the test", () => {});
-  148:   "does something really long and complicated so I have to write a very long name for the testThis is a very",
-  153:   "does something really long and complicated so I have to write a very long name for the test",
-  158:   "does something really long and complicated so I have to write a very long name for the test",
-  167:   does something really long and complicated so I have to write a very long name for the test`, () => {
+  146:   "does something really long and complicated so I have to write a very long name for the test",
+  151:   "does something really long and complicated so I have to write a very long name for the testThis is a very",
+  156:   "does something really long and complicated so I have to write a very long name for the test",
+  161:   "does something really long and complicated so I have to write a very long name for the test",
+  170:   does something really long and complicated so I have to write a very long name for the test`, () => {
 ```