ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

I'm sure you're right, but can you expand on why that's a problem? (Should we not be raising a diagnostic at all?)

@charliermarsh Thanks for the comment!

Should we not be raising a diagnostic at all?

This is another option.

I just found #11243. Should we unpack each item in argvalues as a fix?

In this case, would this be correct?

@pytest.mark.parametrize("x,", [(1,), (2,)])

With the comma inside the string? I don't know enough about pytest.

@charliermarsh Let me check.

import pytest


@pytest.mark.parametrize("x,", [(1,), (2,)])
def test_foo(x):
    assert isinstance(x, int)

_____________________________________________________________________________________________________ test_foo[x0] ______________________________________________________________________________________________________

x = (1,)

    @pytest.mark.parametrize("x,", [(1,), (2,)])
    def test_foo(x):
>       assert isinstance(x, int)
E       assert False
E        +  where False = isinstance((1,), int)

a.py:6: AssertionError
_____________________________________________________________________________________________________ test_foo[x1] ______________________________________________________________________________________________________

x = (2,)

    @pytest.mark.parametrize("x,", [(1,), (2,)])
    def test_foo(x):
>       assert isinstance(x, int)
E       assert False
E        +  where False = isinstance((2,), int)

a.py:6: AssertionError

@pytest.mark.parametrize("x,", [(1,), (2,)]) is equivalent to @pytest.mark.parametrize("x", [(1,), (2,)]).

Source of the relevant pytest code

https://github.com/pytest-dev/pytest/blob/9d4f36d87dae9a968fb527e2cb87e8a507b0beb3/src/_pytest/mark/structures.py#L124-L135

According to the rule. How should a user rewrite your example?

@pytest.mark.parametrize(("x",), [(1,), (2,)])
                       # ^^^^^^
def test_foo(x):
    ...

@MichaReiser

@pytest.mark.parametrize(("x",), [(1,), (2,)])
def test_foo(x):
    ...

should be rewritten to:

@pytest.mark.parametrize("x", [1, 2])
def test_foo(x):
    ...

# I'd prefer this style because I don't need to wonder what x is (tuple vs. int)

Both argnames and items in argvalues need to be fixed. PT006 only fixes argnames now.

(I implemented a fix to fix argvalues as well but didn't push it yet).

(I implemented a fix to fix argvalues as well but didn't push it yet).

That sounds great. I'll have a look when you push the change