{% block title %}Google Maps via GeoDjango{% endblock %}

Title:
Pub date:
Delete:
(None)	%s
Url:
Comment:

') def test_callable_lookup(self): """Admin inline should invoke local callable when its name is listed in readonly_fields""" - response = self.client.get('/admin/admin_inlines/poll/add/') + response = self.client.get(reverse('admin:admin_inlines_poll_add')) self.assertEqual(response.status_code, 200) # Add parent object view should have the child inlines section self.assertContains(response, '

') @@ -123,11 +130,11 @@ def test_help_text(self): using both the stacked and tabular layouts. Ref #8190. """ - response = self.client.get('/admin/admin_inlines/holder4/add/') + response = self.client.get(reverse('admin:admin_inlines_holder4_add')) self.assertContains(response, '

Awesome stacked help text is awesome.

', 4) self.assertContains(response, '', 1) # ReadOnly fields - response = self.client.get('/admin/admin_inlines/capofamiglia/add/') + response = self.client.get(reverse('admin:admin_inlines_capofamiglia_add')) self.assertContains(response, '', 1) def test_inline_hidden_field_no_column(self): @@ -135,7 +142,7 @@ def test_inline_hidden_field_no_column(self): parent = SomeParentModel.objects.create(name='a') SomeChildModel.objects.create(name='b', position='0', parent=parent) SomeChildModel.objects.create(name='c', position='1', parent=parent) - response = self.client.get('/admin/admin_inlines/someparentmodel/%s/' % parent.pk) + response = self.client.get(reverse('admin:admin_inlines_someparentmodel_change', args=(parent.pk,))) self.assertNotContains(response, '') self.assertContains(response, ( '', html=True) @@ -172,7 +179,7 @@ def test_localize_pk_shortcut(self): """ holder = Holder.objects.create(pk=123456789, dummy=42) inner = Inner.objects.create(pk=987654321, holder=holder, dummy=42, readonly='') - response = self.client.get('/admin/admin_inlines/holder/%i/' % holder.id) + response = self.client.get(reverse('admin:admin_inlines_holder_change', args=(holder.id,))) inner_shortcut = 'r/%s/%s/' % (ContentType.objects.get_for_model(inner).pk, inner.pk) self.assertContains(response, inner_shortcut) @@ -184,7 +191,7 @@ def test_custom_pk_shortcut(self): parent = ParentModelWithCustomPk.objects.create(my_own_pk="foo", name="Foo") child1 = ChildModel1.objects.create(my_own_pk="bar", name="Bar", parent=parent) child2 = ChildModel2.objects.create(my_own_pk="baz", name="Baz", parent=parent) - response = self.client.get('/admin/admin_inlines/parentmodelwithcustompk/foo/') + response = self.client.get(reverse('admin:admin_inlines_parentmodelwithcustompk_change', args=('foo',))) child1_shortcut = 'r/%s/%s/' % (ContentType.objects.get_for_model(child1).pk, child1.pk) child2_shortcut = 'r/%s/%s/' % (ContentType.objects.get_for_model(child2).pk, child2.pk) self.assertContains(response, child1_shortcut) @@ -203,7 +210,7 @@ def test_create_inlines_on_inherited_model(self): 'sighting_set-0-place': 'Zone 51', '_save': 'Save', } - response = self.client.post('/admin/admin_inlines/extraterrestrial/add/', data) + response = self.client.post(reverse('admin:admin_inlines_extraterrestrial_add'), data) self.assertEqual(response.status_code, 302) self.assertEqual(Sighting.objects.filter(et__name='Martian').count(), 1) @@ -217,11 +224,11 @@ def test_custom_get_extra_form(self): # The total number of forms will remain the same in either case total_forms_hidden = '' - response = self.client.get('/admin/admin_inlines/binarytree/add/') + response = self.client.get(reverse('admin:admin_inlines_binarytree_add')) self.assertContains(response, max_forms_input % 3) self.assertContains(response, total_forms_hidden) - response = self.client.get("/admin/admin_inlines/binarytree/%d/" % bt_head.id) + response = self.client.get(reverse('admin:admin_inlines_binarytree_change', args=(bt_head.id,))) self.assertContains(response, max_forms_input % 2) self.assertContains(response, total_forms_hidden) @@ -240,7 +247,7 @@ class MinNumInline(TabularInline): min_forms = '' total_forms = '' - request = self.factory.get('/admin/admin_inlines/binarytree/add/') + request = self.factory.get(reverse('admin:admin_inlines_binarytree_add')) request.user = User(username='super', is_superuser=True) response = modeladmin.changeform_view(request) self.assertContains(response, min_forms) @@ -268,20 +275,20 @@ def get_min_num(self, request, obj=None, **kwargs): min_forms = '' total_forms = '' - request = self.factory.get('/admin/admin_inlines/binarytree/add/') + request = self.factory.get(reverse('admin:admin_inlines_binarytree_add')) request.user = User(username='super', is_superuser=True) response = modeladmin.changeform_view(request) self.assertContains(response, min_forms % 2) self.assertContains(response, total_forms % 5) - request = self.factory.get("/admin/admin_inlines/binarytree/%d/" % bt_head.id) + request = self.factory.get(reverse('admin:admin_inlines_binarytree_change', args=(bt_head.id,))) request.user = User(username='super', is_superuser=True) response = modeladmin.changeform_view(request, object_id=str(bt_head.id)) self.assertContains(response, min_forms % 5) self.assertContains(response, total_forms % 8) def test_inline_nonauto_noneditable_pk(self): - response = self.client.get('/admin/admin_inlines/author/add/') + response = self.client.get(reverse('admin:admin_inlines_author_add')) self.assertContains(response, '', html=True) @@ -290,7 +297,7 @@ def test_inline_nonauto_noneditable_pk(self): html=True) def test_inline_editable_pk(self): - response = self.client.get('/admin/admin_inlines/author/add/') + response = self.client.get(reverse('admin:admin_inlines_author_add')) self.assertContains(response, '', html=True, count=1) @@ -301,7 +308,7 @@ def test_inline_editable_pk(self): def test_stacked_inline_edit_form_contains_has_original_class(self): holder = Holder.objects.create(dummy=1) holder.inner_set.create(dummy=1) - response = self.client.get('/admin/admin_inlines/holder/%s/' % holder.pk) + response = self.client.get(reverse('admin:admin_inlines_holder_change', args=(holder.pk,))) self.assertContains( response, '

', @@ -322,10 +329,10 @@ def test_inlines_show_change_link_registered(self): ('inner4stacked', item1.pk), ('inner4tabular', item2.pk), ) - response = self.client.get('/admin/admin_inlines/holder4/%s/' % holder.pk) + response = self.client.get(reverse('admin:admin_inlines_holder4_change', args=(holder.pk,))) self.assertTrue(response.context['inline_admin_formset'].opts.has_registered_model) for model, pk in items: - url = '/admin/admin_inlines/%s/%s/' % (model, pk) + url = reverse('admin:admin_inlines_%s_change' % model, args=(pk,)) self.assertContains(response, 'Author-book relationships') self.assertNotContains(response, 'Add another Author-Book Relationship') self.assertNotContains(response, 'id="id_Author_books-TOTAL_FORMS"') def test_inline_add_fk_noperm(self): - response = self.client.get('/admin/admin_inlines/holder2/add/') + response = self.client.get(reverse('admin:admin_inlines_holder2_add')) # No permissions on Inner2s, so no inline self.assertNotContains(response, '

Inner2s

') self.assertNotContains(response, 'Add another Inner2') @@ -536,7 +543,7 @@ def test_inline_change_fk_noperm(self): def test_inline_add_m2m_add_perm(self): permission = Permission.objects.get(codename='add_book', content_type=self.book_ct) self.user.user_permissions.add(permission) - response = self.client.get('/admin/admin_inlines/author/add/') + response = self.client.get(reverse('admin:admin_inlines_author_add')) # No change permission on Books, so no inline self.assertNotContains(response, '

Author-book relationships

') self.assertNotContains(response, 'Add another Author-Book Relationship') @@ -545,7 +552,7 @@ def test_inline_add_m2m_add_perm(self): def test_inline_add_fk_add_perm(self): permission = Permission.objects.get(codename='add_inner2', content_type=self.inner_ct) self.user.user_permissions.add(permission) - response = self.client.get('/admin/admin_inlines/holder2/add/') + response = self.client.get(reverse('admin:admin_inlines_holder2_add')) # Add permission on inner2s, so we get the inline self.assertContains(response, '

Inner2s

