Update Black Tests (#9455)

This commit is contained in:
Micha Reiser 2024-01-10 13:09:34 +01:00 committed by GitHub
parent ac02d3aedd
commit 58fcd96ac1
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
28 changed files with 602 additions and 56 deletions

View file

@ -11,6 +11,7 @@ class ClassWithSpaceParentheses:
class ClassWithEmptyFunc(object): class ClassWithEmptyFunc(object):
def func_with_blank_parentheses(): def func_with_blank_parentheses():
return 5 return 5

View file

@ -65,3 +65,25 @@ def something():
if flat if flat
else ValuesListIterable else ValuesListIterable
) )
def foo(wait: bool = True):
# This comment is two
# lines long
# This is only one
time.sleep(1) if wait else None
time.sleep(1) if wait else None
# With newline above
time.sleep(1) if wait else None
# Without newline above
time.sleep(1) if wait else None
a = "".join(
(
"", # comment
"" if True else "",
)
)

View file

@ -88,3 +88,23 @@ def something():
if named if named
else FlatValuesListIterable if flat else ValuesListIterable else FlatValuesListIterable if flat else ValuesListIterable
) )
def foo(wait: bool = True):
# This comment is two
# lines long
# This is only one
time.sleep(1) if wait else None
time.sleep(1) if wait else None
# With newline above
time.sleep(1) if wait else None
# Without newline above
time.sleep(1) if wait else None
a = "".join((
"", # comment
"" if True else "",
))

View file

