diff --git a/src/postorius/tests/mailman_api_tests/test_list_members.py b/src/postorius/tests/mailman_api_tests/test_list_members.py index c5dcb29..054071c 100644 --- a/src/postorius/tests/mailman_api_tests/test_list_members.py +++ b/src/postorius/tests/mailman_api_tests/test_list_members.py @@ -74,9 +74,12 @@ def test_page_not_accessible_if_not_logged_in(self): url = reverse('list_members', args=('foo@example.com', )) response = self.client.get(url) - expected_redirect = "%s?next=%s" % (settings.LOGIN_URL, quote(url)) - self.assertRedirects(response, expected_redirect, - fetch_redirect_response=False) + if "%40" not in url: # Django < 1.8 + url = quote(url) + expected_redirect = "http://testserver%s?next=%s" % ( + settings.LOGIN_URL, url) + self.assertEqual(response.status_code, 302) + self.assertEqual(response["location"], expected_redirect) @MM_VCR.use_cassette('list_members_access.yaml') def test_page_not_accessible_for_unprivileged_users(self): diff --git a/src/postorius/tests/mailman_api_tests/test_list_metrics.py b/src/postorius/tests/mailman_api_tests/test_list_metrics.py index 53f70a8..4c12968 100644 --- a/src/postorius/tests/mailman_api_tests/test_list_metrics.py +++ b/src/postorius/tests/mailman_api_tests/test_list_metrics.py @@ -45,9 +45,12 @@ def test_metrics_page_not_accessible_to_anonymous(self): url = reverse('list_metrics', args=['test@example.org']) response = self.client.get(url) - expected_redirect = "%s?next=%s" % (settings.LOGIN_URL, quote(url)) - self.assertRedirects(response, expected_redirect, - fetch_redirect_response=False) + if "%40" not in url: # Django < 1.8 + url = quote(url) + expected_redirect = "http://testserver%s?next=%s" % ( + settings.LOGIN_URL, url) + self.assertEqual(response.status_code, 302) + self.assertEqual(response["location"], expected_redirect) @MM_VCR.use_cassette('test_list_metrics.yaml') def test_metrics_page_contains_metrics(self):