From b43e42ac7a6dcd02b31696fc58ac5604b29a38b9 Mon Sep 17 00:00:00 2001 From: James Valleroy Date: Mon, 20 Jun 2022 08:13:21 -0400 Subject: [PATCH] tests: Add a dummy parameter for middlewares MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit From the Django 4.0 release notes: The get_response argument for django.utils.deprecation.MiddlewareMixin.__init__() is required and doesn’t accept None. It appears that any non-None value can be used here, so I pass in `True` when initializing middlewares for tests. I don't know if this was the intended value, but it does fix the tests. Signed-off-by: James Valleroy Reviewed-by: Sunil Mohan Adapa --- plinth/tests/test_middleware.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/plinth/tests/test_middleware.py b/plinth/tests/test_middleware.py index 26f9dfb6b..1f1463e6d 100644 --- a/plinth/tests/test_middleware.py +++ b/plinth/tests/test_middleware.py @@ -55,7 +55,7 @@ class TestSetupMiddleware: @pytest.fixture(name='middleware') def fixture_middleware(load_cfg): """Fixture for returning middleware.""" - return SetupMiddleware() + return SetupMiddleware(True) @staticmethod @patch('django.urls.reverse', return_value='users:login') @@ -187,7 +187,7 @@ class TestAdminMiddleware: @pytest.fixture(name='middleware') def fixture_middleware(load_cfg): """Fixture for returning middleware.""" - return AdminRequiredMiddleware() + return AdminRequiredMiddleware(True) @staticmethod @pytest.fixture(name='web_request')