@ -5,7 +5,7 @@ def foo2(parameter_1, parameter_2, parameter_3, parameter_4, parameter_5, parame
def foo3(parameter_1, parameter_2, parameter_3, parameter_4, parameter_5, parameter_6, parameter_7): pass def foo3(parameter_1, parameter_2, parameter_3, parameter_4, parameter_5, parameter_6, parameter_7): pass
def foo4(parameter_1, parameter_2, parameter_3, parameter_4, parameter_5, parameter_6, parameter_7): pass def foo4(parameter_1, parameter_2, parameter_3, parameter_4, parameter_5, parameter_6, parameter_7): pass
# Adding some unformatted code covering a wide range of syntaxes. # Adding some unformated code covering a wide range of syntaxes.
if True: if True:
# Incorrectly indented prefix comments. # Incorrectly indented prefix comments.

View file

@ -28,7 +28,7 @@ def foo3(
def foo4(parameter_1, parameter_2, parameter_3, parameter_4, parameter_5, parameter_6, parameter_7): pass def foo4(parameter_1, parameter_2, parameter_3, parameter_4, parameter_5, parameter_6, parameter_7): pass
# Adding some unformatted code covering a wide range of syntaxes. # Adding some unformated code covering a wide range of syntaxes.
if True: if True:
# Incorrectly indented prefix comments. # Incorrectly indented prefix comments.

View file

@ -1,5 +1,6 @@
"""I am a very helpful module docstring. """I am a very helpful module docstring.
With trailing spaces:
Lorem ipsum dolor sit amet, consectetur adipiscing elit, Lorem ipsum dolor sit amet, consectetur adipiscing elit,
sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Ut enim ad minim veniam, Ut enim ad minim veniam,

View file

@ -1,5 +1,6 @@
"""I am a very helpful module docstring. """I am a very helpful module docstring.
With trailing spaces:
Lorem ipsum dolor sit amet, consectetur adipiscing elit, Lorem ipsum dolor sit amet, consectetur adipiscing elit,
sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Ut enim ad minim veniam, Ut enim ad minim veniam,

View file

@ -16,3 +16,15 @@ if sys.version_info > (3, 7):
def function_definition(self): ... def function_definition(self): ...
assignment = 1 assignment = 1
def f2(self) -> str: ... def f2(self) -> str: ...
class TopLevel:
class Nested1:
foo: int
def bar(self): ...
field = 1
class Nested2:
def bar(self): ...
foo: int
field = 1

View file

@ -20,3 +20,16 @@ if sys.version_info > (3, 7):
assignment = 1 assignment = 1
def f2(self) -> str: ... def f2(self) -> str: ...
class TopLevel:
class Nested1:
foo: int
def bar(self): ...
field = 1
class Nested2:
def bar(self): ...
foo: int
field = 1

View file

@ -19,7 +19,7 @@ z: (Short
z: (int) = 2.3 z: (int) = 2.3
z: ((int)) = foo() z: ((int)) = foo()
# In case I go for not enforcing parentheses, this might get improved at the same time # In case I go for not enforcing parantheses, this might get improved at the same time
x = ( x = (
z z
== 9999999999999999999999999999999999999999 == 9999999999999999999999999999999999999999

View file

@ -28,7 +28,7 @@ z: Short | Short2 | Short3 | Short4 = 8
z: int = 2.3 z: int = 2.3
z: int = foo() z: int = foo()
# In case I go for not enforcing parentheses, this might get improved at the same time # In case I go for not enforcing parantheses, this might get improved at the same time
x = ( x = (
z z
== 9999999999999999999999999999999999999999 == 9999999999999999999999999999999999999999

View file

@ -60,3 +60,23 @@ class Cls:
def method(self): def method(self):
pass pass
async def async_fn():
"""Docstring."""
@decorated
async def async_fn():
"""Docstring."""
def top_level(
a: int,
b: str,
) -> Whatever[Generic, Something]:
def nested(x: int) -> int:
pass

View file

@ -57,6 +57,25 @@ def quux():
class Cls: class Cls:
def method(self): def method(self):
pass pass
async def async_fn():
"""Docstring."""
@decorated
async def async_fn():
"""Docstring."""
def top_level(
a: int,
b: str,
) -> Whatever[Generic, Something]:
def nested(x: int) -> int:
pass

View file

@ -56,3 +56,14 @@ if some_condition:
... ...
if already_dummy: ... if already_dummy: ...
class AsyncCls:
async def async_method(self):
...
async def async_function(self):
...
@decorated
async def async_function(self):
...

View file

@ -59,3 +59,14 @@ if some_condition:
if already_dummy: if already_dummy:
... ...
class AsyncCls:
async def async_method(self): ...
async def async_function(self): ...
@decorated
async def async_function(self): ...

View file

@ -79,6 +79,7 @@ def bar(a=1, b: bool = False):
class Baz: class Baz:
def __init__(self): def __init__(self):
pass pass

View file

@ -27,3 +27,7 @@ class MultilineDocstringsAsWell:
and on so many lines... and on so many lines...
""" """
class SingleQuotedDocstring:
"I'm a docstring but I don't even get triple quotes."

View file

@ -22,3 +22,7 @@ class MultilineDocstringsAsWell:
and on so many lines... and on so many lines...
""" """
class SingleQuotedDocstring:
"I'm a docstring but I don't even get triple quotes."

View file

@ -0,0 +1,119 @@
---
source: crates/ruff_python_formatter/tests/fixtures.rs
input_file: crates/ruff_python_formatter/resources/test/fixtures/black/cases/class_blank_parentheses.py
---
## Input
```python
class SimpleClassWithBlankParentheses():
pass
class ClassWithSpaceParentheses ( ):
first_test_data = 90
second_test_data = 100
def test_func(self):
return None
class ClassWithEmptyFunc(object):
def func_with_blank_parentheses():
return 5
def public_func_with_blank_parentheses():
return None
def class_under_the_func_with_blank_parentheses():
class InsideFunc():
pass
class NormalClass (
):
def func_for_testing(self, first, second):
sum = first + second
return sum
```
## Black Differences
```diff
--- Black
+++ Ruff
@@ -11,7 +11,6 @@
class ClassWithEmptyFunc(object):
-
def func_with_blank_parentheses():
return 5
```
## Ruff Output
```python
class SimpleClassWithBlankParentheses:
pass
class ClassWithSpaceParentheses:
first_test_data = 90
second_test_data = 100
def test_func(self):
return None
class ClassWithEmptyFunc(object):
def func_with_blank_parentheses():
return 5
def public_func_with_blank_parentheses():
return None
def class_under_the_func_with_blank_parentheses():
class InsideFunc:
pass
class NormalClass:
def func_for_testing(self, first, second):
sum = first + second
return sum
```
## Black Output
```python
class SimpleClassWithBlankParentheses:
pass
class ClassWithSpaceParentheses:
first_test_data = 90
second_test_data = 100
def test_func(self):
return None
class ClassWithEmptyFunc(object):
def func_with_blank_parentheses():
return 5
def public_func_with_blank_parentheses():
return None
def class_under_the_func_with_blank_parentheses():
class InsideFunc:
pass
class NormalClass:
def func_for_testing(self, first, second):
sum = first + second
return sum
```

View file

@ -72,6 +72,28 @@ def something():
if flat if flat
else ValuesListIterable else ValuesListIterable
) )
def foo(wait: bool = True):
# This comment is two
# lines long
# This is only one
time.sleep(1) if wait else None
time.sleep(1) if wait else None
# With newline above
time.sleep(1) if wait else None
# Without newline above
time.sleep(1) if wait else None
a = "".join(
(
"", # comment
"" if True else "",
)
)
``` ```
## Black Differences ## Black Differences
@ -136,7 +158,7 @@ def something():
for some_boolean_variable in some_iterable for some_boolean_variable in some_iterable
) )
@@ -86,5 +78,7 @@ @@ -86,7 +78,9 @@
clone._iterable_class = ( clone._iterable_class = (
NamedValuesListIterable NamedValuesListIterable
if named if named
@ -145,6 +167,8 @@ def something():
+ if flat + if flat
+ else ValuesListIterable + else ValuesListIterable
) )
``` ```
## Ruff Output ## Ruff Output
@ -234,6 +258,26 @@ def something():
if flat if flat
else ValuesListIterable else ValuesListIterable
) )
def foo(wait: bool = True):
# This comment is two
# lines long
# This is only one
time.sleep(1) if wait else None
time.sleep(1) if wait else None
# With newline above
time.sleep(1) if wait else None
# Without newline above
time.sleep(1) if wait else None
a = "".join((
"", # comment
"" if True else "",
))
``` ```
## Black Output ## Black Output
@ -329,6 +373,26 @@ def something():
if named if named
else FlatValuesListIterable if flat else ValuesListIterable else FlatValuesListIterable if flat else ValuesListIterable
) )
def foo(wait: bool = True):
# This comment is two
# lines long
# This is only one
time.sleep(1) if wait else None
time.sleep(1) if wait else None
# With newline above
time.sleep(1) if wait else None
# Without newline above
time.sleep(1) if wait else None
a = "".join((
"", # comment
"" if True else "",
))
``` ```

