Commit 4e91a6c
committed
Tweak formatting for @pytest.mark.parametrize
This removes the "fmt: off" / "fmt: on" directives around the
`@pytest.mark.parametrize` decoration on test_blob_filter, and
reformats it with black, for consistency with other such
decorations.
The style used there, *if* it could be specified as a rule and thus
used without "fmt:" directives, may be nicer than how black formats
multi-line mark decorations. However, since that decoration was
written, there have been a number of other such decorations, which
have been in black style.
This also removes the only (or only remaining?) "fmt:" directive in
the codebase. As such, it should possibly have been done in #1760.1 parent b438c45 commit 4e91a6c
1 file changed
Lines changed: 9 additions & 8 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -14,14 +14,15 @@ | |||
| 14 | 14 | from git.types import PathLike | |
| 15 | 15 | ||
| 16 | 16 | ||
| 17 | - # fmt: off | ||
| 18 | - @pytest.mark.parametrize('paths, path, expected_result', [ | ||
| 19 | - ((Path("foo"),), Path("foo"), True), | ||
| 20 | - ((Path("foo"),), Path("foo/bar"), True), | ||
| 21 | - ((Path("foo/bar"),), Path("foo"), False), | ||
| 22 | - ((Path("foo"), Path("bar")), Path("foo"), True), | ||
| 23 | - ]) | ||
| 24 | - # fmt: on | ||
| 17 | + @pytest.mark.parametrize( | ||
| 18 | + "paths, path, expected_result", | ||
| 19 | + [ | ||
| 20 | + ((Path("foo"),), Path("foo"), True), | ||
| 21 | + ((Path("foo"),), Path("foo/bar"), True), | ||
| 22 | + ((Path("foo/bar"),), Path("foo"), False), | ||
| 23 | + ((Path("foo"), Path("bar")), Path("foo"), True), | ||
| 24 | + ], | ||
| 25 | + ) | ||
| 25 | 26 | def test_blob_filter(paths: Sequence[PathLike], path: PathLike, expected_result: bool) -> None: | |
| 26 | 27 | """Test the blob filter.""" | |
| 27 | 28 | blob_filter = BlobFilter(paths) | |
0 commit comments