[flake8-pytest-style] Avoid false positive for legacy form of pytest.raises (PT011)

[twentyone212121]

This fix closes #17026

Summary

The check for the PytestRaisesTooBroad rule is now skipped if there is a second positional argument present, which means pytest.raises is used as a function.

Test Plan

Tested on the example from the issue, which now passes the check.

pytest.raises(Exception, func, *func_args, **func_kwargs).match("error message")

[MichaReiser]

Thanks. Could you add a test example to https://github.com/astral-sh/ruff/blob/2b6d66b793adc9e0cc50817b59187b41f39f7691/crates/ruff_linter/resources/test/fixtures/flake8_pytest_style/PT011.py#L1-L0 with your example from the test plan

[twentyone212121]

I added an ok test example. The diff in snapshots is big because error tests are moved by 7 lines

[github-actions]

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

[Copilot]

Copilot reviewed 2 out of 6 changed files in this pull request and generated no comments.

Files not reviewed (4)

• crates/ruff_linter/src/rules/flake8_pytest_style/snapshots/ruff_linter__rules__flake8_pytest_style__tests__PT011_default.snap: Language not supported
• crates/ruff_linter/src/rules/flake8_pytest_style/snapshots/ruff_linter__rules__flake8_pytest_style__tests__PT011_extend_broad_exceptions.snap: Language not supported
• crates/ruff_linter/src/rules/flake8_pytest_style/snapshots/ruff_linter__rules__flake8_pytest_style__tests__PT011_glob_all.snap: Language not supported
• crates/ruff_linter/src/rules/flake8_pytest_style/snapshots/ruff_linter__rules__flake8_pytest_style__tests__PT011_glob_prefix.snap: Language not supported

Comments suppressed due to low confidence (1)

crates/ruff_linter/src/rules/flake8_pytest_style/rules/raises.rs:192

• [nitpick] Using the string literal "func" with a hard-coded index (1) may reduce readability. Consider defining a constant for this argument name and index to clarify its purpose when detecting the second overload.

if call.arguments.find_argument("func", 1).is_none() {