Skip to content

Commit 53bb73f

Browse files
committed
Address review suggestions
#831 (comment)
1 parent 8c3d385 commit 53bb73f

3 files changed

Lines changed: 7 additions & 4 deletions

File tree

ruff.toml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,9 @@ ignore = [
2222
force-single-line = true
2323
lines-after-imports = 2
2424

25+
[lint.flake8-pytest-style]
26+
parametrize-values-type = "tuple"
27+
2528
[lint.per-file-ignores]
2629
# Ignore in the `tests/` directory.
2730
"tests/**.py" = [

tests/pytest/_cli_test.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222

2323
@pytest.mark.parametrize(
2424
('raised_error', 'expected_stderr'),
25-
[
25+
(
2626
pytest.param(
2727
PreCommitTerraformRuntimeError('sentinel'),
2828
'App execution took an unexpected turn: sentinel. Exiting...',
@@ -38,7 +38,7 @@
3838
'User-initiated interrupt: sentinel. Exiting...',
3939
id='ctrl-c',
4040
),
41-
],
41+
),
4242
)
4343
def test_known_interrupts(
4444
capsys: pytest.CaptureFixture[str],

tests/pytest/terraform_docs_replace_test.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ def test_check_is_deprecated() -> None:
3434

3535
@pytest.mark.parametrize(
3636
('parsed_cli_args', 'expected_cmds'),
37-
[
37+
(
3838
pytest.param(Namespace(filenames=[]), [], id='no-files'),
3939
pytest.param(
4040
Namespace(
@@ -62,7 +62,7 @@ def test_check_is_deprecated() -> None:
6262
[],
6363
id='invalid-files',
6464
),
65-
],
65+
),
6666
)
6767
@pytest.mark.filterwarnings(
6868
'ignore:`terraform_docs_replace` hook is DEPRECATED.:UserWarning:'

0 commit comments

Comments
 (0)