[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

authentication/middleware.py

@@ -30,7 +30,7 @@ def process_exception(self, request, exception):
             url = self.get_redirect_uri(request, exception)
 
             if url:  # noqa: RET503
-                url += ("?" in url and "&" or "?") + "message={}&backend={}".format(  # noqa: UP032
+                url += (("?" in url and "&") or "?") + "message={}&backend={}".format(  # noqa: UP032
                     quote(message), backend_name
                 )
                 return redirect(url)

b2b_ecommerce/api_test.py

@@ -31,7 +31,7 @@
 
 
 @pytest.fixture(autouse=True)
-def cybersource_settings(settings):  # noqa: PT004
+def cybersource_settings(settings):
     """
     Set cybersource settings
     """

b2b_ecommerce/views_test.py

@@ -33,7 +33,7 @@
 
 
 @pytest.fixture(autouse=True)
-def ecommerce_settings(settings):  # noqa: PT004
+def ecommerce_settings(settings):
     """
     Set cybersource settings
     """

conftest.py

@@ -65,7 +65,7 @@ def pytest_configure(config):
 
 
 @pytest.fixture(scope="session", autouse=True)
-def clean_up_files():  # noqa: PT004
+def clean_up_files():
     """
     Fixture that removes the media root folder after the suite has finished running,
     effectively deleting any files that were created by factories over the course of the test suite.
@@ -76,7 +76,7 @@ def clean_up_files():  # noqa: PT004
 
 
 @pytest.fixture(scope="session")
-def django_db_setup(django_db_setup, django_db_blocker):  # noqa: ARG001, PT004
+def django_db_setup(django_db_setup, django_db_blocker):  # noqa: ARG001
     """
     Creates all the index pages during the tests setup as index pages are required by the factories.
     """

ecommerce/api_test.py

@@ -117,7 +117,7 @@
 
 
 @pytest.fixture(autouse=True)
-def cybersource_settings(settings):  # noqa: PT004
+def cybersource_settings(settings):
     """
     Set cybersource settings
     """

ecommerce/views_test.py

@@ -92,7 +92,7 @@ def render_json(serializer):
 
 
 @pytest.fixture(autouse=True)
-def ecommerce_settings(settings):  # noqa: PT004
+def ecommerce_settings(settings):
     """
     Set cybersource settings
     """

fixtures/autouse.py

@@ -4,6 +4,6 @@
 
 
 @pytest.fixture(autouse=True)
-def disable_hubspot_api(settings):  # noqa: PT004
+def disable_hubspot_api(settings):
     """Disable Hubspot API by default for tests"""
     settings.MITOL_HUBSPOT_API_PRIVATE_TOKEN = None

fixtures/common.py

@@ -107,7 +107,7 @@ def valid_address_dict():
 
 
 @pytest.fixture
-def nplusone_fail(settings):  # noqa: PT004
+def nplusone_fail(settings):
     """Configures the nplusone app to raise errors"""
     settings.NPLUSONE_RAISE = True
 

mail/api_test.py

@@ -24,7 +24,7 @@
 
 
 @pytest.fixture
-def email_settings(settings):  # noqa: PT004
+def email_settings(settings):
     """Default settings for email tests"""
     settings.MAILGUN_RECIPIENT_OVERRIDE = None
 

sheets/conftest.py

@@ -4,7 +4,7 @@
 
 
 @pytest.fixture(autouse=True)
-def sheets_settings(settings):  # noqa: PT004
+def sheets_settings(settings):
     """Default settings for sheets tests"""
     settings.FEATURES["COUPON_SHEETS"] = True
     settings.SHEETS_REQ_EMAIL_COL = 7

sheets/coupon_request_api_test.py

@@ -19,7 +19,7 @@
 
 
 @pytest.fixture
-def courseware_objects():  # noqa: PT004
+def courseware_objects():
     """Database objects that CSV data depends on"""
     run = CourseRunFactory.create(courseware_id="course-v1:edX+DemoX+Demo_Course")
     ProductVersionFactory.create(product__content_object=run)