') self.assertContains(response, @@ -551,10 +541,7 @@ def test_named_group_field_choices_filter(self): def test_change_list_null_boolean_display(self): Post.objects.create(public=None) - # This hard-codes the URl because it'll fail if it runs - # against the 'admin2' custom admin (which doesn't have the - # Post model). - response = self.client.get("/test_admin/admin/admin_views/post/") + response = self.client.get(reverse('admin:admin_views_post_changelist')) self.assertContains(response, 'icon-unknown.gif') def test_i18n_language_non_english_default(self): @@ -564,7 +551,7 @@ def test_i18n_language_non_english_default(self): is English. See #13388 and #3594 for more details. """ with self.settings(LANGUAGE_CODE='fr'), translation.override('en-us'): - response = self.client.get('/test_admin/admin/jsi18n/') + response = self.client.get(reverse('admin:jsi18n')) self.assertNotContains(response, 'Choisir une heure') def test_i18n_language_non_english_fallback(self): @@ -573,7 +560,7 @@ def test_i18n_language_non_english_fallback(self): in cases where the selected language cannot be found. """ with self.settings(LANGUAGE_CODE='fr'), translation.override('none'): - response = self.client.get('/test_admin/admin/jsi18n/') + response = self.client.get(reverse('admin:jsi18n')) self.assertContains(response, 'Choisir une heure') def test_L10N_deactivated(self): @@ -582,87 +569,91 @@ def test_L10N_deactivated(self): return localized date/time formats. Refs #14824. """ with self.settings(LANGUAGE_CODE='ru', USE_L10N=False), translation.override('none'): - response = self.client.get('/test_admin/admin/jsi18n/') + response = self.client.get(reverse('admin:jsi18n')) self.assertNotContains(response, '%d.%m.%Y %H:%M:%S') self.assertContains(response, '%Y-%m-%d %H:%M:%S') def test_disallowed_filtering(self): with patch_logger('django.security.DisallowedModelAdminLookup', 'error') as calls: - response = self.client.get("/test_admin/admin/admin_views/album/?owner__email__startswith=fuzzy") + response = self.client.get( + "%s?owner__email__startswith=fuzzy" % reverse('admin:admin_views_album_changelist') + ) self.assertEqual(response.status_code, 400) self.assertEqual(len(calls), 1) # Filters are allowed if explicitly included in list_filter - response = self.client.get("/test_admin/admin/admin_views/thing/?color__value__startswith=red") + response = self.client.get("%s?color__value__startswith=red" % reverse('admin:admin_views_thing_changelist')) self.assertEqual(response.status_code, 200) - response = self.client.get("/test_admin/admin/admin_views/thing/?color__value=red") + response = self.client.get("%s?color__value=red" % reverse('admin:admin_views_thing_changelist')) self.assertEqual(response.status_code, 200) # Filters should be allowed if they involve a local field without the # need to whitelist them in list_filter or date_hierarchy. - response = self.client.get("/test_admin/admin/admin_views/person/?age__gt=30") + response = self.client.get("%s?age__gt=30" % reverse('admin:admin_views_person_changelist')) self.assertEqual(response.status_code, 200) e1 = Employee.objects.create(name='Anonymous', gender=1, age=22, alive=True, code='123') e2 = Employee.objects.create(name='Visitor', gender=2, age=19, alive=True, code='124') WorkHour.objects.create(datum=datetime.datetime.now(), employee=e1) WorkHour.objects.create(datum=datetime.datetime.now(), employee=e2) - response = self.client.get("/test_admin/admin/admin_views/workhour/") + response = self.client.get(reverse('admin:admin_views_workhour_changelist')) self.assertEqual(response.status_code, 200) self.assertContains(response, 'employee__person_ptr__exact') - response = self.client.get("/test_admin/admin/admin_views/workhour/?employee__person_ptr__exact=%d" % e1.pk) + response = self.client.get("%s?employee__person_ptr__exact=%d" % ( + reverse('admin:admin_views_workhour_changelist'), e1.pk) + ) self.assertEqual(response.status_code, 200) def test_disallowed_to_field(self): with patch_logger('django.security.DisallowedModelAdminToField', 'error') as calls: - response = self.client.get("/test_admin/admin/admin_views/section/", {TO_FIELD_VAR: 'missing_field'}) + response = self.client.get(reverse('admin:admin_views_section_changelist'), {TO_FIELD_VAR: 'missing_field'}) self.assertEqual(response.status_code, 400) self.assertEqual(len(calls), 1) # Specifying a field that is not referred by any other model registered # to this admin site should raise an exception. with patch_logger('django.security.DisallowedModelAdminToField', 'error') as calls: - response = self.client.get("/test_admin/admin/admin_views/section/", {TO_FIELD_VAR: 'name'}) + response = self.client.get(reverse('admin:admin_views_section_changelist'), {TO_FIELD_VAR: 'name'}) self.assertEqual(response.status_code, 400) self.assertEqual(len(calls), 1) # #23839 - Primary key should always be allowed, even if the referenced model isn't registered. - response = self.client.get("/test_admin/admin/admin_views/notreferenced/", {TO_FIELD_VAR: 'id'}) + response = self.client.get(reverse('admin:admin_views_notreferenced_changelist'), {TO_FIELD_VAR: 'id'}) self.assertEqual(response.status_code, 200) # #23915 - Specifying a field referenced by another model though a m2m should be allowed. - response = self.client.get("/test_admin/admin/admin_views/recipe/", {TO_FIELD_VAR: 'rname'}) + response = self.client.get(reverse('admin:admin_views_recipe_changelist'), {TO_FIELD_VAR: 'rname'}) self.assertEqual(response.status_code, 200) # #23604, #23915 - Specifying a field referenced through a reverse m2m relationship should be allowed. - response = self.client.get("/test_admin/admin/admin_views/ingredient/", {TO_FIELD_VAR: 'iname'}) + response = self.client.get(reverse('admin:admin_views_ingredient_changelist'), {TO_FIELD_VAR: 'iname'}) self.assertEqual(response.status_code, 200) # #23329 - Specifying a field that is not referred by any other model directly registered # to this admin site but registered through inheritance should be allowed. - response = self.client.get("/test_admin/admin/admin_views/referencedbyparent/", {TO_FIELD_VAR: 'name'}) + response = self.client.get(reverse('admin:admin_views_referencedbyparent_changelist'), {TO_FIELD_VAR: 'name'}) self.assertEqual(response.status_code, 200) # #23431 - Specifying a field that is only referred to by a inline of a registered # model should be allowed. - response = self.client.get("/test_admin/admin/admin_views/referencedbyinline/", {TO_FIELD_VAR: 'name'}) + response = self.client.get(reverse('admin:admin_views_referencedbyinline_changelist'), {TO_FIELD_VAR: 'name'}) self.assertEqual(response.status_code, 200) # We also want to prevent the add, change, and delete views from # leaking a disallowed field value. with patch_logger('django.security.DisallowedModelAdminToField', 'error') as calls: - response = self.client.post("/test_admin/admin/admin_views/section/add/", {TO_FIELD_VAR: 'name'}) + response = self.client.post(reverse('admin:admin_views_section_add'), {TO_FIELD_VAR: 'name'}) self.assertEqual(response.status_code, 400) self.assertEqual(len(calls), 1) section = Section.objects.create() with patch_logger('django.security.DisallowedModelAdminToField', 'error') as calls: - response = self.client.post("/test_admin/admin/admin_views/section/%d/" % section.pk, {TO_FIELD_VAR: 'name'}) + response = self.client.post(reverse('admin:admin_views_section_change', args=(section.pk,)), {TO_FIELD_VAR: 'name'}) self.assertEqual(response.status_code, 400) self.assertEqual(len(calls), 1) with patch_logger('django.security.DisallowedModelAdminToField', 'error') as calls: - response = self.client.post("/test_admin/admin/admin_views/section/%d/delete/" % section.pk, {TO_FIELD_VAR: 'name'}) + response = self.client.post(reverse('admin:admin_views_section_delete', args=(section.pk,)), {TO_FIELD_VAR: 'name'}) self.assertEqual(response.status_code, 400) self.assertEqual(len(calls), 1) @@ -673,7 +664,7 @@ def test_allowed_filtering_15103(self): can break. """ # Filters should be allowed if they are defined on a ForeignKey pointing to this model - response = self.client.get("/test_admin/admin/admin_views/inquisition/?leader__name=Palin&leader__age=27") + response = self.client.get("%s?leader__name=Palin&leader__age=27" % reverse('admin:admin_views_inquisition_changelist')) self.assertEqual(response.status_code, 200) def test_popup_dismiss_related(self): @@ -681,7 +672,7 @@ def test_popup_dismiss_related(self): Regression test for ticket 20664 - ensure the pk is properly quoted. """ actor = Actor.objects.create(name="Palin", age=27) - response = self.client.get("/test_admin/admin/admin_views/actor/?%s" % IS_POPUP_VAR) + response = self.client.get("%s?%s" % (reverse('admin:admin_views_actor_changelist'), IS_POPUP_VAR)) self.assertContains(response, "opener.dismissRelatedLookupPopup(window, '%s')" % actor.pk) def test_hide_change_password(self): @@ -694,7 +685,7 @@ def test_hide_change_password(self): user.set_unusable_password() user.save() - response = self.client.get('/test_admin/admin/') + response = self.client.get(reverse('admin:index')) self.assertNotContains(response, reverse('admin:password_change'), msg_prefix='The "change password" link should not be displayed if a user does not have a usable password.') @@ -705,8 +696,7 @@ def test_change_view_with_show_delete_extra_context(self): Refs #10057. """ instance = UndeletableObject.objects.create(name='foo') - response = self.client.get('/test_admin/%s/admin_views/undeletableobject/%d/' % - (self.urlbit, instance.pk)) + response = self.client.get(reverse('admin:admin_views_undeletableobject_change', args=(instance.pk,))) self.assertNotContains(response, 'deletelink') def test_allows_attributeerror_to_bubble_up(self): @@ -720,7 +710,7 @@ def test_allows_attributeerror_to_bubble_up(self): """ Simple.objects.create() with self.assertRaises(AttributeError): - self.client.get('/test_admin/%s/admin_views/simple/' % self.urlbit) + self.client.get(reverse('admin:admin_views_simple_changelist')) def test_changelist_with_no_change_url(self): """ @@ -730,7 +720,7 @@ def test_changelist_with_no_change_url(self): Regression test for #20934 """ UnchangeableObject.objects.create() - response = self.client.get('/test_admin/admin/admin_views/unchangeableobject/') + response = self.client.get(reverse('admin:admin_views_unchangeableobject_changelist')) self.assertEqual(response.status_code, 200) # Check the format of the shown object -- shouldn't contain a change link self.assertContains(response, 'UnchangeableObject object', html=True) @@ -812,7 +802,7 @@ def test_extended_bodyclass_template_change_form(self): Ensure that the admin/change_form.html template uses block.super in the bodyclass block. """ - response = self.client.get('/test_admin/%s/admin_views/section/add/' % self.urlbit) + response = self.client.get(reverse('admin:admin_views_section_add')) self.assertContains(response, 'bodyclass_consistency_check ') def test_extended_bodyclass_template_change_password(self): @@ -821,7 +811,7 @@ def test_extended_bodyclass_template_change_password(self): super in the bodyclass block. """ user = User.objects.get(username='super') - response = self.client.get('/test_admin/%s/auth/user/%s/password/' % (self.urlbit, user.id)) + response = self.client.get(reverse('admin:auth_user_password_change', args=(user.id,))) self.assertContains(response, 'bodyclass_consistency_check ') def test_extended_bodyclass_template_index(self): @@ -829,7 +819,7 @@ def test_extended_bodyclass_template_index(self): Ensure that the admin/index.html template uses block.super in the bodyclass block. """ - response = self.client.get('/test_admin/%s/' % self.urlbit) + response = self.client.get(reverse('admin:index')) self.assertContains(response, 'bodyclass_consistency_check ') def test_extended_bodyclass_change_list(self): @@ -837,7 +827,7 @@ def test_extended_bodyclass_change_list(self): Ensure that the admin/change_list.html' template uses block.super in the bodyclass block. """ - response = self.client.get('/test_admin/%s/admin_views/article/' % self.urlbit) + response = self.client.get(reverse('admin:admin_views_article_changelist')) self.assertContains(response, 'bodyclass_consistency_check ') def test_extended_bodyclass_template_login(self): @@ -846,7 +836,7 @@ def test_extended_bodyclass_template_login(self): bodyclass block. """ self.client.logout() - response = self.client.get('/test_admin/%s/login/' % self.urlbit) + response = self.client.get(reverse('admin:login')) self.assertContains(response, 'bodyclass_consistency_check ') def test_extended_bodyclass_template_delete_confirmation(self): @@ -855,7 +845,7 @@ def test_extended_bodyclass_template_delete_confirmation(self): block.super in the bodyclass block. """ group = Group.objects.create(name="foogroup") - response = self.client.get('/test_admin/%s/auth/group/%s/delete/' % (self.urlbit, group.id)) + response = self.client.get(reverse('admin:auth_group_delete', args=(group.id,))) self.assertContains(response, 'bodyclass_consistency_check ') def test_extended_bodyclass_template_delete_selected_confirmation(self): @@ -870,7 +860,7 @@ def test_extended_bodyclass_template_delete_selected_confirmation(self): 'index': '0', '_selected_action': group.id } - response = self.client.post('/test_admin/%s/auth/group/' % (self.urlbit), post_data) + response = self.client.post(reverse('admin:auth_group_changelist'), post_data) self.assertEqual(response.context['site_header'], 'Django administration') self.assertContains(response, 'bodyclass_consistency_check ') @@ -879,7 +869,7 @@ def test_filter_with_custom_template(self): Ensure that one can use a custom template to render an admin filter. Refs #17515. """ - response = self.client.get("/test_admin/admin/admin_views/color2/") + response = self.client.get(reverse('admin:admin_views_color2_changelist')) self.assertTemplateUsed(response, 'custom_filter_template.html') @@ -887,7 +877,7 @@ def test_filter_with_custom_template(self): ROOT_URLCONF="admin_views.urls") class AdminViewFormUrlTest(TestCase): fixtures = ["admin-views-users.xml"] - urlbit = "admin3" + current_app = "admin3" def setUp(self): self.client.login(username='super', password='secret') @@ -896,7 +886,9 @@ def test_change_form_URL_has_correct_value(self): """ Tests whether change_view has form_url in response.context """ - response = self.client.get('/test_admin/%s/admin_views/section/1/' % self.urlbit) + response = self.client.get( + reverse('admin:admin_views_section_change', args=(1,), current_app=self.current_app) + ) self.assertIn('form_url', response.context, msg='form_url not present in response.context') self.assertEqual(response.context['form_url'], 'pony') @@ -907,7 +899,10 @@ def test_initial_data_can_be_overridden(self): Usually, the initial value is set via the GET params. """ - response = self.client.get('/test_admin/%s/admin_views/restaurant/add/' % self.urlbit, {'name': 'test_value'}) + response = self.client.get( + reverse('admin:admin_views_restaurant_add', current_app=self.current_app), + {'name': 'test_value'} + ) # this would be the usual behaviour self.assertNotContains(response, 'value="test_value"') # this is the overridden behaviour @@ -929,8 +924,7 @@ def test_js_minified_only_if_debug_is_false(self): Refs #17521. """ with override_settings(DEBUG=False): - response = self.client.get( - '/test_admin/%s/admin_views/section/add/' % 'admin') + response = self.client.get(reverse('admin:admin_views_section_add')) self.assertNotContains(response, 'jquery.js') self.assertContains(response, 'jquery.min.js') self.assertNotContains(response, 'prepopulate.js') @@ -942,8 +936,7 @@ def test_js_minified_only_if_debug_is_false(self): self.assertNotContains(response, 'inlines.js') self.assertContains(response, 'inlines.min.js') with override_settings(DEBUG=True): - response = self.client.get( - '/test_admin/%s/admin_views/section/add/' % 'admin') + response = self.client.get(reverse('admin:admin_views_section_add')) self.assertContains(response, 'jquery.js') self.assertNotContains(response, 'jquery.min.js') self.assertContains(response, 'prepopulate.js') @@ -967,7 +960,7 @@ def setUp(self): def test_save_as_duplication(self): """Ensure save as actually creates a new person""" post_data = {'_saveasnew': '', 'name': 'John M', 'gender': 1, 'age': 42} - self.client.post('/test_admin/admin/admin_views/person/1/', post_data) + self.client.post(reverse('admin:admin_views_person_change', args=(1,)), post_data) self.assertEqual(len(Person.objects.filter(name='John M')), 1) self.assertEqual(len(Person.objects.filter(id=1)), 1) @@ -977,24 +970,24 @@ def test_save_as_display(self): invalid data aside save_as_new will not show us a form to overwrite the initial model. """ - response = self.client.get('/test_admin/admin/admin_views/person/1/') + change_url = reverse('admin:admin_views_person_change', args=(1,)) + response = self.client.get(change_url) self.assertTrue(response.context['save_as']) post_data = {'_saveasnew': '', 'name': 'John M', 'gender': 3, 'alive': 'checked'} - response = self.client.post('/test_admin/admin/admin_views/person/1/', post_data) - self.assertEqual(response.context['form_url'], '/test_admin/admin/admin_views/person/add/') + response = self.client.post(change_url, post_data) + self.assertEqual(response.context['form_url'], reverse('admin:admin_views_person_add')) @override_settings(ROOT_URLCONF="admin_views.urls") class CustomModelAdminTest(AdminViewBasicTestCase): - urlbit = "admin2" def test_custom_admin_site_login_form(self): self.client.logout() - response = self.client.get('/test_admin/admin2/', follow=True) + response = self.client.get(reverse('admin2:index'), follow=True) self.assertIsInstance(response, TemplateResponse) self.assertEqual(response.status_code, 200) - login = self.client.post('/test_admin/admin2/login/', { - REDIRECT_FIELD_NAME: '/test_admin/admin2/', + login = self.client.post(reverse('admin2:login'), { + REDIRECT_FIELD_NAME: reverse('admin2:index'), 'username': 'customform', 'password': 'secret', }, follow=True) @@ -1004,20 +997,20 @@ def test_custom_admin_site_login_form(self): def test_custom_admin_site_login_template(self): self.client.logout() - response = self.client.get('/test_admin/admin2/', follow=True) + response = self.client.get(reverse('admin2:index'), follow=True) self.assertIsInstance(response, TemplateResponse) self.assertTemplateUsed(response, 'custom_admin/login.html') self.assertContains(response, 'Hello from a custom login template') def test_custom_admin_site_logout_template(self): - response = self.client.get('/test_admin/admin2/logout/') + response = self.client.get(reverse('admin2:logout')) self.assertIsInstance(response, TemplateResponse) self.assertTemplateUsed(response, 'custom_admin/logout.html') self.assertContains(response, 'Hello from a custom logout template') def test_custom_admin_site_index_view_and_template(self): try: - response = self.client.get('/test_admin/admin2/') + response = self.client.get(reverse('admin2:index')) except TypeError: self.fail('AdminSite.index_template should accept a list of template paths') self.assertIsInstance(response, TemplateResponse) @@ -1025,39 +1018,41 @@ def test_custom_admin_site_index_view_and_template(self): self.assertContains(response, 'Hello from a custom index template *bar*') def test_custom_admin_site_app_index_view_and_template(self): - response = self.client.get('/test_admin/admin2/admin_views/') + response = self.client.get(reverse('admin2:app_list', args=('admin_views',))) self.assertIsInstance(response, TemplateResponse) self.assertTemplateUsed(response, 'custom_admin/app_index.html') self.assertContains(response, 'Hello from a custom app_index template') def test_custom_admin_site_password_change_template(self): - response = self.client.get('/test_admin/admin2/password_change/') + response = self.client.get(reverse('admin2:password_change')) self.assertIsInstance(response, TemplateResponse) self.assertTemplateUsed(response, 'custom_admin/password_change_form.html') self.assertContains(response, 'Hello from a custom password change form template') def test_custom_admin_site_password_change_with_extra_context(self): - response = self.client.get('/test_admin/admin2/password_change/') + response = self.client.get(reverse('admin2:password_change')) self.assertIsInstance(response, TemplateResponse) self.assertTemplateUsed(response, 'custom_admin/password_change_form.html') self.assertContains(response, 'eggs') def test_custom_admin_site_password_change_done_template(self): - response = self.client.get('/test_admin/admin2/password_change/done/') + response = self.client.get(reverse('admin2:password_change_done')) self.assertIsInstance(response, TemplateResponse) self.assertTemplateUsed(response, 'custom_admin/password_change_done.html') self.assertContains(response, 'Hello from a custom password change done template') def test_custom_admin_site_view(self): self.client.login(username='super', password='secret') - response = self.client.get('/test_admin/%s/my_view/' % self.urlbit) + response = self.client.get(reverse('admin2:my_view')) self.assertEqual(response.content, b"Django is a magical pony!") def test_pwd_change_custom_template(self): self.client.login(username='super', password='secret') su = User.objects.get(username='super') try: - response = self.client.get('/test_admin/admin4/auth/user/%s/password/' % su.pk) + response = self.client.get( + reverse('admin4:auth_user_password_change', args=(su.pk,)) + ) except TypeError: self.fail('ModelAdmin.change_user_password_template should accept a list of template paths') self.assertEqual(response.status_code, 200) @@ -1106,48 +1101,49 @@ def setUp(self): get_permission_codename('delete', Section._meta))) # login POST dicts + self.index_url = reverse('admin:index') self.super_login = { - REDIRECT_FIELD_NAME: '/test_admin/admin/', + REDIRECT_FIELD_NAME: self.index_url, 'username': 'super', 'password': 'secret', } self.super_email_login = { - REDIRECT_FIELD_NAME: '/test_admin/admin/', + REDIRECT_FIELD_NAME: self.index_url, 'username': 'super@example.com', 'password': 'secret', } self.super_email_bad_login = { - REDIRECT_FIELD_NAME: '/test_admin/admin/', + REDIRECT_FIELD_NAME: self.index_url, 'username': 'super@example.com', 'password': 'notsecret', } self.adduser_login = { - REDIRECT_FIELD_NAME: '/test_admin/admin/', + REDIRECT_FIELD_NAME: self.index_url, 'username': 'adduser', 'password': 'secret', } self.changeuser_login = { - REDIRECT_FIELD_NAME: '/test_admin/admin/', + REDIRECT_FIELD_NAME: self.index_url, 'username': 'changeuser', 'password': 'secret', } self.deleteuser_login = { - REDIRECT_FIELD_NAME: '/test_admin/admin/', + REDIRECT_FIELD_NAME: self.index_url, 'username': 'deleteuser', 'password': 'secret', } self.nostaff_login = { - REDIRECT_FIELD_NAME: '/test_admin/has_permission_admin/', + REDIRECT_FIELD_NAME: reverse('has_permission_admin:index'), 'username': 'nostaff', 'password': 'secret', } self.joepublic_login = { - REDIRECT_FIELD_NAME: '/test_admin/admin/', + REDIRECT_FIELD_NAME: self.index_url, 'username': 'joepublic', 'password': 'secret', } self.no_username_login = { - REDIRECT_FIELD_NAME: '/test_admin/admin/', + REDIRECT_FIELD_NAME: self.index_url, 'password': 'secret', } @@ -1159,17 +1155,17 @@ def test_login(self): Unsuccessful attempts will continue to render the login page with a 200 status code. """ - login_url = reverse('admin:login') + '?next=/test_admin/admin/' + login_url = '%s?next=%s' % (reverse('admin:login'), reverse('admin:index')) # Super User - response = self.client.get('/test_admin/admin/') + response = self.client.get(self.index_url) self.assertEqual(response.status_code, 302) login = self.client.post(login_url, self.super_login) - self.assertRedirects(login, '/test_admin/admin/') + self.assertRedirects(login, self.index_url) self.assertFalse(login.context) - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) # Test if user enters email address - response = self.client.get('/test_admin/admin/') + response = self.client.get(self.index_url) self.assertEqual(response.status_code, 302) login = self.client.post(login_url, self.super_email_login) self.assertContains(login, ERROR_MESSAGE) @@ -1183,38 +1179,38 @@ def test_login(self): self.assertContains(login, ERROR_MESSAGE) # Add User - response = self.client.get('/test_admin/admin/') + response = self.client.get(self.index_url) self.assertEqual(response.status_code, 302) login = self.client.post(login_url, self.adduser_login) - self.assertRedirects(login, '/test_admin/admin/') + self.assertRedirects(login, self.index_url) self.assertFalse(login.context) - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) # Change User - response = self.client.get('/test_admin/admin/') + response = self.client.get(self.index_url) self.assertEqual(response.status_code, 302) login = self.client.post(login_url, self.changeuser_login) - self.assertRedirects(login, '/test_admin/admin/') + self.assertRedirects(login, self.index_url) self.assertFalse(login.context) - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) # Delete User - response = self.client.get('/test_admin/admin/') + response = self.client.get(self.index_url) self.assertEqual(response.status_code, 302) login = self.client.post(login_url, self.deleteuser_login) - self.assertRedirects(login, '/test_admin/admin/') + self.assertRedirects(login, self.index_url) self.assertFalse(login.context) - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) # Regular User should not be able to login. - response = self.client.get('/test_admin/admin/') + response = self.client.get(self.index_url) self.assertEqual(response.status_code, 302) login = self.client.post(login_url, self.joepublic_login) self.assertEqual(login.status_code, 200) self.assertContains(login, ERROR_MESSAGE) # Requests without username should not return 500 errors. - response = self.client.get('/test_admin/admin/') + response = self.client.get(self.index_url) self.assertEqual(response.status_code, 302) login = self.client.post(login_url, self.no_username_login) self.assertEqual(login.status_code, 200) @@ -1223,37 +1219,37 @@ def test_login(self): def test_login_has_permission(self): # Regular User should not be able to login. - response = self.client.get('/test_admin/has_permission_admin/') + response = self.client.get(reverse('has_permission_admin:index')) self.assertEqual(response.status_code, 302) - login = self.client.post('/test_admin/has_permission_admin/login/', self.joepublic_login) + login = self.client.post(reverse('has_permission_admin:login'), self.joepublic_login) self.assertEqual(login.status_code, 200) self.assertContains(login, 'permission denied') # User with permissions should be able to login. - response = self.client.get('/test_admin/has_permission_admin/') + response = self.client.get(reverse('has_permission_admin:index')) self.assertEqual(response.status_code, 302) - login = self.client.post('/test_admin/has_permission_admin/login/', self.nostaff_login) - self.assertRedirects(login, '/test_admin/has_permission_admin/') + login = self.client.post(reverse('has_permission_admin:login'), self.nostaff_login) + self.assertRedirects(login, reverse('has_permission_admin:index')) self.assertFalse(login.context) - self.client.get('/test_admin/has_permission_admin/logout/') + self.client.get(reverse('has_permission_admin:logout')) # Staff should be able to login. - response = self.client.get('/test_admin/has_permission_admin/') + response = self.client.get(reverse('has_permission_admin:index')) self.assertEqual(response.status_code, 302) - login = self.client.post('/test_admin/has_permission_admin/login/', { - REDIRECT_FIELD_NAME: '/test_admin/has_permission_admin/', + login = self.client.post(reverse('has_permission_admin:login'), { + REDIRECT_FIELD_NAME: reverse('has_permission_admin:index'), 'username': 'deleteuser', 'password': 'secret', }) - self.assertRedirects(login, '/test_admin/has_permission_admin/') + self.assertRedirects(login, reverse('has_permission_admin:index')) self.assertFalse(login.context) - self.client.get('/test_admin/has_permission_admin/logout/') + self.client.get(reverse('has_permission_admin:logout')) def test_login_successfully_redirects_to_original_URL(self): - response = self.client.get('/test_admin/admin/') + response = self.client.get(self.index_url) self.assertEqual(response.status_code, 302) query_string = 'the-answer=42' - redirect_url = '/test_admin/admin/?%s' % query_string + redirect_url = '%s?%s' % (self.index_url, query_string) new_next = {REDIRECT_FIELD_NAME: redirect_url} post_data = self.super_login.copy() post_data.pop(REDIRECT_FIELD_NAME) @@ -1264,14 +1260,14 @@ def test_login_successfully_redirects_to_original_URL(self): def test_double_login_is_not_allowed(self): """Regression test for #19327""" - login_url = reverse('admin:login') + '?next=/test_admin/admin/' + login_url = '%s?next=%s' % (reverse('admin:login'), reverse('admin:index')) - response = self.client.get('/test_admin/admin/') + response = self.client.get(self.index_url) self.assertEqual(response.status_code, 302) # Establish a valid admin session login = self.client.post(login_url, self.super_login) - self.assertRedirects(login, '/test_admin/admin/') + self.assertRedirects(login, self.index_url) self.assertFalse(login.context) # Logging in with non-admin user fails @@ -1281,67 +1277,67 @@ def test_double_login_is_not_allowed(self): # Establish a valid admin session login = self.client.post(login_url, self.super_login) - self.assertRedirects(login, '/test_admin/admin/') + self.assertRedirects(login, self.index_url) self.assertFalse(login.context) # Logging in with admin user while already logged in login = self.client.post(login_url, self.super_login) - self.assertRedirects(login, '/test_admin/admin/') + self.assertRedirects(login, self.index_url) self.assertFalse(login.context) - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) def test_add_view(self): """Test add view restricts access and actually adds items.""" - login_url = reverse('admin:login') + '?next=/test_admin/admin/' + login_url = '%s?next=%s' % (reverse('admin:login'), reverse('admin:index')) add_dict = {'title': 'Døm ikke', 'content': '

