Repository navigation
Warn when passing scope as positional argument #812
Description
Recently I used pytest.mark.asyncio("session") instead of pytest.mark.asyncio(scope="session").
It took me quite a while to find out why the corresponding test was function scoped instead of session scoped.
It would be nice if either the scope could be passed as positional argument as well, or if there would be a helpful warning or error message in that case, instead of silently swallowing the positional argument.
I can contribute a PR for this feature if you let me know which exact behavior you prefer.
Activity
seifertm commentedon Jul 15, 2024
Good finding!
Since the purpose of a positional argument is not immediately obvious from the
pytest.mark.asyncioname, I think we should force a keyword argument here. I agree that issuing a warning when passing a positional argument is an improvement. If you can (still) find the time, I'd appreciate a PR for this.Cito commentedon Jul 18, 2024
Implemented in PR #886 now with tests for all such cases of mistakes in arguments.
chore(deps): update root dev dependencies (minor) (minor)
Merge branch 'renovate/root-dev-dependencies' into 'main'