mirror of
https://github.com/django/django.git
synced 2025-10-31 09:41:08 +00:00
190cd0e49f
Thanks strycore for the report and timgraham for suggesting the
solution.
Backport of 911d9f4ed1 from master
98 lines
4.0 KiB
Python
98 lines
4.0 KiB
Python
from django.conf import settings
|
|
from django.core.checks.urls import (
|
|
check_url_config, get_warning_for_invalid_pattern,
|
|
)
|
|
from django.test import SimpleTestCase
|
|
from django.test.utils import override_settings
|
|
from django.utils import six
|
|
|
|
|
|
class CheckUrlsTest(SimpleTestCase):
|
|
@override_settings(ROOT_URLCONF='check_framework.urls.no_warnings')
|
|
def test_no_warnings(self):
|
|
result = check_url_config(None)
|
|
self.assertEqual(result, [])
|
|
|
|
@override_settings(ROOT_URLCONF='check_framework.urls.warning_in_include')
|
|
def test_check_resolver_recursive(self):
|
|
# The resolver is checked recursively (examining url()s in include()).
|
|
result = check_url_config(None)
|
|
self.assertEqual(len(result), 1)
|
|
warning = result[0]
|
|
self.assertEqual(warning.id, 'urls.W001')
|
|
|
|
@override_settings(ROOT_URLCONF='check_framework.urls.include_with_dollar')
|
|
def test_include_with_dollar(self):
|
|
result = check_url_config(None)
|
|
self.assertEqual(len(result), 1)
|
|
warning = result[0]
|
|
self.assertEqual(warning.id, 'urls.W001')
|
|
expected_msg = "Your URL pattern '^include-with-dollar$' uses include with a regex ending with a '$'."
|
|
self.assertIn(expected_msg, warning.msg)
|
|
|
|
@override_settings(ROOT_URLCONF='check_framework.urls.contains_tuple')
|
|
def test_contains_tuple_not_url_instance(self):
|
|
result = check_url_config(None)
|
|
warning = result[0]
|
|
self.assertEqual(warning.id, 'urls.E004')
|
|
six.assertRegex(self, warning.msg, (
|
|
r"^Your URL pattern \('\^tuple/\$', <function <lambda> at 0x(\w+)>\) is "
|
|
r"invalid. Ensure that urlpatterns is a list of url\(\) instances.$"
|
|
))
|
|
|
|
@override_settings(ROOT_URLCONF='check_framework.urls.beginning_with_slash')
|
|
def test_beginning_with_slash(self):
|
|
result = check_url_config(None)
|
|
self.assertEqual(len(result), 1)
|
|
warning = result[0]
|
|
self.assertEqual(warning.id, 'urls.W002')
|
|
expected_msg = (
|
|
"Your URL pattern '/starting-with-slash/$' has a regex beginning "
|
|
"with a '/'. Remove this slash as it is unnecessary. If this "
|
|
"pattern is targeted in an include(), ensure the include() pattern "
|
|
"has a trailing '/'."
|
|
)
|
|
|
|
self.assertIn(expected_msg, warning.msg)
|
|
|
|
@override_settings(
|
|
ROOT_URLCONF='check_framework.urls.beginning_with_slash',
|
|
APPEND_SLASH=False,
|
|
)
|
|
def test_beginning_with_slash_append_slash(self):
|
|
# It can be useful to start a URL pattern with a slash when
|
|
# APPEND_SLASH=False (#27238).
|
|
result = check_url_config(None)
|
|
self.assertEqual(result, [])
|
|
|
|
@override_settings(ROOT_URLCONF='check_framework.urls.name_with_colon')
|
|
def test_name_with_colon(self):
|
|
result = check_url_config(None)
|
|
self.assertEqual(len(result), 1)
|
|
warning = result[0]
|
|
self.assertEqual(warning.id, 'urls.W003')
|
|
expected_msg = "Your URL pattern '^$' [name='name_with:colon'] has a name including a ':'."
|
|
self.assertIn(expected_msg, warning.msg)
|
|
|
|
@override_settings(ROOT_URLCONF=None)
|
|
def test_no_root_urlconf_in_settings(self):
|
|
delattr(settings, 'ROOT_URLCONF')
|
|
result = check_url_config(None)
|
|
self.assertEqual(result, [])
|
|
|
|
def test_get_warning_for_invalid_pattern_string(self):
|
|
warning = get_warning_for_invalid_pattern('')[0]
|
|
self.assertEqual(
|
|
warning.hint,
|
|
"Try removing the string ''. The list of urlpatterns should "
|
|
"not have a prefix string as the first element.",
|
|
)
|
|
|
|
def test_get_warning_for_invalid_pattern_tuple(self):
|
|
warning = get_warning_for_invalid_pattern((r'^$', lambda x: x))[0]
|
|
self.assertEqual(warning.hint, "Try using url() instead of a tuple.")
|
|
|
|
def test_get_warning_for_invalid_pattern_other(self):
|
|
warning = get_warning_for_invalid_pattern(object())[0]
|
|
self.assertIsNone(warning.hint)
|