great article

', 'date_0': '2008-03-18', 'date_1': '10:54:39', 'section': 1} # Change User should not have access to add articles - self.client.get('/test_admin/admin/') + self.client.get(self.index_url) self.client.post(login_url, self.changeuser_login) # make sure the view removes test cookie self.assertEqual(self.client.session.test_cookie_worked(), False) - response = self.client.get('/test_admin/admin/admin_views/article/add/') + response = self.client.get(reverse('admin:admin_views_article_add')) self.assertEqual(response.status_code, 403) # Try POST just to make sure - post = self.client.post('/test_admin/admin/admin_views/article/add/', add_dict) + post = self.client.post(reverse('admin:admin_views_article_add'), add_dict) self.assertEqual(post.status_code, 403) self.assertEqual(Article.objects.all().count(), 3) - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) # Add user may login and POST to add view, then redirect to admin root - self.client.get('/test_admin/admin/') + self.client.get(self.index_url) self.client.post(login_url, self.adduser_login) - addpage = self.client.get('/test_admin/admin/admin_views/article/add/') - change_list_link = '› Articles' + addpage = self.client.get(reverse('admin:admin_views_article_add')) + change_list_link = '› Articles' % reverse('admin:admin_views_article_changelist') self.assertNotContains(addpage, change_list_link, msg_prefix='User restricted to add permission is given link to change list view in breadcrumbs.') - post = self.client.post('/test_admin/admin/admin_views/article/add/', add_dict) - self.assertRedirects(post, '/test_admin/admin/') + post = self.client.post(reverse('admin:admin_views_article_add'), add_dict) + self.assertRedirects(post, self.index_url) self.assertEqual(Article.objects.all().count(), 4) self.assertEqual(len(mail.outbox), 1) self.assertEqual(mail.outbox[0].subject, 'Greetings from a created object') - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) # Super can add too, but is redirected to the change list view - self.client.get('/test_admin/admin/') + self.client.get(self.index_url) self.client.post(login_url, self.super_login) - addpage = self.client.get('/test_admin/admin/admin_views/article/add/') + addpage = self.client.get(reverse('admin:admin_views_article_add')) self.assertContains(addpage, change_list_link, msg_prefix='Unrestricted user is not given link to change list view in breadcrumbs.') - post = self.client.post('/test_admin/admin/admin_views/article/add/', add_dict) - self.assertRedirects(post, '/test_admin/admin/admin_views/article/') + post = self.client.post(reverse('admin:admin_views_article_add'), add_dict) + self.assertRedirects(post, reverse('admin:admin_views_article_changelist')) self.assertEqual(Article.objects.all().count(), 5) - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) # 8509 - if a normal user is already logged in, it is possible # to change user into the superuser without error self.client.login(username='joepublic', password='secret') # Check and make sure that if user expires, data still persists - self.client.get('/test_admin/admin/') + self.client.get(self.index_url) self.client.post(login_url, self.super_login) # make sure the view removes test cookie self.assertEqual(self.client.session.test_cookie_worked(), False) @@ -1349,97 +1345,101 @@ def test_add_view(self): def test_change_view(self): """Change view should restrict access and allow users to edit items.""" - login_url = reverse('admin:login') + '?next=/test_admin/admin/' + login_url = '%s?next=%s' % (reverse('admin:login'), reverse('admin:index')) change_dict = {'title': 'Ikke fordømt', 'content': '

