Skip to content

Commit 4d18ae7

Browse files
Copilotlucemia
andcommitted
Fix linting issues and add DOC501 exception for test files
Co-authored-by: lucemia <432851+lucemia@users.noreply.github.com>
1 parent 11129a1 commit 4d18ae7

File tree

4 files changed

+14
-20
lines changed

4 files changed

+14
-20
lines changed

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,7 @@ ignore = [
168168
]
169169

170170
[tool.ruff.lint.per-file-ignores]
171-
"**/tests/**.py" = ["D103", "D104", "D100"]
171+
"**/tests/**.py" = ["D103", "D104", "D100", "DOC501"]
172172

173173
[tool.ruff.format]
174174
quote-style = "double"

src/ffmpeg/common/tests/test_schema.py

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,8 @@ def test_filter_to_def_property() -> None:
3030
filter_def = FFMpegFilter(
3131
name="scale",
3232
description="Scale video",
33-
stream_typings_input=(
34-
FFMpegIOType(name="default", type=StreamType.video),
35-
),
36-
stream_typings_output=(
37-
FFMpegIOType(name="default", type=StreamType.video),
38-
),
33+
stream_typings_input=(FFMpegIOType(name="default", type=StreamType.video),),
34+
stream_typings_output=(FFMpegIOType(name="default", type=StreamType.video),),
3935
)
4036

4137
simple_def = filter_def.to_def
@@ -65,9 +61,7 @@ def test_filter_input_typings_static() -> None:
6561
filter_def = FFMpegFilter(
6662
name="scale",
6763
description="Scale video",
68-
stream_typings_input=(
69-
FFMpegIOType(name="default", type=StreamType.video),
70-
),
64+
stream_typings_input=(FFMpegIOType(name="default", type=StreamType.video),),
7165
stream_typings_output=(),
7266
)
7367

@@ -130,9 +124,7 @@ def test_filter_output_typings_static() -> None:
130124
name="scale",
131125
description="Scale video",
132126
stream_typings_input=(),
133-
stream_typings_output=(
134-
FFMpegIOType(name="default", type=StreamType.video),
135-
),
127+
stream_typings_output=(FFMpegIOType(name="default", type=StreamType.video),),
136128
)
137129

138130
typings = filter_def.output_typings

src/ffmpeg/tests/test_schema.py

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,12 @@ def test_ffmpeg_option_group_basic() -> None:
2727

2828
def test_ffmpeg_option_group_as_av_options() -> None:
2929
"""Test as_av_options converts boolean to 0/1."""
30-
options = FFMpegOptionGroup(
31-
{"flag1": True, "flag2": False, "string": "value", "number": 42}
32-
)
30+
options = FFMpegOptionGroup({
31+
"flag1": True,
32+
"flag2": False,
33+
"string": "value",
34+
"number": 42,
35+
})
3336

3437
av_options = options.as_av_options()
3538

@@ -41,9 +44,7 @@ def test_ffmpeg_option_group_as_av_options() -> None:
4144

4245
def test_ffmpeg_option_group_as_av_options_none() -> None:
4346
"""Test as_av_options filters out None values."""
44-
options = FFMpegOptionGroup(
45-
{"keep": "value", "remove": None, "also_keep": 123}
46-
)
47+
options = FFMpegOptionGroup({"keep": "value", "remove": None, "also_keep": 123})
4748

4849
av_options = options.as_av_options()
4950

src/ffmpeg/utils/tests/test_view.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,10 @@
33
import pytest
44

55
import ffmpeg
6-
from ..view import _get_node_color, view
6+
77
from ...dag.nodes import InputNode
88
from ...utils.frozendict import FrozenDict
9+
from ..view import _get_node_color, view
910

1011

1112
def test_get_node_color_input() -> None:

0 commit comments

Comments
 (0)