Skip to content

Commit

Permalink
[flake8-pytest-style] Add a space after comma in CSV output (`PT006…
Browse files Browse the repository at this point in the history
…`) (#12853)

## Summary

See #12703. This only addresses the first bullet point, adding a space
after the comma in the suggested fix from list/tuple to string.

## Test Plan

Updated the snapshots and compared.
  • Loading branch information
uranusjr authored Aug 13, 2024
1 parent 7027344 commit 82a3e69
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -292,7 +292,7 @@ fn elts_to_csv(elts: &[Expr], generator: Generator) -> Option<String> {
.fold(String::new(), |mut acc, elt| {
if let Expr::StringLiteral(ast::ExprStringLiteral { value, .. }) = elt {
if !acc.is_empty() {
acc.push(',');
acc.push_str(", ");
}
acc.push_str(value.to_str());
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ PT006.py:24:26: PT006 [*] Wrong type passed to first argument of `@pytest.mark.p
22 22 |
23 23 |
24 |-@pytest.mark.parametrize(("param1", "param2"), [(1, 2), (3, 4)])
24 |+@pytest.mark.parametrize("param1,param2", [(1, 2), (3, 4)])
24 |+@pytest.mark.parametrize("param1, param2", [(1, 2), (3, 4)])
25 25 | def test_tuple(param1, param2):
26 26 | ...
27 27 |
Expand Down Expand Up @@ -53,7 +53,7 @@ PT006.py:34:26: PT006 [*] Wrong type passed to first argument of `@pytest.mark.p
32 32 |
33 33 |
34 |-@pytest.mark.parametrize(["param1", "param2"], [(1, 2), (3, 4)])
34 |+@pytest.mark.parametrize("param1,param2", [(1, 2), (3, 4)])
34 |+@pytest.mark.parametrize("param1, param2", [(1, 2), (3, 4)])
35 35 | def test_list(param1, param2):
36 36 | ...
37 37 |
Expand Down

0 comments on commit 82a3e69

Please sign in to comment.