edited article

', 'date_0': '2008-03-18', 'date_1': '10:54:39', 'section': 1} + article_change_url = reverse('admin:admin_views_article_change', args=(1,)) + article_changelist_url = reverse('admin:admin_views_article_changelist') # add user should not be able to view the list of article or change any of them - self.client.get('/test_admin/admin/') + self.client.get(self.index_url) self.client.post(login_url, self.adduser_login) - response = self.client.get('/test_admin/admin/admin_views/article/') + response = self.client.get(article_changelist_url) self.assertEqual(response.status_code, 403) - response = self.client.get('/test_admin/admin/admin_views/article/1/') + response = self.client.get(article_change_url) self.assertEqual(response.status_code, 403) - post = self.client.post('/test_admin/admin/admin_views/article/1/', change_dict) + post = self.client.post(article_change_url, change_dict) self.assertEqual(post.status_code, 403) - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) # change user can view all items and edit them - self.client.get('/test_admin/admin/') + self.client.get(self.index_url) self.client.post(login_url, self.changeuser_login) - response = self.client.get('/test_admin/admin/admin_views/article/') + response = self.client.get(article_changelist_url) self.assertEqual(response.status_code, 200) - response = self.client.get('/test_admin/admin/admin_views/article/1/') + response = self.client.get(article_change_url) self.assertEqual(response.status_code, 200) - post = self.client.post('/test_admin/admin/admin_views/article/1/', change_dict) - self.assertRedirects(post, '/test_admin/admin/admin_views/article/') + post = self.client.post(article_change_url, change_dict) + self.assertRedirects(post, article_changelist_url) self.assertEqual(Article.objects.get(pk=1).content, '

edited article