View file

@ -12,7 +12,7 @@ def foo2(parameter_1, parameter_2, parameter_3, parameter_4, parameter_5, parame
def foo3(parameter_1, parameter_2, parameter_3, parameter_4, parameter_5, parameter_6, parameter_7): pass def foo3(parameter_1, parameter_2, parameter_3, parameter_4, parameter_5, parameter_6, parameter_7): pass
def foo4(parameter_1, parameter_2, parameter_3, parameter_4, parameter_5, parameter_6, parameter_7): pass def foo4(parameter_1, parameter_2, parameter_3, parameter_4, parameter_5, parameter_6, parameter_7): pass
# Adding some unformatted code covering a wide range of syntaxes. # Adding some unformated code covering a wide range of syntaxes.
if True: if True:
# Incorrectly indented prefix comments. # Incorrectly indented prefix comments.
@ -89,7 +89,7 @@ async def test_async_unformatted( ): # Trailing comment with extra leading
+ pass + pass
+ +
# Adding some unformatted code covering a wide range of syntaxes. # Adding some unformated code covering a wide range of syntaxes.
if True: if True:
- # Incorrectly indented prefix comments. - # Incorrectly indented prefix comments.
@ -207,7 +207,7 @@ def foo4(
pass pass
# Adding some unformatted code covering a wide range of syntaxes. # Adding some unformated code covering a wide range of syntaxes.
if True: if True:
# Incorrectly indented prefix comments. # Incorrectly indented prefix comments.
@ -279,7 +279,7 @@ def foo3(
def foo4(parameter_1, parameter_2, parameter_3, parameter_4, parameter_5, parameter_6, parameter_7): pass def foo4(parameter_1, parameter_2, parameter_3, parameter_4, parameter_5, parameter_6, parameter_7): pass
# Adding some unformatted code covering a wide range of syntaxes. # Adding some unformated code covering a wide range of syntaxes.
if True: if True:
# Incorrectly indented prefix comments. # Incorrectly indented prefix comments.

View file

@ -0,0 +1,133 @@
---
source: crates/ruff_python_formatter/tests/fixtures.rs
input_file: crates/ruff_python_formatter/resources/test/fixtures/black/cases/module_docstring_2.py
---
## Input
```python
"""I am a very helpful module docstring.
With trailing spaces:
Lorem ipsum dolor sit amet, consectetur adipiscing elit,
sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Ut enim ad minim veniam,
quis nostrud exercitation ullamco laboris
nisi ut aliquip ex ea commodo consequat.
Duis aute irure dolor in reprehenderit in voluptate
velit esse cillum dolore eu fugiat nulla pariatur.
Excepteur sint occaecat cupidatat non proident,
sunt in culpa qui officia deserunt mollit anim id est laborum.
"""
a = 1
"""Look at me I'm a docstring...
............................................................
............................................................
............................................................
............................................................
............................................................
............................................................
............................................................
........................................................NOT!
"""
b = 2
```
## Black Differences
```diff
--- Black
+++ Ruff
@@ -1,6 +1,6 @@
"""I am a very helpful module docstring.
-With trailing spaces:
+With trailing spaces:
Lorem ipsum dolor sit amet, consectetur adipiscing elit,
sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Ut enim ad minim veniam,
```
## Ruff Output
```python
"""I am a very helpful module docstring.
With trailing spaces:
Lorem ipsum dolor sit amet, consectetur adipiscing elit,
sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Ut enim ad minim veniam,
quis nostrud exercitation ullamco laboris
nisi ut aliquip ex ea commodo consequat.
Duis aute irure dolor in reprehenderit in voluptate
velit esse cillum dolore eu fugiat nulla pariatur.
Excepteur sint occaecat cupidatat non proident,
sunt in culpa qui officia deserunt mollit anim id est laborum.
"""
a = 1
"""Look at me I'm a docstring...
............................................................
............................................................
............................................................
............................................................
............................................................
............................................................
............................................................
........................................................NOT!
"""
b = 2
```
## Black Output
```python
"""I am a very helpful module docstring.
With trailing spaces:
Lorem ipsum dolor sit amet, consectetur adipiscing elit,
sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Ut enim ad minim veniam,
quis nostrud exercitation ullamco laboris
nisi ut aliquip ex ea commodo consequat.
Duis aute irure dolor in reprehenderit in voluptate
velit esse cillum dolore eu fugiat nulla pariatur.
Excepteur sint occaecat cupidatat non proident,
sunt in culpa qui officia deserunt mollit anim id est laborum.
"""
a = 1
"""Look at me I'm a docstring...
............................................................
............................................................
............................................................
............................................................
............................................................
............................................................
............................................................
........................................................NOT!
"""
b = 2
```

View file

@ -23,6 +23,18 @@ if sys.version_info > (3, 7):
def function_definition(self): ... def function_definition(self): ...
assignment = 1 assignment = 1
def f2(self) -> str: ... def f2(self) -> str: ...
class TopLevel:
class Nested1:
foo: int
def bar(self): ...
field = 1
class Nested2:
def bar(self): ...
foo: int
field = 1
``` ```
## Black Differences ## Black Differences
@ -40,7 +52,7 @@ if sys.version_info > (3, 7):
outer_attr_after_inner_stub: int outer_attr_after_inner_stub: int
class Inner: class Inner:
@@ -9,14 +11,19 @@ @@ -9,27 +11,35 @@
outer_attr: int outer_attr: int
@ -60,6 +72,22 @@ if sys.version_info > (3, 7):
assignment = 1 assignment = 1
def f2(self) -> str: ... def f2(self) -> str: ...
+
class TopLevel:
class Nested1:
foo: int
+
def bar(self): ...
field = 1
class Nested2:
def bar(self): ...
+
foo: int
field = 1
``` ```
## Ruff Output ## Ruff Output
@ -94,6 +122,22 @@ if sys.version_info > (3, 7):
assignment = 1 assignment = 1
def f2(self) -> str: ... def f2(self) -> str: ...
class TopLevel:
class Nested1:
foo: int
def bar(self): ...
field = 1
class Nested2:
def bar(self): ...
foo: int
field = 1
``` ```
## Black Output ## Black Output
@ -121,6 +165,19 @@ if sys.version_info > (3, 7):
assignment = 1 assignment = 1
def f2(self) -> str: ... def f2(self) -> str: ...
class TopLevel:
class Nested1:
foo: int
def bar(self): ...
field = 1
class Nested2:
def bar(self): ...
foo: int
field = 1
``` ```

View file

@ -26,7 +26,7 @@ z: (Short
z: (int) = 2.3 z: (int) = 2.3
z: ((int)) = foo() z: ((int)) = foo()
# In case I go for not enforcing parentheses, this might get improved at the same time # In case I go for not enforcing parantheses, this might get improved at the same time
x = ( x = (
z z
== 9999999999999999999999999999999999999999 == 9999999999999999999999999999999999999999
@ -165,7 +165,7 @@ z: Short | Short2 | Short3 | Short4 = 8
z: int = 2.3 z: int = 2.3
z: int = foo() z: int = foo()
# In case I go for not enforcing parentheses, this might get improved at the same time # In case I go for not enforcing parantheses, this might get improved at the same time
x = ( x = (
z z
== 9999999999999999999999999999999999999999 == 9999999999999999999999999999999999999999
@ -269,7 +269,7 @@ z: Short | Short2 | Short3 | Short4 = 8
z: int = 2.3 z: int = 2.3
z: int = foo() z: int = foo()
# In case I go for not enforcing parentheses, this might get improved at the same time # In case I go for not enforcing parantheses, this might get improved at the same time
x = ( x = (
z z
== 9999999999999999999999999999999999999999 == 9999999999999999999999999999999999999999

View file

@ -67,6 +67,26 @@ class Cls:
def method(self): def method(self):
pass pass
async def async_fn():
"""Docstring."""
@decorated
async def async_fn():
"""Docstring."""
def top_level(
a: int,
b: str,
) -> Whatever[Generic, Something]:
def nested(x: int) -> int:
pass
``` ```
## Black Differences ## Black Differences
@ -82,7 +102,7 @@ class Cls:
# This is also now fine # This is also now fine
a = 123 a = 123
@@ -14,49 +13,39 @@ @@ -14,52 +13,41 @@
a = 123 a = 123
if y: if y:
@ -129,9 +149,19 @@ class Cls:
class Cls: class Cls:
-
def method(self): def method(self):
- -
pass pass
@@ -76,6 +64,5 @@
a: int,
b: str,
) -> Whatever[Generic, Something]:
-
def nested(x: int) -> int:
pass
``` ```
## Ruff Output ## Ruff Output
@ -188,6 +218,23 @@ def quux():
class Cls: class Cls:
def method(self): def method(self):
pass pass
async def async_fn():
"""Docstring."""
@decorated
async def async_fn():
"""Docstring."""
def top_level(
a: int,
b: str,
) -> Whatever[Generic, Something]:
def nested(x: int) -> int:
pass
``` ```
## Black Output ## Black Output
@ -252,9 +299,28 @@ def quux():
class Cls: class Cls:
def method(self): def method(self):
pass pass
async def async_fn():
"""Docstring."""
@decorated
async def async_fn():
"""Docstring."""
def top_level(
a: int,
b: str,
) -> Whatever[Generic, Something]:
def nested(x: int) -> int:
pass
``` ```

View file

@ -211,7 +211,7 @@ a = [
pass pass
@@ -68,13 +68,12 @@ @@ -68,25 +68,23 @@
def foo(): def foo():
pass pass
@ -226,7 +226,11 @@ a = [
pass pass
@@ -85,7 +84,7 @@ class Baz:
-
def __init__(self):
pass
def something(self): def something(self):
pass pass
@ -427,6 +431,7 @@ def bar(a=1, b: bool = False):
class Baz: class Baz:
def __init__(self): def __init__(self):
pass pass

View file

@ -1,38 +0,0 @@
---
source: crates/ruff_python_formatter/tests/fixtures.rs
input_file: crates/ruff_python_formatter/resources/test/fixtures/black/cases/preview_pep_572.py
---
## Input
```python
x[(a:=0):]
x[:(a:=0)]
```
## Black Differences
```diff
--- Black
+++ Ruff
@@ -1,2 +1,2 @@
-x[(a := 0):]
-x[:(a := 0)]
+x[(a := 0) :]
+x[: (a := 0)]
```
## Ruff Output
```python
x[(a := 0) :]
x[: (a := 0)]
```
## Black Output
```python
x[(a := 0):]
x[:(a := 0)]
```