Closed ninoseki closed 1 year ago
This is a bit opinionated PR that removes pylint: disable=unused-argument and use @pytest.mark.usefixtures.
pylint: disable=unused-argument
@pytest.mark.usefixtures
Pytest provides usefixtures marker to use a fixture. It enables to use a fixture without having it as an argument. pylint: disable=unused-argument is not necessary if you use it. (See https://docs.pytest.org/en/7.1.x/how-to/fixtures.html#use-fixtures-in-classes-and-modules-with-usefixtures for details) I think it's cleaner than inline-disabling the lint rule.
usefixtures
This is a bit opinionated PR that removes
pylint: disable=unused-argumentand use@pytest.mark.usefixtures.Pytest provides
usefixturesmarker to use a fixture. It enables to use a fixture without having it as an argument.pylint: disable=unused-argumentis not necessary if you use it. (See https://docs.pytest.org/en/7.1.x/how-to/fixtures.html#use-fixtures-in-classes-and-modules-with-usefixtures for details) I think it's cleaner than inline-disabling the lint rule.