') # one error in form should produce singular error message, multiple errors plural change_dict['title'] = '' - post = self.client.post('/test_admin/admin/admin_views/article/1/', change_dict) + post = self.client.post(article_change_url, change_dict) self.assertContains(post, 'Please correct the error below.', msg_prefix='Singular error message not found in response to post with one error') change_dict['content'] = '' - post = self.client.post('/test_admin/admin/admin_views/article/1/', change_dict) + post = self.client.post(article_change_url, change_dict) self.assertContains(post, 'Please correct the errors below.', msg_prefix='Plural error message not found in response to post with multiple errors') - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) # Test redirection when using row-level change permissions. Refs #11513. - RowLevelChangePermissionModel.objects.create(id=1, name="odd id") - RowLevelChangePermissionModel.objects.create(id=2, name="even id") + r1 = RowLevelChangePermissionModel.objects.create(id=1, name="odd id") + r2 = RowLevelChangePermissionModel.objects.create(id=2, name="even id") + change_url_1 = reverse('admin:admin_views_rowlevelchangepermissionmodel_change', args=(r1.pk,)) + change_url_2 = reverse('admin:admin_views_rowlevelchangepermissionmodel_change', args=(r2.pk,)) for login_dict in [self.super_login, self.changeuser_login, self.adduser_login, self.deleteuser_login]: self.client.post(login_url, login_dict) - response = self.client.get('/test_admin/admin/admin_views/rowlevelchangepermissionmodel/1/') + response = self.client.get(change_url_1) self.assertEqual(response.status_code, 403) - response = self.client.post('/test_admin/admin/admin_views/rowlevelchangepermissionmodel/1/', {'name': 'changed'}) + response = self.client.post(change_url_1, {'name': 'changed'}) self.assertEqual(RowLevelChangePermissionModel.objects.get(id=1).name, 'odd id') self.assertEqual(response.status_code, 403) - response = self.client.get('/test_admin/admin/admin_views/rowlevelchangepermissionmodel/2/') + response = self.client.get(change_url_2) self.assertEqual(response.status_code, 200) - response = self.client.post('/test_admin/admin/admin_views/rowlevelchangepermissionmodel/2/', {'name': 'changed'}) + response = self.client.post(change_url_2, {'name': 'changed'}) self.assertEqual(RowLevelChangePermissionModel.objects.get(id=2).name, 'changed') - self.assertRedirects(response, '/test_admin/admin/') - self.client.get('/test_admin/admin/logout/') + self.assertRedirects(response, self.index_url) + self.client.get(reverse('admin:logout')) for login_dict in [self.joepublic_login, self.no_username_login]: self.client.post(login_url, login_dict) - response = self.client.get('/test_admin/admin/admin_views/rowlevelchangepermissionmodel/1/', follow=True) + response = self.client.get(change_url_1, follow=True) self.assertEqual(response.status_code, 200) self.assertContains(response, 'login-form') - response = self.client.post('/test_admin/admin/admin_views/rowlevelchangepermissionmodel/1/', {'name': 'changed'}, follow=True) + response = self.client.post(change_url_1, {'name': 'changed'}, follow=True) self.assertEqual(RowLevelChangePermissionModel.objects.get(id=1).name, 'odd id') self.assertEqual(response.status_code, 200) self.assertContains(response, 'login-form') - response = self.client.get('/test_admin/admin/admin_views/rowlevelchangepermissionmodel/2/', follow=True) + response = self.client.get(change_url_2, follow=True) self.assertEqual(response.status_code, 200) self.assertContains(response, 'login-form') - response = self.client.post('/test_admin/admin/admin_views/rowlevelchangepermissionmodel/2/', {'name': 'changed again'}, follow=True) + response = self.client.post(change_url_2, {'name': 'changed again'}, follow=True) self.assertEqual(RowLevelChangePermissionModel.objects.get(id=2).name, 'changed') self.assertEqual(response.status_code, 200) self.assertContains(response, 'login-form') - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) def test_history_view(self): """History view should restrict access.""" - login_url = reverse('admin:login') + '?next=/test_admin/admin/' + login_url = '%s?next=%s' % (reverse('admin:login'), reverse('admin:index')) # add user should not be able to view the list of article or change any of them - self.client.get('/test_admin/admin/') + self.client.get(self.index_url) self.client.post(login_url, self.adduser_login) - response = self.client.get('/test_admin/admin/admin_views/article/1/history/') + response = self.client.get(reverse('admin:admin_views_article_history', args=(1,))) self.assertEqual(response.status_code, 403) - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) # change user can view all items and edit them - self.client.get('/test_admin/admin/') + self.client.get(self.index_url) self.client.post(login_url, self.changeuser_login) - response = self.client.get('/test_admin/admin/admin_views/article/1/history/') + response = self.client.get(reverse('admin:admin_views_article_history', args=(1,))) self.assertEqual(response.status_code, 200) # Test redirection when using row-level change permissions. Refs #11513. @@ -1447,33 +1447,37 @@ def test_history_view(self): RowLevelChangePermissionModel.objects.create(id=2, name="even id") for login_dict in [self.super_login, self.changeuser_login, self.adduser_login, self.deleteuser_login]: self.client.post(login_url, login_dict) - response = self.client.get('/test_admin/admin/admin_views/rowlevelchangepermissionmodel/1/history/') + response = self.client.get(reverse('admin:admin_views_rowlevelchangepermissionmodel_history', args=(1,))) self.assertEqual(response.status_code, 403) - response = self.client.get('/test_admin/admin/admin_views/rowlevelchangepermissionmodel/2/history/') + response = self.client.get(reverse('admin:admin_views_rowlevelchangepermissionmodel_history', args=(2,))) self.assertEqual(response.status_code, 200) - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) for login_dict in [self.joepublic_login, self.no_username_login]: self.client.post(login_url, login_dict) - response = self.client.get('/test_admin/admin/admin_views/rowlevelchangepermissionmodel/1/history/', follow=True) + response = self.client.get( + reverse('admin:admin_views_rowlevelchangepermissionmodel_history', args=(1,)), follow=True + ) self.assertEqual(response.status_code, 200) self.assertContains(response, 'login-form') - response = self.client.get('/test_admin/admin/admin_views/rowlevelchangepermissionmodel/2/history/', follow=True) + response = self.client.get( + reverse('admin:admin_views_rowlevelchangepermissionmodel_history', args=(2,)), follow=True + ) self.assertEqual(response.status_code, 200) self.assertContains(response, 'login-form') - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) def test_conditionally_show_add_section_link(self): """ The foreign key widget should only show the "add related" button if the user has permission to add that related item. """ - login_url = reverse('admin:login') + '?next=/test_admin/admin/' + login_url = '%s?next=%s' % (reverse('admin:login'), reverse('admin:index')) # Set up and log in user. - url = '/test_admin/admin/admin_views/article/add/' + url = reverse('admin:admin_views_article_add') add_link_text = 'add_id_section' self.client.post(login_url, self.adduser_login) # The user can't add sections yet, so they shouldn't see the "add @@ -1498,7 +1502,7 @@ def get_change_related(response): login_url = reverse('admin:login') # Set up and log in user. - url = '/test_admin/admin/admin_views/article/add/' + url = reverse('admin:admin_views_article_add') change_link_text = 'change_id_section' self.client.post(login_url, self.adduser_login) # The user can't change sections yet, so they shouldn't see the "change @@ -1525,9 +1529,9 @@ def get_delete_related(response): login_url = reverse('admin:login') # Set up and log in user. - url = '/test_admin/admin/admin_views/article/add/' + url = reverse('admin:admin_views_article_add') delete_link_text = 'delete_id_sub_section' - self.client.get('/test_admin/admin/') + self.client.get(self.index_url) self.client.post(login_url, self.adduser_login) # The user can't delete sections yet, so they shouldn't see the "delete # section" link. @@ -1544,85 +1548,86 @@ def get_delete_related(response): self.assertContains(response, delete_link_text) def test_custom_model_admin_templates(self): - login_url = reverse('admin:login') + '?next=/test_admin/admin/' - self.client.get('/test_admin/admin/') + login_url = '%s?next=%s' % (reverse('admin:login'), reverse('admin:index')) + self.client.get(self.index_url) self.client.post(login_url, self.super_login) # Test custom change list template with custom extra context - response = self.client.get('/test_admin/admin/admin_views/customarticle/') + response = self.client.get(reverse('admin:admin_views_customarticle_changelist')) self.assertContains(response, "var hello = 'Hello!';") self.assertTemplateUsed(response, 'custom_admin/change_list.html') # Test custom add form template - response = self.client.get('/test_admin/admin/admin_views/customarticle/add/') + response = self.client.get(reverse('admin:admin_views_customarticle_add')) self.assertTemplateUsed(response, 'custom_admin/add_form.html') # Add an article so we can test delete, change, and history views - post = self.client.post('/test_admin/admin/admin_views/customarticle/add/', { + post = self.client.post(reverse('admin:admin_views_customarticle_add'), { 'content': '

great article

', 'date_0': '2008-03-18', 'date_1': '10:54:39' }) - self.assertRedirects(post, '/test_admin/admin/admin_views/customarticle/') + self.assertRedirects(post, reverse('admin:admin_views_customarticle_changelist')) self.assertEqual(CustomArticle.objects.all().count(), 1) article_pk = CustomArticle.objects.all()[0].pk # Test custom delete, change, and object history templates # Test custom change form template - response = self.client.get('/test_admin/admin/admin_views/customarticle/%d/' % article_pk) + response = self.client.get(reverse('admin:admin_views_customarticle_change', args=(article_pk,))) self.assertTemplateUsed(response, 'custom_admin/change_form.html') - response = self.client.get('/test_admin/admin/admin_views/customarticle/%d/delete/' % article_pk) + response = self.client.get(reverse('admin:admin_views_customarticle_delete', args=(article_pk,))) self.assertTemplateUsed(response, 'custom_admin/delete_confirmation.html') - response = self.client.post('/test_admin/admin/admin_views/customarticle/', data={ + response = self.client.post(reverse('admin:admin_views_customarticle_changelist'), data={ 'index': 0, 'action': ['delete_selected'], '_selected_action': ['1'], }) self.assertTemplateUsed(response, 'custom_admin/delete_selected_confirmation.html') - response = self.client.get('/test_admin/admin/admin_views/customarticle/%d/history/' % article_pk) + response = self.client.get(reverse('admin:admin_views_customarticle_history', args=(article_pk,))) self.assertTemplateUsed(response, 'custom_admin/object_history.html') - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) def test_delete_view(self): """Delete view should restrict access and actually delete items.""" - login_url = reverse('admin:login') + '?next=/test_admin/admin/' + login_url = '%s?next=%s' % (reverse('admin:login'), reverse('admin:index')) delete_dict = {'post': 'yes'} + delete_url = reverse('admin:admin_views_article_delete', args=(1,)) # add user should not be able to delete articles - self.client.get('/test_admin/admin/') + self.client.get(self.index_url) self.client.post(login_url, self.adduser_login) - response = self.client.get('/test_admin/admin/admin_views/article/1/delete/') + response = self.client.get(delete_url) self.assertEqual(response.status_code, 403) - post = self.client.post('/test_admin/admin/admin_views/article/1/delete/', delete_dict) + post = self.client.post(delete_url, delete_dict) self.assertEqual(post.status_code, 403) self.assertEqual(Article.objects.all().count(), 3) - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) # Delete user can delete - self.client.get('/test_admin/admin/') + self.client.get(self.index_url) self.client.post(login_url, self.deleteuser_login) - response = self.client.get('/test_admin/admin/admin_views/section/1/delete/') + response = self.client.get(reverse('admin:admin_views_section_delete', args=(1,))) self.assertContains(response, "

Summary

") self.assertContains(response, "

Articles: 3

") # test response contains link to related Article self.assertContains(response, "admin_views/article/1/") - response = self.client.get('/test_admin/admin/admin_views/article/1/delete/') + response = self.client.get(delete_url) self.assertContains(response, "admin_views/article/1/") self.assertContains(response, "

Summary

") self.assertContains(response, "

Articles: 1

") self.assertEqual(response.status_code, 200) - post = self.client.post('/test_admin/admin/admin_views/article/1/delete/', delete_dict) - self.assertRedirects(post, '/test_admin/admin/') + post = self.client.post(delete_url, delete_dict) + self.assertRedirects(post, self.index_url) self.assertEqual(Article.objects.all().count(), 2) self.assertEqual(len(mail.outbox), 1) self.assertEqual(mail.outbox[0].subject, 'Greetings from a deleted object') article_ct = ContentType.objects.get_for_model(Article) logged = LogEntry.objects.get(content_type=article_ct, action_flag=DELETION) self.assertEqual(logged.object_id, '1') - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) def test_disabled_permissions_when_logged_in(self): self.client.login(username='super', password='secret') @@ -1630,11 +1635,11 @@ def test_disabled_permissions_when_logged_in(self): superuser.is_active = False superuser.save() - response = self.client.get('/test_admin/admin/', follow=True) + response = self.client.get(self.index_url, follow=True) self.assertContains(response, 'id="login-form"') self.assertNotContains(response, 'Log out') - response = self.client.get('/test_admin/admin/secure-view/', follow=True) + response = self.client.get(reverse('secure_view'), follow=True) self.assertContains(response, 'id="login-form"') def test_disabled_staff_permissions_when_logged_in(self): @@ -1643,11 +1648,11 @@ def test_disabled_staff_permissions_when_logged_in(self): superuser.is_staff = False superuser.save() - response = self.client.get('/test_admin/admin/', follow=True) + response = self.client.get(self.index_url, follow=True) self.assertContains(response, 'id="login-form"') self.assertNotContains(response, 'Log out') - response = self.client.get('/test_admin/admin/secure-view/', follow=True) + response = self.client.get(reverse('secure_view'), follow=True) self.assertContains(response, 'id="login-form"') def test_app_index_fail_early(self): @@ -1655,7 +1660,7 @@ def test_app_index_fail_early(self): If a user has no module perms, avoid iterating over all the modeladmins in the registry. """ - login_url = reverse('admin:login') + '?next=/test_admin/admin/' + login_url = '%s?next=%s' % (reverse('admin:login'), reverse('admin:index')) opts = Article._meta change_user = User.objects.get(username='changeuser') permission = get_perm(Article, get_permission_codename('change', opts)) @@ -1664,12 +1669,12 @@ def test_app_index_fail_early(self): # the user has no module permissions, because this module doesn't exist change_user.user_permissions.remove(permission) - response = self.client.get('/test_admin/admin/admin_views/') + response = self.client.get(reverse('admin:app_list', args=('admin_views',))) self.assertEqual(response.status_code, 403) # the user now has module permissions change_user.user_permissions.add(permission) - response = self.client.get('/test_admin/admin/admin_views/') + response = self.client.get(reverse('admin:app_list', args=('admin_views',))) self.assertEqual(response.status_code, 200) def test_shortcut_view_only_available_to_staff(self): @@ -1678,7 +1683,7 @@ def test_shortcut_view_only_available_to_staff(self): """ model_ctype = ContentType.objects.get_for_model(ModelWithStringPrimaryKey) obj = ModelWithStringPrimaryKey.objects.create(string_pk='foo') - shortcut_url = "/test_admin/admin/r/%s/%s/" % (model_ctype.pk, obj.pk) + shortcut_url = reverse('admin:view_on_site', args=(model_ctype.pk, obj.pk)) # Not logged in: we should see the login page. response = self.client.get(shortcut_url, follow=True) @@ -1698,31 +1703,31 @@ def test_has_module_permission(self): have any permission for that module (add, change, or delete), so that the module is displayed on the admin index page. """ - login_url = reverse('admin:login') + '?next=/test_admin/admin/' + login_url = '%s?next=%s' % (reverse('admin:login'), reverse('admin:index')) self.client.post(login_url, self.super_login) - response = self.client.get('/test_admin/admin/') + response = self.client.get(self.index_url) self.assertContains(response, 'admin_views') self.assertContains(response, 'Articles') - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) self.client.post(login_url, self.adduser_login) - response = self.client.get('/test_admin/admin/') + response = self.client.get(self.index_url) self.assertContains(response, 'admin_views') self.assertContains(response, 'Articles') - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) self.client.post(login_url, self.changeuser_login) - response = self.client.get('/test_admin/admin/') + response = self.client.get(self.index_url) self.assertContains(response, 'admin_views') self.assertContains(response, 'Articles') - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) self.client.post(login_url, self.deleteuser_login) - response = self.client.get('/test_admin/admin/') + response = self.client.get(self.index_url) self.assertContains(response, 'admin_views') self.assertContains(response, 'Articles') - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) def test_overriding_has_module_permission(self): """ @@ -1730,31 +1735,32 @@ def test_overriding_has_module_permission(self): In this case, it always returns False, so the module should not be displayed on the admin index page for any users. """ - login_url = reverse('admin:login') + '?next=/test_admin/admin7/' + login_url = '%s?next=%s' % (reverse('admin7:login'), reverse('admin7:index')) + index_url = reverse('admin7:index') self.client.post(login_url, self.super_login) - response = self.client.get('/test_admin/admin7/') + response = self.client.get(index_url) self.assertNotContains(response, 'admin_views') self.assertNotContains(response, 'Articles') - self.client.get('/test_admin/admin7/logout/') + self.client.get(reverse('admin7:logout')) self.client.post(login_url, self.adduser_login) - response = self.client.get('/test_admin/admin7/') + response = self.client.get(index_url) self.assertNotContains(response, 'admin_views') self.assertNotContains(response, 'Articles') - self.client.get('/test_admin/admin7/logout/') + self.client.get(reverse('admin7:logout')) self.client.post(login_url, self.changeuser_login) - response = self.client.get('/test_admin/admin7/') + response = self.client.get(index_url) self.assertNotContains(response, 'admin_views') self.assertNotContains(response, 'Articles') - self.client.get('/test_admin/admin7/logout/') + self.client.get(reverse('admin7:logout')) self.client.post(login_url, self.deleteuser_login) - response = self.client.get('/test_admin/admin7/') + response = self.client.get(index_url) self.assertNotContains(response, 'admin_views') self.assertNotContains(response, 'Articles') - self.client.get('/test_admin/admin7/logout/') + self.client.get(reverse('admin7:logout')) @override_settings(PASSWORD_HASHERS=('django.contrib.auth.hashers.SHA1PasswordHasher',), @@ -1773,19 +1779,19 @@ def setUp(self): # login POST dict self.changeuser_login = { - REDIRECT_FIELD_NAME: '/test_admin/admin/', + REDIRECT_FIELD_NAME: reverse('admin:index'), 'username': 'changeuser', 'password': 'secret', } def test_no_standard_modeladmin_urls(self): """Admin index views don't break when user's ModelAdmin removes standard urls""" - self.client.get('/test_admin/admin/') + self.client.get(reverse('admin:index')) r = self.client.post(reverse('admin:login'), self.changeuser_login) - r = self.client.get('/test_admin/admin/') + r = self.client.get(reverse('admin:index')) # we shouldn't get a 500 error caused by a NoReverseMatch self.assertEqual(r.status_code, 200) - self.client.get('/test_admin/admin/logout/') + self.client.get(reverse('admin:logout')) @skipUnlessDBFeature('can_defer_constraint_checks') @@ -1802,8 +1808,12 @@ def test_nesting(self): Objects should be nested to display the relationships that cause them to be scheduled for deletion. """ - pattern = re.compile(br"""

Plot: World Domination\s*

Plot details: almost finished""") - response = self.client.get('/test_admin/admin/admin_views/villain/%s/delete/' % quote(1)) + pattern = re.compile(force_bytes( + r'
Plot: World Domination\s*
- Plot details: almost finished' % ( + reverse('admin:admin_views_plot_change', args=(1,)), + reverse('admin:admin_views_plotdetails_change', args=(1,))) + )) + response = self.client.get(reverse('admin:admin_views_villain_delete', args=(1,))) six.assertRegex(self, response.content, pattern) def test_cyclic(self): @@ -1812,9 +1822,13 @@ def test_cyclic(self): listed once. """ - one = """
- Cyclic one: I am recursive""" - two = """
- Cyclic two: I am recursive too""" - response = self.client.get('/test_admin/admin/admin_views/cyclicone/%s/delete/' % quote(1)) + one = '
- Cyclic one: I am recursive' % ( + reverse('admin:admin_views_cyclicone_change', args=(1,)), + ) + two = '
- Cyclic two: I am recursive too' % ( + reverse('admin:admin_views_cyclictwo_change', args=(1,)), + ) + response = self.client.get(reverse('admin:admin_views_cyclicone_delete', args=(1,))) self.assertContains(response, one, 1) self.assertContains(response, two, 1) @@ -1828,7 +1842,7 @@ def test_perms_needed(self): self.assertTrue(self.client.login(username='deleteuser', password='secret')) - response = self.client.get('/test_admin/admin/admin_views/plot/%s/delete/' % quote(1)) + response = self.client.get(reverse('admin:admin_views_plot_delete', args=(1,))) self.assertContains(response, "your account doesn't have permission to delete the following types of objects") self.assertContains(response, "
- plot details
- Answer: Because.
- Answer: Yes.
- Answer: Because.
- Answer: Yes.
- Secret hideout: underground bunker""" - response = self.client.get('/test_admin/admin/admin_views/villain/%s/delete/' % quote(1)) + response = self.client.get(reverse('admin:admin_views_villain_delete', args=(1,))) self.assertContains(response, should_contain, 1) def test_multiple_fkeys_to_same_model(self): @@ -1854,10 +1874,12 @@ def test_multiple_fkeys_to_same_model(self): objects to delete. """ - should_contain = """
- Plot: World Domination""" - response = self.client.get('/test_admin/admin/admin_views/villain/%s/delete/' % quote(1)) + should_contain = '
- Plot: World Domination' % reverse( + 'admin:admin_views_plot_change', args=(1,) + ) + response = self.client.get(reverse('admin:admin_views_villain_delete', args=(1,))) self.assertContains(response, should_contain) - response = self.client.get('/test_admin/admin/admin_views/villain/%s/delete/' % quote(2)) + response = self.client.get(reverse('admin:admin_views_villain_delete', args=(2,))) self.assertContains(response, should_contain) def test_multiple_fkeys_to_same_instance(self): @@ -1867,8 +1889,10 @@ def test_multiple_fkeys_to_same_instance(self): once. """ - should_contain = """
- Plot: World Peace
- Plot: World Peace
- Villain: Bob""", - """
- Super villain: Bob""", - """
- Secret hideout: floating castle""", - """
- Super secret hideout: super floating castle!""" + '
- Villain: Bob' % reverse('admin:admin_views_villain_change', args=(3,)), + '
- Super villain: Bob' % reverse('admin:admin_views_supervillain_change', args=(3,)), + '
- Secret hideout: floating castle', + '
- Super secret hideout: super floating castle!', ] - response = self.client.get('/test_admin/admin/admin_views/villain/%s/delete/' % quote(3)) + response = self.client.get(reverse('admin:admin_views_villain_delete', args=(3,))) for should in should_contain: self.assertContains(response, should, 1) - response = self.client.get('/test_admin/admin/admin_views/supervillain/%s/delete/' % quote(3)) + response = self.client.get(reverse('admin:admin_views_supervillain_delete', args=(3,))) for should in should_contain: self.assertContains(response, should, 1) @@ -1899,8 +1923,9 @@ def test_generic_relations(self): """ plot = Plot.objects.get(pk=3) tag = FunkyTag.objects.create(content_object=plot, name='hott') - should_contain = """
- Funky tag: hott""" % tag.id - response = self.client.get('/test_admin/admin/admin_views/plot/%s/delete/' % quote(3)) + should_contain = '
- Funky tag: hott' % reverse( + 'admin:admin_views_funkytag_change', args=(tag.id,)) + response = self.client.get(reverse('admin:admin_views_plot_delete', args=(3,))) self.assertContains(response, should_contain) @@ -1915,7 +1940,7 @@ def setUp(self): def test_generic_content_object_in_list_display(self): plot = Plot.objects.get(pk=3) FunkyTag.objects.create(content_object=plot, name='hott') - response = self.client.get('/test_admin/admin/admin_views/funkytag/') + response = self.client.get(reverse('admin:admin_views_funkytag_changelist')) self.assertContains(response, "%s" % plot) @@ -1939,36 +1964,38 @@ def test_get_history_view(self): key should work. Refs #12349, #18550. """ - response = self.client.get('/test_admin/admin/admin_views/modelwithstringprimarykey/%s/history/' % quote(self.pk)) + response = self.client.get(reverse('admin:admin_views_modelwithstringprimarykey_history', args=(self.pk,))) self.assertContains(response, escape(self.pk)) self.assertContains(response, 'Changed something') self.assertEqual(response.status_code, 200) def test_get_change_view(self): "Retrieving the object using urlencoded form of primary key should work" - response = self.client.get('/test_admin/admin/admin_views/modelwithstringprimarykey/%s/' % quote(self.pk)) + response = self.client.get(reverse('admin:admin_views_modelwithstringprimarykey_change', args=(self.pk,))) self.assertContains(response, escape(self.pk)) self.assertEqual(response.status_code, 200) def test_changelist_to_changeform_link(self): "Link to the changeform of the object in changelist should use reverse() and be quoted -- #18072" - prefix = '/test_admin/admin/admin_views/modelwithstringprimarykey/' - response = self.client.get(prefix) + response = self.client.get(reverse('admin:admin_views_modelwithstringprimarykey_changelist')) # this URL now comes through reverse(), thus url quoting and iri_to_uri encoding pk_final_url = escape(iri_to_uri(quote(self.pk))) - should_contain = """%s""" % (prefix, pk_final_url, escape(self.pk)) + change_url = reverse( + 'admin:admin_views_modelwithstringprimarykey_change', args=('__fk__',) + ).replace('__fk__', pk_final_url) + should_contain = '%s' % (change_url, escape(self.pk)) self.assertContains(response, should_contain) def test_recentactions_link(self): "The link from the recent actions list referring to the changeform of the object should be quoted" - response = self.client.get('/test_admin/admin/') + response = self.client.get(reverse('admin:index')) link = reverse('admin:admin_views_modelwithstringprimarykey_change', args=(quote(self.pk),)) should_contain = """%s""" % (escape(link), escape(self.pk)) self.assertContains(response, should_contain) def test_recentactions_without_content_type(self): "If a LogEntry is missing content_type it will not display it in span tag under the hyperlink." - response = self.client.get('/test_admin/admin/') + response = self.client.get(reverse('admin:index')) link = reverse('admin:admin_views_modelwithstringprimarykey_change', args=(quote(self.pk),)) should_contain = """%s""" % (escape(link), escape(self.pk)) self.assertContains(response, should_contain) @@ -1982,7 +2009,7 @@ def test_recentactions_without_content_type(self): logentry.save() counted_presence_before = response.content.count(force_bytes(should_contain)) - response = self.client.get('/test_admin/admin/') + response = self.client.get(reverse('admin:index')) counted_presence_after = response.content.count(force_bytes(should_contain)) self.assertEqual(counted_presence_before - 1, counted_presence_after) @@ -1991,25 +2018,31 @@ def test_logentry_get_admin_url(self): "LogEntry.get_admin_url returns a URL to edit the entry's object or None for non-existent (possibly deleted) models" log_entry_model = "modelwithstringprimarykey" # capitalized in Recent Actions logentry = LogEntry.objects.get(content_type__model__iexact=log_entry_model) - model = "modelwithstringprimarykey" - desired_admin_url = "/test_admin/admin/admin_views/%s/%s/" % (model, iri_to_uri(quote(self.pk))) + desired_admin_url = reverse('admin:admin_views_modelwithstringprimarykey_change', args=(quote(self.pk),)) + self.assertEqual(logentry.get_admin_url(), desired_admin_url) + self.assertIn(iri_to_uri(quote(self.pk)), logentry.get_admin_url()) logentry.content_type.model = "non-existent" self.assertEqual(logentry.get_admin_url(), None) def test_deleteconfirmation_link(self): "The link from the delete confirmation page referring back to the changeform of the object should be quoted" - response = self.client.get('/test_admin/admin/admin_views/modelwithstringprimarykey/%s/delete/' % quote(self.pk)) + response = self.client.get(reverse('admin:admin_views_modelwithstringprimarykey_delete', args=(quote(self.pk),))) # this URL now comes through reverse(), thus url quoting and iri_to_uri encoding - should_contain = """/%s/">%s""" % (escape(iri_to_uri(quote(self.pk))), escape(self.pk)) + change_url = reverse( + 'admin:admin_views_modelwithstringprimarykey_change', args=('__fk__',) + ).replace('__fk__', escape(iri_to_uri(quote(self.pk)))) + should_contain = '%s' % (change_url, escape(self.pk)) self.assertContains(response, should_contain) def test_url_conflicts_with_add(self): "A model with a primary key that ends with add should be visible" add_model = ModelWithStringPrimaryKey(pk="i have something to add") add_model.save() - response = self.client.get('/test_admin/admin/admin_views/modelwithstringprimarykey/%s/' % quote(add_model.pk)) + response = self.client.get( + reverse('admin:admin_views_modelwithstringprimarykey_change', args=(quote(add_model.pk),)) + ) should_contain = """
  Change model with string primary key
  """ self.assertContains(response, should_contain) @@ -2017,7 +2050,9 @@ def test_url_conflicts_with_delete(self): "A model with a primary key that ends with delete should be visible" delete_model = ModelWithStringPrimaryKey(pk="delete") delete_model.save() - response = self.client.get('/test_admin/admin/admin_views/modelwithstringprimarykey/%s/' % quote(delete_model.pk)) + response = self.client.get( + reverse('admin:admin_views_modelwithstringprimarykey_change', args=(quote(delete_model.pk),)) + ) should_contain = """
  Change model with string primary key
  """ self.assertContains(response, should_contain) @@ -2025,7 +2060,9 @@ def test_url_conflicts_with_history(self): "A model with a primary key that ends with history should be visible" history_model = ModelWithStringPrimaryKey(pk="history") history_model.save() - response = self.client.get('/test_admin/admin/admin_views/modelwithstringprimarykey/%s/' % quote(history_model.pk)) + response = self.client.get( + reverse('admin:admin_views_modelwithstringprimarykey_change', args=(quote(history_model.pk),)) + ) should_contain = """
  Change model with string primary key
  """ self.assertContains(response, should_contain) @@ -2033,7 +2070,9 @@ def test_shortcut_view_with_escaping(self): "'View on site should' work properly with char fields" model = ModelWithStringPrimaryKey(pk='abc_123') model.save() - response = self.client.get('/test_admin/admin/admin_views/modelwithstringprimarykey/%s/' % quote(model.pk)) + response = self.client.get( + reverse('admin:admin_views_modelwithstringprimarykey_change', args=(quote(model.pk),)) + ) should_contain = '/%s/" class="viewsitelink">' % model.pk self.assertContains(response, should_contain) @@ -2057,7 +2096,7 @@ def test_redirect_on_add_view_continue_button(self): like slash or underscore, these characters must be escaped (see #22266) """ response = self.client.post( - '/test_admin/admin/admin_views/modelwithstringprimarykey/add/', + reverse('admin:admin_views_modelwithstringprimarykey_add'), { 'string_pk': '123/history', "_continue": "1", # Save and continue editing @@ -2065,13 +2104,7 @@ def test_redirect_on_add_view_continue_button(self): ) self.assertEqual(response.status_code, 302) # temporary redirect - self.assertEqual( - response['location'], - ( - 'http://testserver/test_admin/admin/admin_views/' - 'modelwithstringprimarykey/123_2Fhistory/' # PK is quoted - ) - ) + self.assertIn('/123_2Fhistory/', response['location']) # PK is quoted @override_settings(PASSWORD_HASHERS=('django.contrib.auth.hashers.SHA1PasswordHasher',), @@ -2086,7 +2119,7 @@ def test_secure_view_shows_login_if_not_logged_in(self): """ Ensure that we see the admin login form. """ - secure_url = '/test_admin/admin/secure-view/' + secure_url = reverse('secure_view') response = self.client.get(secure_url) self.assertRedirects(response, '%s?next=%s' % (reverse('admin:login'), secure_url)) response = self.client.get(secure_url, follow=True) @@ -2132,7 +2165,7 @@ def test_unicode_edit(self): "chapter_set-5-content": "", } - response = self.client.post('/test_admin/admin/admin_views/book/1/', post_data) + response = self.client.post(reverse('admin:admin_views_book_change', args=(1,)), post_data) self.assertEqual(response.status_code, 302) # redirect somewhere def test_unicode_delete(self): @@ -2140,10 +2173,11 @@ def test_unicode_delete(self): Ensure that the delete_view handles non-ASCII characters """ delete_dict = {'post': 'yes'} - response = self.client.get('/test_admin/admin/admin_views/book/1/delete/') + delete_url = reverse('admin:admin_views_book_delete', args=(1,)) + response = self.client.get(delete_url) self.assertEqual(response.status_code, 200) - response = self.client.post('/test_admin/admin/admin_views/book/1/delete/', delete_dict) - self.assertRedirects(response, '/test_admin/admin/admin_views/book/') + response = self.client.post(delete_url, delete_dict) + self.assertRedirects(response, reverse('admin:admin_views_book_changelist')) @override_settings(PASSWORD_HASHERS=('django.contrib.auth.hashers.SHA1PasswordHasher',), @@ -2157,21 +2191,21 @@ def setUp(self): def test_inheritance(self): Podcast.objects.create(name="This Week in Django", release_date=datetime.date.today()) - response = self.client.get('/test_admin/admin/admin_views/podcast/') + response = self.client.get(reverse('admin:admin_views_podcast_changelist')) self.assertEqual(response.status_code, 200) def test_inheritance_2(self): Vodcast.objects.create(name="This Week in Django", released=True) - response = self.client.get('/test_admin/admin/admin_views/vodcast/') + response = self.client.get(reverse('admin:admin_views_vodcast_changelist')) self.assertEqual(response.status_code, 200) def test_custom_pk(self): Language.objects.create(iso='en', name='English', english_name='English') - response = self.client.get('/test_admin/admin/admin_views/language/') + response = self.client.get(reverse('admin:admin_views_language_changelist')) self.assertEqual(response.status_code, 200) def test_changelist_input_html(self): - response = self.client.get('/test_admin/admin/admin_views/person/') + response = self.client.get(reverse('admin:admin_views_person_changelist')) # 2 inputs per object(the field and the hidden id field) = 6 # 4 management hidden fields = 4 # 4 action inputs (3 regular checkboxes, 1 checkbox to select all) @@ -2204,7 +2238,7 @@ def test_post_messages(self): "_save": "Save", } - response = self.client.post('/test_admin/admin/admin_views/person/', + response = self.client.post(reverse('admin:admin_views_person_changelist'), data, follow=True) self.assertEqual(len(response.context['messages']), 1) @@ -2226,7 +2260,7 @@ def test_post_submission(self): "_save": "Save", } - self.client.post('/test_admin/admin/admin_views/person/', data) + self.client.post(reverse('admin:admin_views_person_changelist'), data) self.assertEqual(Person.objects.get(name="John Mauchly").alive, False) self.assertEqual(Person.objects.get(name="Grace Hopper").gender, 2) @@ -2247,7 +2281,7 @@ def test_post_submission(self): "_save": "Save", } - self.client.post('/test_admin/admin/admin_views/person/?gender__exact=1', data) + self.client.post(reverse('admin:admin_views_person_changelist') + '?gender__exact=1', data) self.assertEqual(Person.objects.get(name="John Mauchly").alive, True) @@ -2262,7 +2296,7 @@ def test_post_submission(self): "_save": "Save", } - self.client.post('/test_admin/admin/admin_views/person/?q=john', data) + self.client.post(reverse('admin:admin_views_person_changelist') + '?q=john', data) self.assertEqual(Person.objects.get(name="John Mauchly").alive, False) @@ -2297,8 +2331,14 @@ def test_non_field_errors(self): "_save": "Save", } - response = self.client.post('/test_admin/admin/admin_views/fooddelivery/', data) - self.assertContains(response, '
  - Food delivery with this Driver and Restaurant already exists.
  ', 1, html=True) + response = self.client.post(reverse('admin:admin_views_fooddelivery_changelist'), data) + self.assertContains( + response, + '
  - Food delivery ' + 'with this Driver and Restaurant already exists.
  ', + 1, + html=True + ) data = { "form-TOTAL_FORMS": "3", @@ -2324,8 +2364,14 @@ def test_non_field_errors(self): "_save": "Save", } - response = self.client.post('/test_admin/admin/admin_views/fooddelivery/', data) - self.assertContains(response, '
  - Food delivery with this Driver and Restaurant already exists.
  ', 2, html=True) + response = self.client.post(reverse('admin:admin_views_fooddelivery_changelist'), data) + self.assertContains( + response, + '
  - Food delivery ' + 'with this Driver and Restaurant already exists.
  ', + 2, + html=True + ) def test_non_form_errors(self): # test if non-form errors are handled; ticket #12716 @@ -2342,7 +2388,7 @@ def test_non_form_errors(self): # and not an action "Go". "_save": "Save", } - response = self.client.post('/test_admin/admin/admin_views/person/', data) + response = self.client.post(reverse('admin:admin_views_person_changelist'), data) self.assertContains(response, "Grace is not a Zombie") def test_non_form_errors_is_errorlist(self): @@ -2358,7 +2404,7 @@ def test_non_form_errors_is_errorlist(self): "_save": "Save", } - response = self.client.post('/test_admin/admin/admin_views/person/', data) + response = self.client.post(reverse('admin:admin_views_person_changelist'), data) non_form_errors = response.context['cl'].formset.non_form_errors() self.assertIsInstance(non_form_errors, ErrorList) self.assertEqual(str(non_form_errors), str(ErrorList(["Grace is not a Zombie"]))) @@ -2397,7 +2443,7 @@ def test_list_editable_ordering(self): # and not an action "Go". "_save": "Save", } - response = self.client.post('/test_admin/admin/admin_views/category/', data) + response = self.client.post(reverse('admin:admin_views_category_changelist'), data) # Successful post will redirect self.assertEqual(response.status_code, 302) @@ -2415,11 +2461,11 @@ def test_list_editable_pagination(self): UnorderedObject.objects.create(id=1, name='Unordered object #1') UnorderedObject.objects.create(id=2, name='Unordered object #2') UnorderedObject.objects.create(id=3, name='Unordered object #3') - response = self.client.get('/test_admin/admin/admin_views/unorderedobject/') + response = self.client.get(reverse('admin:admin_views_unorderedobject_changelist')) self.assertContains(response, 'Unordered object #3') self.assertContains(response, 'Unordered object #2') self.assertNotContains(response, 'Unordered object #1') - response = self.client.get('/test_admin/admin/admin_views/unorderedobject/?p=1') + response = self.client.get(reverse('admin:admin_views_unorderedobject_changelist') + '?p=1') self.assertNotContains(response, 'Unordered object #3') self.assertNotContains(response, 'Unordered object #2') self.assertContains(response, 'Unordered object #1') @@ -2446,7 +2492,7 @@ def test_list_editable_action_submit(self): "_selected_action": ['3'], "action": ['', 'delete_selected'], } - self.client.post('/test_admin/admin/admin_views/person/', data) + self.client.post(reverse('admin:admin_views_person_changelist'), data) self.assertEqual(Person.objects.get(name="John Mauchly").alive, True) self.assertEqual(Person.objects.get(name="Grace Hopper").gender, 1) @@ -2473,7 +2519,7 @@ def test_list_editable_action_choices(self): "_selected_action": ['1'], "action": ['', 'delete_selected'], } - self.client.post('/test_admin/admin/admin_views/person/', data) + self.client.post(reverse('admin:admin_views_person_changelist'), data) self.assertEqual(Person.objects.get(name="John Mauchly").alive, False) self.assertEqual(Person.objects.get(name="Grace Hopper").gender, 2) @@ -2482,9 +2528,9 @@ def test_list_editable_popup(self): """ Fields should not be list-editable in popups. """ - response = self.client.get('/test_admin/admin/admin_views/person/') + response = self.client.get(reverse('admin:admin_views_person_changelist')) self.assertNotEqual(response.context['cl'].list_editable, ()) - response = self.client.get('/test_admin/admin/admin_views/person/?%s' % IS_POPUP_VAR) + response = self.client.get(reverse('admin:admin_views_person_changelist') + '?%s' % IS_POPUP_VAR) self.assertEqual(response.context['cl'].list_editable, ()) def test_pk_hidden_fields(self): @@ -2496,7 +2542,7 @@ def test_pk_hidden_fields(self): """ story1 = Story.objects.create(title='The adventures of Guido', content='Once upon a time in Djangoland...') story2 = Story.objects.create(title='Crouching Tiger, Hidden Python', content='The Python was sneaking into...') - response = self.client.get('/test_admin/admin/admin_views/story/') + response = self.client.get(reverse('admin:admin_views_story_changelist')) self.assertContains(response, 'id="id_form-0-id"', 1) # Only one hidden field, in a separate place than the table. self.assertContains(response, 'id="id_form-1-id"', 1) self.assertContains(response, '
  \n\n
  ' % (story2.id, story1.id), html=True) @@ -2512,7 +2558,7 @@ def test_pk_hidden_fields_with_list_display_links(self): story2 = OtherStory.objects.create(title='Crouching Tiger, Hidden Python', content='The Python was sneaking into...') link1 = reverse('admin:admin_views_otherstory_change', args=(story1.pk,)) link2 = reverse('admin:admin_views_otherstory_change', args=(story2.pk,)) - response = self.client.get('/test_admin/admin/admin_views/otherstory/') + response = self.client.get(reverse('admin:admin_views_otherstory_changelist')) self.assertContains(response, 'id="id_form-0-id"', 1) # Only one hidden field, in a separate place than the table. self.assertContains(response, 'id="id_form-1-id"', 1) self.assertContains(response, '
  \n\n
  ' % (story2.id, story1.id), html=True) @@ -2531,34 +2577,35 @@ def setUp(self): def test_search_on_sibling_models(self): "Check that a search that mentions sibling models" - response = self.client.get('/test_admin/admin/admin_views/recommendation/?q=bar') + response = self.client.get(reverse('admin:admin_views_recommendation_changelist') + '?q=bar') # confirm the search returned 1 object self.assertContains(response, "\n1 recommendation\n") def test_with_fk_to_field(self): - """Ensure that the to_field GET parameter is preserved when a search + """ + Ensure that the to_field GET parameter is preserved when a search is performed. Refs #10918. """ - response = self.client.get('/test_admin/admin/auth/user/?q=joe&%s=id' % TO_FIELD_VAR) + response = self.client.get(reverse('admin:auth_user_changelist') + '?q=joe&%s=id' % TO_FIELD_VAR) self.assertContains(response, "\n1 user\n") self.assertContains(response, '' % TO_FIELD_VAR, html=True) def test_exact_matches(self): - response = self.client.get('/test_admin/admin/admin_views/recommendation/?q=bar') + response = self.client.get(reverse('admin:admin_views_recommendation_changelist') + '?q=bar') # confirm the search returned one object self.assertContains(response, "\n1 recommendation\n") - response = self.client.get('/test_admin/admin/admin_views/recommendation/?q=ba') + response = self.client.get(reverse('admin:admin_views_recommendation_changelist') + '?q=ba') # confirm the search returned zero objects self.assertContains(response, "\n0 recommendations\n") def test_beginning_matches(self): - response = self.client.get('/test_admin/admin/admin_views/person/?q=Gui') + response = self.client.get(reverse('admin:admin_views_person_changelist') + '?q=Gui') # confirm the search returned one object self.assertContains(response, "\n1 person\n") self.assertContains(response, "Guido") - response = self.client.get('/test_admin/admin/admin_views/person/?q=uido') + response = self.client.get(reverse('admin:admin_views_person_changelist') + '?q=uido') # confirm the search returned zero objects self.assertContains(response, "\n0 persons\n") self.assertNotContains(response, "Guido") @@ -2567,12 +2614,12 @@ def test_pluggable_search(self): PluggableSearchPerson.objects.create(name="Bob", age=10) PluggableSearchPerson.objects.create(name="Amy", age=20) - response = self.client.get('/test_admin/admin/admin_views/pluggablesearchperson/?q=Bob') + response = self.client.get(reverse('admin:admin_views_pluggablesearchperson_changelist') + '?q=Bob') # confirm the search returned one object self.assertContains(response, "\n1 pluggable search person\n") self.assertContains(response, "Bob") - response = self.client.get('/test_admin/admin/admin_views/pluggablesearchperson/?q=20') + response = self.client.get(reverse('admin:admin_views_pluggablesearchperson_changelist') + '?q=20') # confirm the search returned one object self.assertContains(response, "\n1 pluggable search person\n") self.assertContains(response, "Amy") @@ -2585,7 +2632,7 @@ def test_reset_link(self): # + 1 for filtered result + 1 for filtered count # + 1 for total count with self.assertNumQueries(5): - response = self.client.get('/test_admin/admin/admin_views/person/?q=Gui') + response = self.client.get(reverse('admin:admin_views_person_changelist') + '?q=Gui') self.assertContains(response, """1 result (3 total)""", html=True) @@ -2598,7 +2645,7 @@ def test_no_total_count(self): # 1 query for session + 1 for fetching user # + 1 for filtered result + 1 for filtered count with self.assertNumQueries(4): - response = self.client.get('/test_admin/admin/admin_views/recommendation/?q=bar') + response = self.client.get(reverse('admin:admin_views_recommendation_changelist') + '?q=bar') self.assertContains(response, """1 result (Show all)""", html=True) @@ -2621,7 +2668,7 @@ def test_inline(self): name_re = re.compile(b'name="(.*?)"') # test the add case - response = self.client.get('/test_admin/admin/admin_views/persona/add/') + response = self.client.get(reverse('admin:admin_views_persona_add')) names = name_re.findall(response.content) # make sure we have no duplicate HTML names self.assertEqual(len(names), len(set(names))) @@ -2640,7 +2687,7 @@ def test_inline(self): "accounts-2-0-username": bar_user, } - response = self.client.post('/test_admin/admin/admin_views/persona/add/', post_data) + response = self.client.post(reverse('admin:admin_views_persona_add'), post_data) self.assertEqual(response.status_code, 302) # redirect somewhere self.assertEqual(Persona.objects.count(), 1) self.assertEqual(FooAccount.objects.count(), 1) @@ -2655,7 +2702,7 @@ def test_inline(self): # test the edit case - response = self.client.get('/test_admin/admin/admin_views/persona/%d/' % persona_id) + response = self.client.get(reverse('admin:admin_views_persona_change', args=(persona_id,))) names = name_re.findall(response.content) # make sure we have no duplicate HTML names self.assertEqual(len(names), len(set(names))) @@ -2679,7 +2726,7 @@ def test_inline(self): "accounts-2-0-account_ptr": str(bar_id), "accounts-2-0-persona": str(persona_id), } - response = self.client.post('/test_admin/admin/admin_views/persona/%d/' % persona_id, post_data) + response = self.client.post(reverse('admin:admin_views_persona_change', args=(persona_id,)), post_data) self.assertEqual(response.status_code, 302) self.assertEqual(Persona.objects.count(), 1) self.assertEqual(FooAccount.objects.count(), 1) @@ -2704,7 +2751,7 @@ def test_model_admin_custom_action(self): 'action': 'mail_admin', 'index': 0, } - self.client.post('/test_admin/admin/admin_views/subscriber/', action_data) + self.client.post(reverse('admin:admin_views_subscriber_changelist'), action_data) self.assertEqual(len(mail.outbox), 1) self.assertEqual(mail.outbox[0].subject, 'Greetings from a ModelAdmin action') @@ -2720,14 +2767,14 @@ def test_model_admin_default_delete_action(self): 'action': 'delete_selected', 'post': 'yes', } - confirmation = self.client.post('/test_admin/admin/admin_views/subscriber/', action_data) + confirmation = self.client.post(reverse('admin:admin_views_subscriber_changelist'), action_data) self.assertIsInstance(confirmation, TemplateResponse) self.assertContains(confirmation, "Are you sure you want to delete the selected subscribers?") self.assertContains(confirmation, "
  Summary
  ") self.assertContains(confirmation, "
- Subscribers: 3
- External subscribers: 1
- Answer: Because.
- Answer: Yes.
- Answer: Because.
- Answer: Yes.

Username:	{{ form.username }}
Password:	{{ form.password }}

Built-in filters

Oh Noes!

{% trans "Summary" %}

{{ message }}

Author-book relationships

Change model with string primary key

Change model with string primary key

Change model with string primary key

Summary

Built-in tags