diff options
author | django-bot <ops@djangoproject.com> | 2022-02-03 20:24:19 +0100 |
---|---|---|
committer | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2022-02-07 20:37:05 +0100 |
commit | 9c19aff7c7561e3a82978a272ecdaad40dda5c00 (patch) | |
tree | f0506b668a013d0063e5fba3dbf4863b466713ba /tests/admin_inlines | |
parent | f68fa8b45dfac545cfc4111d4e52804c86db68d3 (diff) | |
download | django-9c19aff7c7561e3a82978a272ecdaad40dda5c00.tar.gz |
Refs #33476 -- Reformatted code with Black.
Diffstat (limited to 'tests/admin_inlines')
-rw-r--r-- | tests/admin_inlines/admin.py | 150 | ||||
-rw-r--r-- | tests/admin_inlines/models.py | 66 | ||||
-rw-r--r-- | tests/admin_inlines/test_templates.py | 24 | ||||
-rw-r--r-- | tests/admin_inlines/tests.py | 1443 | ||||
-rw-r--r-- | tests/admin_inlines/urls.py | 8 |
5 files changed, 1045 insertions, 646 deletions
diff --git a/tests/admin_inlines/admin.py b/tests/admin_inlines/admin.py index 116556db7e..0ec56d71b3 100644 --- a/tests/admin_inlines/admin.py +++ b/tests/admin_inlines/admin.py @@ -4,16 +4,57 @@ from django.core.exceptions import ValidationError from django.db import models from .models import ( - Author, BinaryTree, CapoFamiglia, Chapter, Child, ChildModel1, ChildModel2, - Class, Consigliere, Course, CourseProxy, CourseProxy1, CourseProxy2, - EditablePKBook, ExtraTerrestrial, Fashionista, FootNote, Holder, Holder2, - Holder3, Holder4, Holder5, Inner, Inner2, Inner3, Inner4Stacked, - Inner4Tabular, Inner5Stacked, Inner5Tabular, NonAutoPKBook, - NonAutoPKBookChild, Novel, NovelReadonlyChapter, OutfitItem, - ParentModelWithCustomPk, Person, Poll, Profile, ProfileCollection, - Question, ReadOnlyInline, ShoppingWeakness, ShowInlineChild, - ShowInlineParent, Sighting, SomeChildModel, SomeParentModel, SottoCapo, - Teacher, Title, TitleCollection, + Author, + BinaryTree, + CapoFamiglia, + Chapter, + Child, + ChildModel1, + ChildModel2, + Class, + Consigliere, + Course, + CourseProxy, + CourseProxy1, + CourseProxy2, + EditablePKBook, + ExtraTerrestrial, + Fashionista, + FootNote, + Holder, + Holder2, + Holder3, + Holder4, + Holder5, + Inner, + Inner2, + Inner3, + Inner4Stacked, + Inner4Tabular, + Inner5Stacked, + Inner5Tabular, + NonAutoPKBook, + NonAutoPKBookChild, + Novel, + NovelReadonlyChapter, + OutfitItem, + ParentModelWithCustomPk, + Person, + Poll, + Profile, + ProfileCollection, + Question, + ReadOnlyInline, + ShoppingWeakness, + ShowInlineChild, + ShowInlineParent, + Sighting, + SomeChildModel, + SomeParentModel, + SottoCapo, + Teacher, + Title, + TitleCollection, ) site = admin.AdminSite(name="admin") @@ -25,17 +66,17 @@ class BookInline(admin.TabularInline): class NonAutoPKBookTabularInline(admin.TabularInline): model = NonAutoPKBook - classes = ('collapse',) + classes = ("collapse",) class NonAutoPKBookChildTabularInline(admin.TabularInline): model = NonAutoPKBookChild - classes = ('collapse',) + classes = ("collapse",) class NonAutoPKBookStackedInline(admin.StackedInline): model = NonAutoPKBook - classes = ('collapse',) + classes = ("collapse",) class EditablePKBookTabularInline(admin.TabularInline): @@ -48,8 +89,11 @@ class EditablePKBookStackedInline(admin.StackedInline): class AuthorAdmin(admin.ModelAdmin): inlines = [ - BookInline, NonAutoPKBookTabularInline, NonAutoPKBookStackedInline, - EditablePKBookTabularInline, EditablePKBookStackedInline, + BookInline, + NonAutoPKBookTabularInline, + NonAutoPKBookStackedInline, + EditablePKBookTabularInline, + EditablePKBookStackedInline, NonAutoPKBookChildTabularInline, ] @@ -57,25 +101,24 @@ class AuthorAdmin(admin.ModelAdmin): class InnerInline(admin.StackedInline): model = Inner can_delete = False - readonly_fields = ('readonly',) # For bug #13174 tests. + readonly_fields = ("readonly",) # For bug #13174 tests. class HolderAdmin(admin.ModelAdmin): - class Media: - js = ('my_awesome_admin_scripts.js',) + js = ("my_awesome_admin_scripts.js",) class ReadOnlyInlineInline(admin.TabularInline): model = ReadOnlyInline - readonly_fields = ['name'] + readonly_fields = ["name"] class InnerInline2(admin.StackedInline): model = Inner2 class Media: - js = ('my_awesome_inline_scripts.js',) + js = ("my_awesome_inline_scripts.js",) class InnerInline2Tabular(admin.TabularInline): @@ -84,17 +127,17 @@ class InnerInline2Tabular(admin.TabularInline): class CustomNumberWidget(forms.NumberInput): class Media: - js = ('custom_number.js',) + js = ("custom_number.js",) class InnerInline3(admin.StackedInline): model = Inner3 formfield_overrides = { - models.IntegerField: {'widget': CustomNumberWidget}, + models.IntegerField: {"widget": CustomNumberWidget}, } class Media: - js = ('my_awesome_inline_scripts.js',) + js = ("my_awesome_inline_scripts.js",) class TitleForm(forms.ModelForm): @@ -131,12 +174,12 @@ class Holder4Admin(admin.ModelAdmin): class Inner5StackedInline(admin.StackedInline): model = Inner5Stacked - classes = ('collapse',) + classes = ("collapse",) class Inner5TabularInline(admin.TabularInline): model = Inner5Tabular - classes = ('collapse',) + classes = ("collapse",) class Holder5Admin(admin.ModelAdmin): @@ -153,7 +196,7 @@ class WeaknessForm(forms.ModelForm): class Meta: model = ShoppingWeakness - fields = '__all__' + fields = "__all__" class WeaknessInlineCustomForm(admin.TabularInline): @@ -166,7 +209,7 @@ class FootNoteForm(forms.ModelForm): class Meta: model = FootNote - fields = '__all__' + fields = "__all__" class FootNoteNonEditableInlineCustomForm(admin.TabularInline): @@ -179,25 +222,25 @@ class FootNoteNonEditableInlineCustomForm(admin.TabularInline): class QuestionInline(admin.TabularInline): model = Question - readonly_fields = ['call_me'] + readonly_fields = ["call_me"] def call_me(self, obj): - return 'Callable in QuestionInline' + return "Callable in QuestionInline" class PollAdmin(admin.ModelAdmin): inlines = [QuestionInline] def call_me(self, obj): - return 'Callable in PollAdmin' + return "Callable in PollAdmin" class ChapterInline(admin.TabularInline): model = Chapter - readonly_fields = ['call_me'] + readonly_fields = ["call_me"] def call_me(self, obj): - return 'Callable in ChapterInline' + return "Callable in ChapterInline" class NovelAdmin(admin.ModelAdmin): @@ -261,32 +304,31 @@ class SightingInline(admin.TabularInline): # admin and form for #18263 class SomeChildModelForm(forms.ModelForm): - class Meta: - fields = '__all__' + fields = "__all__" model = SomeChildModel widgets = { - 'position': forms.HiddenInput, + "position": forms.HiddenInput, } - labels = {'readonly_field': 'Label from ModelForm.Meta'} - help_texts = {'readonly_field': 'Help text from ModelForm.Meta'} + labels = {"readonly_field": "Label from ModelForm.Meta"} + help_texts = {"readonly_field": "Help text from ModelForm.Meta"} def __init__(self, *args, **kwargs): super().__init__(*args, **kwargs) - self.fields['name'].label = 'new label' + self.fields["name"].label = "new label" class SomeChildModelInline(admin.TabularInline): model = SomeChildModel form = SomeChildModelForm - readonly_fields = ('readonly_field',) + readonly_fields = ("readonly_field",) class StudentInline(admin.StackedInline): model = Child extra = 1 fieldsets = [ - ('Name', {'fields': ('name',), 'classes': ('collapse',)}), + ("Name", {"fields": ("name",), "classes": ("collapse",)}), ] @@ -306,7 +348,7 @@ class FashonistaStackedInline(admin.StackedInline): class ClassStackedHorizontal(admin.StackedInline): model = Class extra = 1 - filter_horizontal = ['person'] + filter_horizontal = ["person"] class ClassAdminStackedHorizontal(admin.ModelAdmin): @@ -316,7 +358,7 @@ class ClassAdminStackedHorizontal(admin.ModelAdmin): class ClassTabularHorizontal(admin.TabularInline): model = Class extra = 1 - filter_horizontal = ['person'] + filter_horizontal = ["person"] class ClassAdminTabularHorizontal(admin.ModelAdmin): @@ -326,7 +368,7 @@ class ClassAdminTabularHorizontal(admin.ModelAdmin): class ClassTabularVertical(admin.TabularInline): model = Class extra = 1 - filter_vertical = ['person'] + filter_vertical = ["person"] class ClassAdminTabularVertical(admin.ModelAdmin): @@ -336,7 +378,7 @@ class ClassAdminTabularVertical(admin.ModelAdmin): class ClassStackedVertical(admin.StackedInline): model = Class extra = 1 - filter_vertical = ['person'] + filter_vertical = ["person"] class ClassAdminStackedVertical(admin.ModelAdmin): @@ -346,13 +388,13 @@ class ClassAdminStackedVertical(admin.ModelAdmin): class ChildHiddenFieldForm(forms.ModelForm): class Meta: model = SomeChildModel - fields = ['name', 'position', 'parent'] - widgets = {'position': forms.HiddenInput} + fields = ["name", "position", "parent"] + widgets = {"position": forms.HiddenInput} def _post_clean(self): super()._post_clean() if self.instance is not None and self.instance.position == 1: - self.add_error(None, ValidationError('A non-field error')) + self.add_error(None, ValidationError("A non-field error")) class ChildHiddenFieldTabularInline(admin.TabularInline): @@ -363,13 +405,13 @@ class ChildHiddenFieldTabularInline(admin.TabularInline): class ChildHiddenFieldInFieldsGroupStackedInline(admin.StackedInline): model = SomeChildModel form = ChildHiddenFieldForm - fields = [('name', 'position')] + fields = [("name", "position")] class ChildHiddenFieldOnSingleLineStackedInline(admin.StackedInline): model = SomeChildModel form = ChildHiddenFieldForm - fields = ('name', 'position') + fields = ("name", "position") class ShowInlineChildInline(admin.StackedInline): @@ -399,7 +441,9 @@ site.register(Fashionista, inlines=[InlineWeakness]) site.register(Holder4, Holder4Admin) site.register(Holder5, Holder5Admin) site.register(Author, AuthorAdmin) -site.register(CapoFamiglia, inlines=[ConsigliereInline, SottoCapoInline, ReadOnlyInlineInline]) +site.register( + CapoFamiglia, inlines=[ConsigliereInline, SottoCapoInline, ReadOnlyInlineInline] +) site.register(ProfileCollection, inlines=[ProfileInline]) site.register(ParentModelWithCustomPk, inlines=[ChildModel1Inline, ChildModel2Inline]) site.register(BinaryTree, inlines=[BinaryTreeAdmin]) @@ -416,9 +460,9 @@ site.register(CourseProxy1, ClassAdminTabularVertical) site.register(CourseProxy2, ClassAdminTabularHorizontal) site.register(ShowInlineParent, ShowInlineParentAdmin) # Used to test hidden fields in tabular and stacked inlines. -site2 = admin.AdminSite(name='tabular_inline_hidden_field_admin') +site2 = admin.AdminSite(name="tabular_inline_hidden_field_admin") site2.register(SomeParentModel, inlines=[ChildHiddenFieldTabularInline]) -site3 = admin.AdminSite(name='stacked_inline_hidden_field_in_group_admin') +site3 = admin.AdminSite(name="stacked_inline_hidden_field_in_group_admin") site3.register(SomeParentModel, inlines=[ChildHiddenFieldInFieldsGroupStackedInline]) -site4 = admin.AdminSite(name='stacked_inline_hidden_field_on_single_line_admin') +site4 = admin.AdminSite(name="stacked_inline_hidden_field_on_single_line_admin") site4.register(SomeParentModel, inlines=[ChildHiddenFieldOnSingleLineStackedInline]) diff --git a/tests/admin_inlines/models.py b/tests/admin_inlines/models.py index 6125e06c28..47c5b91828 100644 --- a/tests/admin_inlines/models.py +++ b/tests/admin_inlines/models.py @@ -31,7 +31,7 @@ class Child(models.Model): parent = GenericForeignKey() def __str__(self): - return 'I am %s, a child of %s' % (self.name, self.parent) + return "I am %s, a child of %s" % (self.name, self.parent) class Book(models.Model): @@ -44,7 +44,7 @@ class Book(models.Model): class Author(models.Model): name = models.CharField(max_length=50) books = models.ManyToManyField(Book) - person = models.OneToOneField('Person', models.CASCADE, null=True) + person = models.OneToOneField("Person", models.CASCADE, null=True) class NonAutoPKBook(models.Model): @@ -80,7 +80,7 @@ class Inner(models.Model): readonly = models.CharField("Inner readonly label", max_length=1) def get_absolute_url(self): - return '/inner/' + return "/inner/" class Holder2(models.Model): @@ -100,6 +100,7 @@ class Inner3(models.Model): dummy = models.IntegerField() holder = models.ForeignKey(Holder3, models.CASCADE) + # Models for ticket #8190 @@ -113,7 +114,9 @@ class Inner4Stacked(models.Model): class Meta: constraints = [ - models.UniqueConstraint(fields=['dummy', 'holder'], name='unique_stacked_dummy_per_holder') + models.UniqueConstraint( + fields=["dummy", "holder"], name="unique_stacked_dummy_per_holder" + ) ] @@ -123,9 +126,12 @@ class Inner4Tabular(models.Model): class Meta: constraints = [ - models.UniqueConstraint(fields=['dummy', 'holder'], name='unique_tabular_dummy_per_holder') + models.UniqueConstraint( + fields=["dummy", "holder"], name="unique_tabular_dummy_per_holder" + ) ] + # Models for ticket #31441 @@ -135,7 +141,7 @@ class Holder5(models.Model): class Inner5Stacked(models.Model): name = models.CharField(max_length=10) - select = models.CharField(choices=(('1', 'One'), ('2', 'Two')), max_length=10) + select = models.CharField(choices=(("1", "One"), ("2", "Two")), max_length=10) text = models.TextField() dummy = models.IntegerField() holder = models.ForeignKey(Holder5, models.CASCADE) @@ -143,7 +149,7 @@ class Inner5Stacked(models.Model): class Inner5Tabular(models.Model): name = models.CharField(max_length=10) - select = models.CharField(choices=(('1', 'One'), ('2', 'Two')), max_length=10) + select = models.CharField(choices=(("1", "One"), ("2", "Two")), max_length=10) text = models.TextField() dummy = models.IntegerField() holder = models.ForeignKey(Holder5, models.CASCADE) @@ -162,13 +168,16 @@ class OutfitItem(models.Model): class Fashionista(models.Model): person = models.OneToOneField(Person, models.CASCADE, primary_key=True) - weaknesses = models.ManyToManyField(OutfitItem, through='ShoppingWeakness', blank=True) + weaknesses = models.ManyToManyField( + OutfitItem, through="ShoppingWeakness", blank=True + ) class ShoppingWeakness(models.Model): fashionista = models.ForeignKey(Fashionista, models.CASCADE) item = models.ForeignKey(OutfitItem, models.CASCADE) + # Models for #13510 @@ -177,10 +186,13 @@ class TitleCollection(models.Model): class Title(models.Model): - collection = models.ForeignKey(TitleCollection, models.SET_NULL, blank=True, null=True) + collection = models.ForeignKey( + TitleCollection, models.SET_NULL, blank=True, null=True + ) title1 = models.CharField(max_length=100) title2 = models.CharField(max_length=100) + # Models for #15424 @@ -198,7 +210,6 @@ class Novel(models.Model): class NovelReadonlyChapter(Novel): - class Meta: proxy = True @@ -212,9 +223,11 @@ class FootNote(models.Model): """ Model added for ticket 19838 """ + chapter = models.ForeignKey(Chapter, models.PROTECT) note = models.CharField(max_length=40) + # Models for #16838 @@ -223,22 +236,23 @@ class CapoFamiglia(models.Model): class Consigliere(models.Model): - name = models.CharField(max_length=100, help_text='Help text for Consigliere') - capo_famiglia = models.ForeignKey(CapoFamiglia, models.CASCADE, related_name='+') + name = models.CharField(max_length=100, help_text="Help text for Consigliere") + capo_famiglia = models.ForeignKey(CapoFamiglia, models.CASCADE, related_name="+") class SottoCapo(models.Model): name = models.CharField(max_length=100) - capo_famiglia = models.ForeignKey(CapoFamiglia, models.CASCADE, related_name='+') + capo_famiglia = models.ForeignKey(CapoFamiglia, models.CASCADE, related_name="+") class ReadOnlyInline(models.Model): - name = models.CharField(max_length=100, help_text='Help text for ReadOnlyInline') + name = models.CharField(max_length=100, help_text="Help text for ReadOnlyInline") capo_famiglia = models.ForeignKey(CapoFamiglia, models.CASCADE) # Models for #18433 + class ParentModelWithCustomPk(models.Model): my_own_pk = models.CharField(max_length=100, primary_key=True) name = models.CharField(max_length=100) @@ -250,7 +264,7 @@ class ChildModel1(models.Model): parent = models.ForeignKey(ParentModelWithCustomPk, models.CASCADE) def get_absolute_url(self): - return '/child_model1/' + return "/child_model1/" class ChildModel2(models.Model): @@ -259,13 +273,14 @@ class ChildModel2(models.Model): parent = models.ForeignKey(ParentModelWithCustomPk, models.CASCADE) def get_absolute_url(self): - return '/child_model2/' + return "/child_model2/" # Models for #19425 class BinaryTree(models.Model): name = models.CharField(max_length=100) - parent = models.ForeignKey('self', models.SET_NULL, null=True, blank=True) + parent = models.ForeignKey("self", models.SET_NULL, null=True, blank=True) + # Models for #19524 @@ -304,24 +319,21 @@ class Course(models.Model): class Class(models.Model): - person = models.ManyToManyField(Person, verbose_name='attendant') + person = models.ManyToManyField(Person, verbose_name="attendant") course = models.ForeignKey(Course, on_delete=models.CASCADE) class CourseProxy(Course): - class Meta: proxy = True class CourseProxy1(Course): - class Meta: proxy = True class CourseProxy2(Course): - class Meta: proxy = True @@ -340,22 +352,24 @@ class ProfileCollection(models.Model): class Profile(models.Model): - collection = models.ForeignKey(ProfileCollection, models.SET_NULL, blank=True, null=True) + collection = models.ForeignKey( + ProfileCollection, models.SET_NULL, blank=True, null=True + ) first_name = models.CharField(max_length=100) last_name = models.CharField(max_length=100) class VerboseNameProfile(Profile): class Meta: - verbose_name = 'Model with verbose name only' + verbose_name = "Model with verbose name only" class VerboseNamePluralProfile(Profile): class Meta: - verbose_name_plural = 'Model with verbose name plural only' + verbose_name_plural = "Model with verbose name plural only" class BothVerboseNameProfile(Profile): class Meta: - verbose_name = 'Model with both - name' - verbose_name_plural = 'Model with both - plural name' + verbose_name = "Model with both - name" + verbose_name_plural = "Model with both - plural name" diff --git a/tests/admin_inlines/test_templates.py b/tests/admin_inlines/test_templates.py index 5c8d7ce0c1..9ed4d65944 100644 --- a/tests/admin_inlines/test_templates.py +++ b/tests/admin_inlines/test_templates.py @@ -7,17 +7,19 @@ from django.test import SimpleTestCase class TestTemplates(SimpleTestCase): def test_javascript_escaping(self): context = { - 'inline_admin_formset': { - 'inline_formset_data': json.dumps({ - 'formset': {'prefix': 'my-prefix'}, - 'opts': {'verbose_name': 'verbose name\\'}, - }), + "inline_admin_formset": { + "inline_formset_data": json.dumps( + { + "formset": {"prefix": "my-prefix"}, + "opts": {"verbose_name": "verbose name\\"}, + } + ), }, } - output = render_to_string('admin/edit_inline/stacked.html', context) - self.assertIn('"prefix": "my-prefix"', output) - self.assertIn('"verbose_name": "verbose name\\\\"', output) + output = render_to_string("admin/edit_inline/stacked.html", context) + self.assertIn(""prefix": "my-prefix"", output) + self.assertIn(""verbose_name": "verbose name\\\\"", output) - output = render_to_string('admin/edit_inline/tabular.html', context) - self.assertIn('"prefix": "my-prefix"', output) - self.assertIn('"verbose_name": "verbose name\\\\"', output) + output = render_to_string("admin/edit_inline/tabular.html", context) + self.assertIn(""prefix": "my-prefix"", output) + self.assertIn(""verbose_name": "verbose name\\\\"", output) diff --git a/tests/admin_inlines/tests.py b/tests/admin_inlines/tests.py index 54bbec28aa..7db86f5663 100644 --- a/tests/admin_inlines/tests.py +++ b/tests/admin_inlines/tests.py @@ -6,27 +6,58 @@ from django.contrib.contenttypes.models import ContentType from django.test import RequestFactory, TestCase, override_settings from django.urls import reverse -from .admin import InnerInline, site as admin_site +from .admin import InnerInline +from .admin import site as admin_site from .models import ( - Author, BinaryTree, Book, BothVerboseNameProfile, Chapter, Child, - ChildModel1, ChildModel2, Fashionista, FootNote, Holder, Holder2, Holder3, - Holder4, Inner, Inner2, Inner3, Inner4Stacked, Inner4Tabular, Novel, - OutfitItem, Parent, ParentModelWithCustomPk, Person, Poll, Profile, - ProfileCollection, Question, ShowInlineParent, Sighting, SomeChildModel, - SomeParentModel, Teacher, VerboseNamePluralProfile, VerboseNameProfile, + Author, + BinaryTree, + Book, + BothVerboseNameProfile, + Chapter, + Child, + ChildModel1, + ChildModel2, + Fashionista, + FootNote, + Holder, + Holder2, + Holder3, + Holder4, + Inner, + Inner2, + Inner3, + Inner4Stacked, + Inner4Tabular, + Novel, + OutfitItem, + Parent, + ParentModelWithCustomPk, + Person, + Poll, + Profile, + ProfileCollection, + Question, + ShowInlineParent, + Sighting, + SomeChildModel, + SomeParentModel, + Teacher, + VerboseNamePluralProfile, + VerboseNameProfile, ) INLINE_CHANGELINK_HTML = 'class="inlinechangelink">Change</a>' class TestDataMixin: - @classmethod def setUpTestData(cls): - cls.superuser = User.objects.create_superuser(username='super', email='super@example.com', password='secret') + cls.superuser = User.objects.create_superuser( + username="super", email="super@example.com", password="secret" + ) -@override_settings(ROOT_URLCONF='admin_inlines.urls') +@override_settings(ROOT_URLCONF="admin_inlines.urls") class TestInline(TestDataMixin, TestCase): factory = RequestFactory() @@ -36,22 +67,24 @@ class TestInline(TestDataMixin, TestCase): cls.holder = Holder.objects.create(dummy=13) Inner.objects.create(dummy=42, holder=cls.holder) - cls.parent = SomeParentModel.objects.create(name='a') - SomeChildModel.objects.create(name='b', position='0', parent=cls.parent) - SomeChildModel.objects.create(name='c', position='1', parent=cls.parent) + cls.parent = SomeParentModel.objects.create(name="a") + SomeChildModel.objects.create(name="b", position="0", parent=cls.parent) + SomeChildModel.objects.create(name="c", position="1", parent=cls.parent) cls.view_only_user = User.objects.create_user( - username='user', password='pwd', is_staff=True, + username="user", + password="pwd", + is_staff=True, ) parent_ct = ContentType.objects.get_for_model(SomeParentModel) child_ct = ContentType.objects.get_for_model(SomeChildModel) permission = Permission.objects.get( - codename='view_someparentmodel', + codename="view_someparentmodel", content_type=parent_ct, ) cls.view_only_user.user_permissions.add(permission) permission = Permission.objects.get( - codename='view_somechildmodel', + codename="view_somechildmodel", content_type=child_ct, ) cls.view_only_user.user_permissions.add(permission) @@ -64,61 +97,65 @@ class TestInline(TestDataMixin, TestCase): can_delete should be passed to inlineformset factory. """ response = self.client.get( - reverse('admin:admin_inlines_holder_change', args=(self.holder.id,)) + reverse("admin:admin_inlines_holder_change", args=(self.holder.id,)) ) - inner_formset = response.context['inline_admin_formsets'][0].formset + inner_formset = response.context["inline_admin_formsets"][0].formset expected = InnerInline.can_delete actual = inner_formset.can_delete - self.assertEqual(expected, actual, 'can_delete must be equal') + self.assertEqual(expected, actual, "can_delete must be equal") def test_readonly_stacked_inline_label(self): """Bug #13174.""" holder = Holder.objects.create(dummy=42) - Inner.objects.create(holder=holder, dummy=42, readonly='') + Inner.objects.create(holder=holder, dummy=42, readonly="") response = self.client.get( - reverse('admin:admin_inlines_holder_change', args=(holder.id,)) + reverse("admin:admin_inlines_holder_change", args=(holder.id,)) ) - self.assertContains(response, '<label>Inner readonly label:</label>') + self.assertContains(response, "<label>Inner readonly label:</label>") def test_many_to_many_inlines(self): "Autogenerated many-to-many inlines are displayed correctly (#13407)" - response = self.client.get(reverse('admin:admin_inlines_author_add')) + response = self.client.get(reverse("admin:admin_inlines_author_add")) # The heading for the m2m inline block uses the right text - self.assertContains(response, '<h2>Author-book relationships</h2>') + self.assertContains(response, "<h2>Author-book relationships</h2>") # The "add another" label is correct - self.assertContains(response, 'Add another Author-book relationship') + self.assertContains(response, "Add another Author-book relationship") # The '+' is dropped from the autogenerated form prefix (Author_books+) self.assertContains(response, 'id="id_Author_books-TOTAL_FORMS"') def test_inline_primary(self): - person = Person.objects.create(firstname='Imelda') - item = OutfitItem.objects.create(name='Shoes') + person = Person.objects.create(firstname="Imelda") + item = OutfitItem.objects.create(name="Shoes") # Imelda likes shoes, but can't carry her own bags. data = { - 'shoppingweakness_set-TOTAL_FORMS': 1, - 'shoppingweakness_set-INITIAL_FORMS': 0, - 'shoppingweakness_set-MAX_NUM_FORMS': 0, - '_save': 'Save', - 'person': person.id, - 'max_weight': 0, - 'shoppingweakness_set-0-item': item.id, + "shoppingweakness_set-TOTAL_FORMS": 1, + "shoppingweakness_set-INITIAL_FORMS": 0, + "shoppingweakness_set-MAX_NUM_FORMS": 0, + "_save": "Save", + "person": person.id, + "max_weight": 0, + "shoppingweakness_set-0-item": item.id, } - response = self.client.post(reverse('admin:admin_inlines_fashionista_add'), data) + response = self.client.post( + reverse("admin:admin_inlines_fashionista_add"), data + ) self.assertEqual(response.status_code, 302) - self.assertEqual(len(Fashionista.objects.filter(person__firstname='Imelda')), 1) + self.assertEqual(len(Fashionista.objects.filter(person__firstname="Imelda")), 1) def test_tabular_inline_column_css_class(self): """ Field names are included in the context to output a field-specific CSS class name in the column headers. """ - response = self.client.get(reverse('admin:admin_inlines_poll_add')) - text_field, call_me_field = list(response.context['inline_admin_formset'].fields()) + response = self.client.get(reverse("admin:admin_inlines_poll_add")) + text_field, call_me_field = list( + response.context["inline_admin_formset"].fields() + ) # Editable field. - self.assertEqual(text_field['name'], 'text') + self.assertEqual(text_field["name"], "text") self.assertContains(response, '<th class="column-text required">') # Read-only field. - self.assertEqual(call_me_field['name'], 'call_me') + self.assertEqual(call_me_field["name"], "call_me") self.assertContains(response, '<th class="column-call_me">') def test_custom_form_tabular_inline_label(self): @@ -126,28 +163,32 @@ class TestInline(TestDataMixin, TestCase): A model form with a form field specified (TitleForm.title1) should have its label rendered in the tabular inline. """ - response = self.client.get(reverse('admin:admin_inlines_titlecollection_add')) - self.assertContains(response, '<th class="column-title1 required">Title1</th>', html=True) + response = self.client.get(reverse("admin:admin_inlines_titlecollection_add")) + self.assertContains( + response, '<th class="column-title1 required">Title1</th>', html=True + ) def test_custom_form_tabular_inline_extra_field_label(self): - response = self.client.get(reverse('admin:admin_inlines_outfititem_add')) - _, extra_field = list(response.context['inline_admin_formset'].fields()) - self.assertEqual(extra_field['label'], 'Extra field') + response = self.client.get(reverse("admin:admin_inlines_outfititem_add")) + _, extra_field = list(response.context["inline_admin_formset"].fields()) + self.assertEqual(extra_field["label"], "Extra field") def test_non_editable_custom_form_tabular_inline_extra_field_label(self): - response = self.client.get(reverse('admin:admin_inlines_chapter_add')) - _, extra_field = list(response.context['inline_admin_formset'].fields()) - self.assertEqual(extra_field['label'], 'Extra field') + response = self.client.get(reverse("admin:admin_inlines_chapter_add")) + _, extra_field = list(response.context["inline_admin_formset"].fields()) + self.assertEqual(extra_field["label"], "Extra field") def test_custom_form_tabular_inline_overridden_label(self): """ SomeChildModelForm.__init__() overrides the label of a form field. That label is displayed in the TabularInline. """ - response = self.client.get(reverse('admin:admin_inlines_someparentmodel_add')) - field = list(response.context['inline_admin_formset'].fields())[0] - self.assertEqual(field['label'], 'new label') - self.assertContains(response, '<th class="column-name required">New label</th>', html=True) + response = self.client.get(reverse("admin:admin_inlines_someparentmodel_add")) + field = list(response.context["inline_admin_formset"].fields())[0] + self.assertEqual(field["label"], "new label") + self.assertContains( + response, '<th class="column-name required">New label</th>', html=True + ) def test_tabular_non_field_errors(self): """ @@ -155,68 +196,72 @@ class TestInline(TestDataMixin, TestCase): for colspan. """ data = { - 'title_set-TOTAL_FORMS': 1, - 'title_set-INITIAL_FORMS': 0, - 'title_set-MAX_NUM_FORMS': 0, - '_save': 'Save', - 'title_set-0-title1': 'a title', - 'title_set-0-title2': 'a different title', + "title_set-TOTAL_FORMS": 1, + "title_set-INITIAL_FORMS": 0, + "title_set-MAX_NUM_FORMS": 0, + "_save": "Save", + "title_set-0-title1": "a title", + "title_set-0-title2": "a different title", } - response = self.client.post(reverse('admin:admin_inlines_titlecollection_add'), data) + response = self.client.post( + reverse("admin:admin_inlines_titlecollection_add"), data + ) # Here colspan is "4": two fields (title1 and title2), one hidden field and the delete checkbox. self.assertContains( response, '<tr class="row-form-errors"><td colspan="4"><ul class="errorlist nonfield">' - '<li>The two titles must be the same</li></ul></td></tr>' + "<li>The two titles must be the same</li></ul></td></tr>", ) def test_no_parent_callable_lookup(self): """Admin inline `readonly_field` shouldn't invoke parent ModelAdmin callable""" # Identically named callable isn't present in the parent ModelAdmin, # rendering of the add view shouldn't explode - response = self.client.get(reverse('admin:admin_inlines_novel_add')) + response = self.client.get(reverse("admin:admin_inlines_novel_add")) # View should have the child inlines section self.assertContains( response, - '<div class="js-inline-admin-formset inline-group" id="chapter_set-group"' + '<div class="js-inline-admin-formset inline-group" id="chapter_set-group"', ) def test_callable_lookup(self): """Admin inline should invoke local callable when its name is listed in readonly_fields""" - response = self.client.get(reverse('admin:admin_inlines_poll_add')) + response = self.client.get(reverse("admin:admin_inlines_poll_add")) # Add parent object view should have the child inlines section self.assertContains( response, - '<div class="js-inline-admin-formset inline-group" id="question_set-group"' + '<div class="js-inline-admin-formset inline-group" id="question_set-group"', ) # The right callable should be used for the inline readonly_fields # column cells - self.assertContains(response, '<p>Callable in QuestionInline</p>') + self.assertContains(response, "<p>Callable in QuestionInline</p>") def test_help_text(self): """ The inlines' model field help texts are displayed when using both the stacked and tabular layouts. """ - response = self.client.get(reverse('admin:admin_inlines_holder4_add')) - self.assertContains(response, '<div class="help">Awesome stacked help text is awesome.</div>', 4) + response = self.client.get(reverse("admin:admin_inlines_holder4_add")) + self.assertContains( + response, '<div class="help">Awesome stacked help text is awesome.</div>', 4 + ) self.assertContains( response, '<img src="/static/admin/img/icon-unknown.svg" ' 'class="help help-tooltip" width="10" height="10" ' 'alt="(Awesome tabular help text is awesome.)" ' 'title="Awesome tabular help text is awesome.">', - 1 + 1, ) # ReadOnly fields - response = self.client.get(reverse('admin:admin_inlines_capofamiglia_add')) + response = self.client.get(reverse("admin:admin_inlines_capofamiglia_add")) self.assertContains( response, '<img src="/static/admin/img/icon-unknown.svg" ' 'class="help help-tooltip" width="10" height="10" ' 'alt="(Help text for ReadOnlyInline)" ' 'title="Help text for ReadOnlyInline">', - 1 + 1, ) def test_tabular_model_form_meta_readonly_field(self): @@ -224,22 +269,24 @@ class TestInline(TestDataMixin, TestCase): Tabular inlines use ModelForm.Meta.help_texts and labels for read-only fields. """ - response = self.client.get(reverse('admin:admin_inlines_someparentmodel_add')) + response = self.client.get(reverse("admin:admin_inlines_someparentmodel_add")) self.assertContains( response, '<img src="/static/admin/img/icon-unknown.svg" ' 'class="help help-tooltip" width="10" height="10" ' 'alt="(Help text from ModelForm.Meta)" ' - 'title="Help text from ModelForm.Meta">' + 'title="Help text from ModelForm.Meta">', ) - self.assertContains(response, 'Label from ModelForm.Meta') + self.assertContains(response, "Label from ModelForm.Meta") def test_inline_hidden_field_no_column(self): """#18263 -- Make sure hidden fields don't get a column in tabular inlines""" - 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(reverse('admin:admin_inlines_someparentmodel_change', args=(parent.pk,))) + 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( + reverse("admin:admin_inlines_someparentmodel_change", args=(parent.pk,)) + ) self.assertNotContains(response, '<td class="field-position">') self.assertInHTML( '<input id="id_somechildmodel_set-1-position" ' @@ -254,13 +301,20 @@ class TestInline(TestDataMixin, TestCase): """ self.client.force_login(self.view_only_user) url = reverse( - 'tabular_inline_hidden_field_admin:admin_inlines_someparentmodel_change', + "tabular_inline_hidden_field_admin:admin_inlines_someparentmodel_change", args=(self.parent.pk,), ) response = self.client.get(url) - self.assertInHTML('<th class="column-position hidden">Position</th>', response.rendered_content) - self.assertInHTML('<td class="field-position hidden"><p>0</p></td>', response.rendered_content) - self.assertInHTML('<td class="field-position hidden"><p>1</p></td>', response.rendered_content) + self.assertInHTML( + '<th class="column-position hidden">Position</th>', + response.rendered_content, + ) + self.assertInHTML( + '<td class="field-position hidden"><p>0</p></td>', response.rendered_content + ) + self.assertInHTML( + '<td class="field-position hidden"><p>1</p></td>', response.rendered_content + ) def test_stacked_inline_hidden_field_with_view_only_permissions(self): """ @@ -269,12 +323,14 @@ class TestInline(TestDataMixin, TestCase): """ self.client.force_login(self.view_only_user) url = reverse( - 'stacked_inline_hidden_field_in_group_admin:admin_inlines_someparentmodel_change', + "stacked_inline_hidden_field_in_group_admin:admin_inlines_someparentmodel_change", args=(self.parent.pk,), ) response = self.client.get(url) # The whole line containing name + position fields is not hidden. - self.assertContains(response, '<div class="form-row field-name field-position">') + self.assertContains( + response, '<div class="form-row field-name field-position">' + ) # The div containing the position field is hidden. self.assertInHTML( '<div class="fieldBox field-position hidden">' @@ -289,54 +345,58 @@ class TestInline(TestDataMixin, TestCase): response.rendered_content, ) - def test_stacked_inline_single_hidden_field_in_line_with_view_only_permissions(self): + def test_stacked_inline_single_hidden_field_in_line_with_view_only_permissions( + self, + ): """ Content of hidden field is not visible in stacked inline when user has view-only permission and the field is grouped on a separate line. """ self.client.force_login(self.view_only_user) url = reverse( - 'stacked_inline_hidden_field_on_single_line_admin:admin_inlines_someparentmodel_change', + "stacked_inline_hidden_field_on_single_line_admin:admin_inlines_someparentmodel_change", args=(self.parent.pk,), ) response = self.client.get(url) # The whole line containing position field is hidden. self.assertInHTML( '<div class="form-row hidden field-position">' - '<div><label>Position:</label>' + "<div><label>Position:</label>" '<div class="readonly">0</div></div></div>', response.rendered_content, ) self.assertInHTML( '<div class="form-row hidden field-position">' - '<div><label>Position:</label>' + "<div><label>Position:</label>" '<div class="readonly">1</div></div></div>', response.rendered_content, ) - def test_tabular_inline_with_hidden_field_non_field_errors_has_correct_colspan(self): + def test_tabular_inline_with_hidden_field_non_field_errors_has_correct_colspan( + self, + ): """ In tabular inlines, when a form has non-field errors, those errors are rendered in a table line with a single cell spanning the whole table width. Colspan must be equal to the number of visible columns. """ - parent = SomeParentModel.objects.create(name='a') - child = SomeChildModel.objects.create(name='b', position='0', parent=parent) + parent = SomeParentModel.objects.create(name="a") + child = SomeChildModel.objects.create(name="b", position="0", parent=parent) url = reverse( - 'tabular_inline_hidden_field_admin:admin_inlines_someparentmodel_change', + "tabular_inline_hidden_field_admin:admin_inlines_someparentmodel_change", args=(parent.id,), ) data = { - 'name': parent.name, - 'somechildmodel_set-TOTAL_FORMS': 1, - 'somechildmodel_set-INITIAL_FORMS': 1, - 'somechildmodel_set-MIN_NUM_FORMS': 0, - 'somechildmodel_set-MAX_NUM_FORMS': 1000, - '_save': 'Save', - 'somechildmodel_set-0-id': child.id, - 'somechildmodel_set-0-parent': parent.id, - 'somechildmodel_set-0-name': child.name, - 'somechildmodel_set-0-position': 1, + "name": parent.name, + "somechildmodel_set-TOTAL_FORMS": 1, + "somechildmodel_set-INITIAL_FORMS": 1, + "somechildmodel_set-MIN_NUM_FORMS": 0, + "somechildmodel_set-MAX_NUM_FORMS": 1000, + "_save": "Save", + "somechildmodel_set-0-id": child.id, + "somechildmodel_set-0-parent": parent.id, + "somechildmodel_set-0-name": child.name, + "somechildmodel_set-0-position": 1, } response = self.client.post(url, data) # Form has 3 visible columns and 1 hidden column. @@ -344,7 +404,7 @@ class TestInline(TestDataMixin, TestCase): '<thead><tr><th class="original"></th>' '<th class="column-name required">Name</th>' '<th class="column-position required hidden">Position</th>' - '<th>Delete?</th></tr></thead>', + "<th>Delete?</th></tr></thead>", response.rendered_content, ) # The non-field error must be spanned on 3 (visible) columns. @@ -358,28 +418,32 @@ class TestInline(TestDataMixin, TestCase): """ Multiple inlines with related_name='+' have correct form prefixes. """ - response = self.client.get(reverse('admin:admin_inlines_capofamiglia_add')) - self.assertContains(response, '<input type="hidden" name="-1-0-id" id="id_-1-0-id">', html=True) + response = self.client.get(reverse("admin:admin_inlines_capofamiglia_add")) + self.assertContains( + response, '<input type="hidden" name="-1-0-id" id="id_-1-0-id">', html=True + ) self.assertContains( response, '<input type="hidden" name="-1-0-capo_famiglia" id="id_-1-0-capo_famiglia">', - html=True + html=True, ) self.assertContains( response, '<input id="id_-1-0-name" type="text" class="vTextField" name="-1-0-name" maxlength="100">', - html=True + html=True, + ) + self.assertContains( + response, '<input type="hidden" name="-2-0-id" id="id_-2-0-id">', html=True ) - self.assertContains(response, '<input type="hidden" name="-2-0-id" id="id_-2-0-id">', html=True) self.assertContains( response, '<input type="hidden" name="-2-0-capo_famiglia" id="id_-2-0-capo_famiglia">', - html=True + html=True, ) self.assertContains( response, '<input id="id_-2-0-name" type="text" class="vTextField" name="-2-0-name" maxlength="100">', - html=True + html=True, ) @override_settings(USE_THOUSAND_SEPARATOR=True) @@ -389,9 +453,14 @@ class TestInline(TestDataMixin, TestCase): separators. """ holder = Holder.objects.create(pk=123456789, dummy=42) - inner = Inner.objects.create(pk=987654321, holder=holder, dummy=42, readonly='') - 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) + inner = Inner.objects.create(pk=987654321, holder=holder, dummy=42, readonly="") + 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) def test_custom_pk_shortcut(self): @@ -402,9 +471,17 @@ class TestInline(TestDataMixin, TestCase): 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(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) + 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) self.assertContains(response, child2_shortcut) @@ -413,16 +490,18 @@ class TestInline(TestDataMixin, TestCase): An object can be created with inlines when it inherits another class. """ data = { - 'name': 'Martian', - 'sighting_set-TOTAL_FORMS': 1, - 'sighting_set-INITIAL_FORMS': 0, - 'sighting_set-MAX_NUM_FORMS': 0, - 'sighting_set-0-place': 'Zone 51', - '_save': 'Save', + "name": "Martian", + "sighting_set-TOTAL_FORMS": 1, + "sighting_set-INITIAL_FORMS": 0, + "sighting_set-MAX_NUM_FORMS": 0, + "sighting_set-0-place": "Zone 51", + "_save": "Save", } - response = self.client.post(reverse('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) + self.assertEqual(Sighting.objects.filter(et__name="Martian").count(), 1) def test_custom_get_extra_form(self): bt_head = BinaryTree.objects.create(name="Tree Head") @@ -438,11 +517,13 @@ class TestInline(TestDataMixin, TestCase): '<input id="id_binarytree_set-TOTAL_FORMS" ' 'name="binarytree_set-TOTAL_FORMS" type="hidden" value="2">' ) - response = self.client.get(reverse('admin:admin_inlines_binarytree_add')) + response = self.client.get(reverse("admin:admin_inlines_binarytree_add")) self.assertInHTML(max_forms_input % 3, response.rendered_content) self.assertInHTML(total_forms_hidden, response.rendered_content) - response = self.client.get(reverse('admin:admin_inlines_binarytree_change', args=(bt_head.id,))) + response = self.client.get( + reverse("admin:admin_inlines_binarytree_change", args=(bt_head.id,)) + ) self.assertInHTML(max_forms_input % 2, response.rendered_content) self.assertInHTML(total_forms_hidden, response.rendered_content) @@ -450,6 +531,7 @@ class TestInline(TestDataMixin, TestCase): """ min_num and extra determine number of forms. """ + class MinNumInline(TabularInline): model = BinaryTree min_num = 2 @@ -465,8 +547,8 @@ class TestInline(TestDataMixin, TestCase): '<input id="id_binarytree_set-TOTAL_FORMS" ' 'name="binarytree_set-TOTAL_FORMS" type="hidden" value="5">' ) - request = self.factory.get(reverse('admin:admin_inlines_binarytree_add')) - request.user = User(username='super', is_superuser=True) + request = self.factory.get(reverse("admin:admin_inlines_binarytree_add")) + request.user = User(username="super", is_superuser=True) response = modeladmin.changeform_view(request) self.assertInHTML(min_forms, response.rendered_content) self.assertInHTML(total_forms, response.rendered_content) @@ -494,76 +576,80 @@ class TestInline(TestDataMixin, TestCase): '<input id="id_binarytree_set-TOTAL_FORMS" ' 'name="binarytree_set-TOTAL_FORMS" type="hidden" value="%d">' ) - request = self.factory.get(reverse('admin:admin_inlines_binarytree_add')) - request.user = User(username='super', is_superuser=True) + request = self.factory.get(reverse("admin:admin_inlines_binarytree_add")) + request.user = User(username="super", is_superuser=True) response = modeladmin.changeform_view(request) self.assertInHTML(min_forms % 2, response.rendered_content) self.assertInHTML(total_forms % 5, response.rendered_content) - request = self.factory.get(reverse('admin:admin_inlines_binarytree_change', args=(bt_head.id,))) - request.user = User(username='super', is_superuser=True) + 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.assertInHTML(min_forms % 5, response.rendered_content) self.assertInHTML(total_forms % 8, response.rendered_content) def test_inline_nonauto_noneditable_pk(self): - response = self.client.get(reverse('admin:admin_inlines_author_add')) + response = self.client.get(reverse("admin:admin_inlines_author_add")) self.assertContains( response, '<input id="id_nonautopkbook_set-0-rand_pk" ' 'name="nonautopkbook_set-0-rand_pk" type="hidden">', - html=True + html=True, ) self.assertContains( response, '<input id="id_nonautopkbook_set-2-0-rand_pk" ' 'name="nonautopkbook_set-2-0-rand_pk" type="hidden">', - html=True + html=True, ) def test_inline_nonauto_noneditable_inherited_pk(self): - response = self.client.get(reverse('admin:admin_inlines_author_add')) + response = self.client.get(reverse("admin:admin_inlines_author_add")) self.assertContains( response, '<input id="id_nonautopkbookchild_set-0-nonautopkbook_ptr" ' 'name="nonautopkbookchild_set-0-nonautopkbook_ptr" type="hidden">', - html=True + html=True, ) self.assertContains( response, '<input id="id_nonautopkbookchild_set-2-nonautopkbook_ptr" ' 'name="nonautopkbookchild_set-2-nonautopkbook_ptr" type="hidden">', - html=True + html=True, ) def test_inline_editable_pk(self): - response = self.client.get(reverse('admin:admin_inlines_author_add')) + response = self.client.get(reverse("admin:admin_inlines_author_add")) self.assertContains( response, '<input class="vIntegerField" id="id_editablepkbook_set-0-manual_pk" ' 'name="editablepkbook_set-0-manual_pk" type="number">', - html=True, count=1 + html=True, + count=1, ) self.assertContains( response, '<input class="vIntegerField" id="id_editablepkbook_set-2-0-manual_pk" ' 'name="editablepkbook_set-2-0-manual_pk" type="number">', - html=True, count=1 + html=True, + count=1, ) 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(reverse('admin:admin_inlines_holder_change', args=(holder.pk,))) + response = self.client.get( + reverse("admin:admin_inlines_holder_change", args=(holder.pk,)) + ) self.assertContains( response, '<div class="inline-related has_original" id="inner_set-0">', - count=1 + count=1, ) self.assertContains( - response, - '<div class="inline-related" id="inner_set-1">', - count=1 + response, '<div class="inline-related" id="inner_set-1">', count=1 ) def test_inlines_show_change_link_registered(self): @@ -572,60 +658,76 @@ class TestInline(TestDataMixin, TestCase): item1 = Inner4Stacked.objects.create(dummy=1, holder=holder) item2 = Inner4Tabular.objects.create(dummy=1, holder=holder) items = ( - ('inner4stacked', item1.pk), - ('inner4tabular', item2.pk), + ("inner4stacked", item1.pk), + ("inner4tabular", item2.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 ) - 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 = reverse('admin:admin_inlines_%s_change' % model, args=(pk,)) - self.assertContains(response, '<a href="%s" %s' % (url, INLINE_CHANGELINK_HTML)) + url = reverse("admin:admin_inlines_%s_change" % model, args=(pk,)) + self.assertContains( + response, '<a href="%s" %s' % (url, INLINE_CHANGELINK_HTML) + ) def test_inlines_show_change_link_unregistered(self): "Inlines `show_change_link` disabled for unregistered models." parent = ParentModelWithCustomPk.objects.create(my_own_pk="foo", name="Foo") ChildModel1.objects.create(my_own_pk="bar", name="Bar", parent=parent) ChildModel2.objects.create(my_own_pk="baz", name="Baz", parent=parent) - response = self.client.get(reverse('admin:admin_inlines_parentmodelwithcustompk_change', args=('foo',))) - self.assertFalse(response.context['inline_admin_formset'].opts.has_registered_model) + response = self.client.get( + reverse("admin:admin_inlines_parentmodelwithcustompk_change", args=("foo",)) + ) + self.assertFalse( + response.context["inline_admin_formset"].opts.has_registered_model + ) self.assertNotContains(response, INLINE_CHANGELINK_HTML) def test_tabular_inline_show_change_link_false_registered(self): "Inlines `show_change_link` disabled by default." poll = Poll.objects.create(name="New poll") Question.objects.create(poll=poll) - response = self.client.get(reverse('admin:admin_inlines_poll_change', args=(poll.pk,))) - self.assertTrue(response.context['inline_admin_formset'].opts.has_registered_model) + response = self.client.get( + reverse("admin:admin_inlines_poll_change", args=(poll.pk,)) + ) + self.assertTrue( + response.context["inline_admin_formset"].opts.has_registered_model + ) self.assertNotContains(response, INLINE_CHANGELINK_HTML) def test_noneditable_inline_has_field_inputs(self): """Inlines without change permission shows field inputs on add form.""" - response = self.client.get(reverse('admin:admin_inlines_novelreadonlychapter_add')) + response = self.client.get( + reverse("admin:admin_inlines_novelreadonlychapter_add") + ) self.assertContains( response, '<input type="text" name="chapter_set-0-name" ' 'class="vTextField" maxlength="40" id="id_chapter_set-0-name">', - html=True + html=True, ) def test_inlines_plural_heading_foreign_key(self): - response = self.client.get(reverse('admin:admin_inlines_holder4_add')) - self.assertContains(response, '<h2>Inner4 stackeds</h2>', html=True) - self.assertContains(response, '<h2>Inner4 tabulars</h2>', html=True) + response = self.client.get(reverse("admin:admin_inlines_holder4_add")) + self.assertContains(response, "<h2>Inner4 stackeds</h2>", html=True) + self.assertContains(response, "<h2>Inner4 tabulars</h2>", html=True) def test_inlines_singular_heading_one_to_one(self): - response = self.client.get(reverse('admin:admin_inlines_person_add')) - self.assertContains(response, '<h2>Author</h2>', html=True) # Tabular. - self.assertContains(response, '<h2>Fashionista</h2>', html=True) # Stacked. + response = self.client.get(reverse("admin:admin_inlines_person_add")) + self.assertContains(response, "<h2>Author</h2>", html=True) # Tabular. + self.assertContains(response, "<h2>Fashionista</h2>", html=True) # Stacked. def test_inlines_based_on_model_state(self): parent = ShowInlineParent.objects.create(show_inlines=False) data = { - 'show_inlines': 'on', - '_save': 'Save', + "show_inlines": "on", + "_save": "Save", } change_url = reverse( - 'admin:admin_inlines_showinlineparent_change', + "admin:admin_inlines_showinlineparent_change", args=(parent.id,), ) response = self.client.post(change_url, data) @@ -634,9 +736,8 @@ class TestInline(TestDataMixin, TestCase): self.assertIs(parent.show_inlines, True) -@override_settings(ROOT_URLCONF='admin_inlines.urls') +@override_settings(ROOT_URLCONF="admin_inlines.urls") class TestInlineMedia(TestDataMixin, TestCase): - def setUp(self): self.client.force_login(self.superuser) @@ -644,41 +745,40 @@ class TestInlineMedia(TestDataMixin, TestCase): holder = Holder(dummy=13) holder.save() Inner(dummy=42, holder=holder).save() - change_url = reverse('admin:admin_inlines_holder_change', args=(holder.id,)) + change_url = reverse("admin:admin_inlines_holder_change", args=(holder.id,)) response = self.client.get(change_url) - self.assertContains(response, 'my_awesome_admin_scripts.js') + self.assertContains(response, "my_awesome_admin_scripts.js") def test_inline_media_only_inline(self): holder = Holder3(dummy=13) holder.save() Inner3(dummy=42, holder=holder).save() - change_url = reverse('admin:admin_inlines_holder3_change', args=(holder.id,)) + change_url = reverse("admin:admin_inlines_holder3_change", args=(holder.id,)) response = self.client.get(change_url) self.assertEqual( - response.context['inline_admin_formsets'][0].media._js, + response.context["inline_admin_formsets"][0].media._js, [ - 'admin/js/vendor/jquery/jquery.min.js', - 'my_awesome_inline_scripts.js', - 'custom_number.js', - 'admin/js/jquery.init.js', - 'admin/js/inlines.js', - ] + "admin/js/vendor/jquery/jquery.min.js", + "my_awesome_inline_scripts.js", + "custom_number.js", + "admin/js/jquery.init.js", + "admin/js/inlines.js", + ], ) - self.assertContains(response, 'my_awesome_inline_scripts.js') + self.assertContains(response, "my_awesome_inline_scripts.js") def test_all_inline_media(self): holder = Holder2(dummy=13) holder.save() Inner2(dummy=42, holder=holder).save() - change_url = reverse('admin:admin_inlines_holder2_change', args=(holder.id,)) + change_url = reverse("admin:admin_inlines_holder2_change", args=(holder.id,)) response = self.client.get(change_url) - self.assertContains(response, 'my_awesome_admin_scripts.js') - self.assertContains(response, 'my_awesome_inline_scripts.js') + self.assertContains(response, "my_awesome_admin_scripts.js") + self.assertContains(response, "my_awesome_inline_scripts.js") -@override_settings(ROOT_URLCONF='admin_inlines.urls') +@override_settings(ROOT_URLCONF="admin_inlines.urls") class TestInlineAdminForm(TestCase): - def test_immutable_content_type(self): """Regression for #9362 The problem depends only on InlineAdminForm and its "original" @@ -686,46 +786,48 @@ class TestInlineAdminForm(TestCase): need to check that the content_type argument of Child isn't altered by the internals of the inline form.""" - sally = Teacher.objects.create(name='Sally') - john = Parent.objects.create(name='John') - joe = Child.objects.create(name='Joe', teacher=sally, parent=john) + sally = Teacher.objects.create(name="Sally") + john = Parent.objects.create(name="John") + joe = Child.objects.create(name="Joe", teacher=sally, parent=john) iaf = InlineAdminForm(None, None, {}, {}, joe) parent_ct = ContentType.objects.get_for_model(Parent) self.assertEqual(iaf.original.content_type, parent_ct) -@override_settings(ROOT_URLCONF='admin_inlines.urls') +@override_settings(ROOT_URLCONF="admin_inlines.urls") class TestInlineProtectedOnDelete(TestDataMixin, TestCase): - def setUp(self): self.client.force_login(self.superuser) def test_deleting_inline_with_protected_delete_does_not_validate(self): - lotr = Novel.objects.create(name='Lord of the rings') - chapter = Chapter.objects.create(novel=lotr, name='Many Meetings') - foot_note = FootNote.objects.create(chapter=chapter, note='yadda yadda') + lotr = Novel.objects.create(name="Lord of the rings") + chapter = Chapter.objects.create(novel=lotr, name="Many Meetings") + foot_note = FootNote.objects.create(chapter=chapter, note="yadda yadda") - change_url = reverse('admin:admin_inlines_novel_change', args=(lotr.id,)) + change_url = reverse("admin:admin_inlines_novel_change", args=(lotr.id,)) response = self.client.get(change_url) data = { - 'name': lotr.name, - 'chapter_set-TOTAL_FORMS': 1, - 'chapter_set-INITIAL_FORMS': 1, - 'chapter_set-MAX_NUM_FORMS': 1000, - '_save': 'Save', - 'chapter_set-0-id': chapter.id, - 'chapter_set-0-name': chapter.name, - 'chapter_set-0-novel': lotr.id, - 'chapter_set-0-DELETE': 'on' + "name": lotr.name, + "chapter_set-TOTAL_FORMS": 1, + "chapter_set-INITIAL_FORMS": 1, + "chapter_set-MAX_NUM_FORMS": 1000, + "_save": "Save", + "chapter_set-0-id": chapter.id, + "chapter_set-0-name": chapter.name, + "chapter_set-0-novel": lotr.id, + "chapter_set-0-DELETE": "on", } response = self.client.post(change_url, data) - self.assertContains(response, "Deleting chapter %s would require deleting " - "the following protected related objects: foot note %s" - % (chapter, foot_note)) + self.assertContains( + response, + "Deleting chapter %s would require deleting " + "the following protected related objects: foot note %s" + % (chapter, foot_note), + ) -@override_settings(ROOT_URLCONF='admin_inlines.urls') +@override_settings(ROOT_URLCONF="admin_inlines.urls") class TestInlinePermissions(TestCase): """ Make sure the admin respects permissions for objects that are edited @@ -734,8 +836,8 @@ class TestInlinePermissions(TestCase): @classmethod def setUpTestData(cls): - cls.user = User(username='admin', is_staff=True, is_active=True) - cls.user.set_password('secret') + cls.user = User(username="admin", is_staff=True, is_active=True) + cls.user.set_password("secret") cls.user.save() cls.author_ct = ContentType.objects.get_for_model(Author) @@ -746,115 +848,159 @@ class TestInlinePermissions(TestCase): # User always has permissions to add and change Authors, and Holders, # the main (parent) models of the inlines. Permissions on the inlines # vary per test. - permission = Permission.objects.get(codename='add_author', content_type=cls.author_ct) + permission = Permission.objects.get( + codename="add_author", content_type=cls.author_ct + ) cls.user.user_permissions.add(permission) - permission = Permission.objects.get(codename='change_author', content_type=cls.author_ct) + permission = Permission.objects.get( + codename="change_author", content_type=cls.author_ct + ) cls.user.user_permissions.add(permission) - permission = Permission.objects.get(codename='add_holder2', content_type=cls.holder_ct) + permission = Permission.objects.get( + codename="add_holder2", content_type=cls.holder_ct + ) cls.user.user_permissions.add(permission) - permission = Permission.objects.get(codename='change_holder2', content_type=cls.holder_ct) + permission = Permission.objects.get( + codename="change_holder2", content_type=cls.holder_ct + ) cls.user.user_permissions.add(permission) - author = Author.objects.create(pk=1, name='The Author') - cls.book = author.books.create(name='The inline Book') - cls.author_change_url = reverse('admin:admin_inlines_author_change', args=(author.id,)) + author = Author.objects.create(pk=1, name="The Author") + cls.book = author.books.create(name="The inline Book") + cls.author_change_url = reverse( + "admin:admin_inlines_author_change", args=(author.id,) + ) # Get the ID of the automatically created intermediate model for the Author-Book m2m - author_book_auto_m2m_intermediate = Author.books.through.objects.get(author=author, book=cls.book) + author_book_auto_m2m_intermediate = Author.books.through.objects.get( + author=author, book=cls.book + ) cls.author_book_auto_m2m_intermediate_id = author_book_auto_m2m_intermediate.pk cls.holder = Holder2.objects.create(dummy=13) cls.inner2 = Inner2.objects.create(dummy=42, holder=cls.holder) def setUp(self): - self.holder_change_url = reverse('admin:admin_inlines_holder2_change', args=(self.holder.id,)) + self.holder_change_url = reverse( + "admin:admin_inlines_holder2_change", args=(self.holder.id,) + ) self.client.force_login(self.user) def test_inline_add_m2m_noperm(self): - response = self.client.get(reverse('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, '<h2>Author-book relationships</h2>') - self.assertNotContains(response, 'Add another Author-Book Relationship') + self.assertNotContains(response, "<h2>Author-book relationships</h2>") + 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(reverse('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, '<h2>Inner2s</h2>') - self.assertNotContains(response, 'Add another Inner2') + self.assertNotContains(response, "<h2>Inner2s</h2>") + self.assertNotContains(response, "Add another Inner2") self.assertNotContains(response, 'id="id_inner2_set-TOTAL_FORMS"') def test_inline_change_m2m_noperm(self): response = self.client.get(self.author_change_url) # No change permission on books, so no inline - self.assertNotContains(response, '<h2>Author-book relationships</h2>') - self.assertNotContains(response, 'Add another Author-Book Relationship') + self.assertNotContains(response, "<h2>Author-book relationships</h2>") + self.assertNotContains(response, "Add another Author-Book Relationship") self.assertNotContains(response, 'id="id_Author_books-TOTAL_FORMS"') def test_inline_change_fk_noperm(self): response = self.client.get(self.holder_change_url) # No permissions on Inner2s, so no inline - self.assertNotContains(response, '<h2>Inner2s</h2>') - self.assertNotContains(response, 'Add another Inner2') + self.assertNotContains(response, "<h2>Inner2s</h2>") + self.assertNotContains(response, "Add another Inner2") self.assertNotContains(response, 'id="id_inner2_set-TOTAL_FORMS"') def test_inline_add_m2m_view_only_perm(self): - permission = Permission.objects.get(codename='view_book', content_type=self.book_ct) + permission = Permission.objects.get( + codename="view_book", content_type=self.book_ct + ) self.user.user_permissions.add(permission) - response = self.client.get(reverse('admin:admin_inlines_author_add')) + response = self.client.get(reverse("admin:admin_inlines_author_add")) # View-only inlines. (It could be nicer to hide the empty, non-editable # inlines on the add page.) - self.assertIs(response.context['inline_admin_formset'].has_view_permission, True) - self.assertIs(response.context['inline_admin_formset'].has_add_permission, False) - self.assertIs(response.context['inline_admin_formset'].has_change_permission, False) - self.assertIs(response.context['inline_admin_formset'].has_delete_permission, False) - self.assertContains(response, '<h2>Author-book relationships</h2>') + self.assertIs( + response.context["inline_admin_formset"].has_view_permission, True + ) + self.assertIs( + response.context["inline_admin_formset"].has_add_permission, False + ) + self.assertIs( + response.context["inline_admin_formset"].has_change_permission, False + ) + self.assertIs( + response.context["inline_admin_formset"].has_delete_permission, False + ) + self.assertContains(response, "<h2>Author-book relationships</h2>") self.assertContains( response, '<input type="hidden" name="Author_books-TOTAL_FORMS" value="0" ' 'id="id_Author_books-TOTAL_FORMS">', html=True, ) - self.assertNotContains(response, 'Add another Author-Book Relationship') + self.assertNotContains(response, "Add another Author-Book Relationship") def test_inline_add_m2m_add_perm(self): - permission = Permission.objects.get(codename='add_book', content_type=self.book_ct) + permission = Permission.objects.get( + codename="add_book", content_type=self.book_ct + ) self.user.user_permissions.add(permission) - response = self.client.get(reverse('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, '<h2>Author-book relationships</h2>') - self.assertNotContains(response, 'Add another Author-Book Relationship') + self.assertNotContains(response, "<h2>Author-book relationships</h2>") + self.assertNotContains(response, "Add another Author-Book Relationship") self.assertNotContains(response, 'id="id_Author_books-TOTAL_FORMS"') def test_inline_add_fk_add_perm(self): - permission = Permission.objects.get(codename='add_inner2', content_type=self.inner_ct) + permission = Permission.objects.get( + codename="add_inner2", content_type=self.inner_ct + ) self.user.user_permissions.add(permission) - response = self.client.get(reverse('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, '<h2>Inner2s</h2>') - self.assertContains(response, 'Add another Inner2') - self.assertContains(response, '<input type="hidden" id="id_inner2_set-TOTAL_FORMS" ' - 'value="3" name="inner2_set-TOTAL_FORMS">', html=True) + self.assertContains(response, "<h2>Inner2s</h2>") + self.assertContains(response, "Add another Inner2") + self.assertContains( + response, + '<input type="hidden" id="id_inner2_set-TOTAL_FORMS" ' + 'value="3" name="inner2_set-TOTAL_FORMS">', + html=True, + ) def test_inline_change_m2m_add_perm(self): - permission = Permission.objects.get(codename='add_book', content_type=self.book_ct) + permission = Permission.objects.get( + codename="add_book", content_type=self.book_ct + ) self.user.user_permissions.add(permission) response = self.client.get(self.author_change_url) # No change permission on books, so no inline - self.assertNotContains(response, '<h2>Author-book relationships</h2>') - self.assertNotContains(response, 'Add another Author-Book Relationship') + self.assertNotContains(response, "<h2>Author-book relationships</h2>") + self.assertNotContains(response, "Add another Author-Book Relationship") self.assertNotContains(response, 'id="id_Author_books-TOTAL_FORMS"') self.assertNotContains(response, 'id="id_Author_books-0-DELETE"') def test_inline_change_m2m_view_only_perm(self): - permission = Permission.objects.get(codename='view_book', content_type=self.book_ct) + permission = Permission.objects.get( + codename="view_book", content_type=self.book_ct + ) self.user.user_permissions.add(permission) response = self.client.get(self.author_change_url) # View-only inlines. - self.assertIs(response.context['inline_admin_formset'].has_view_permission, True) - self.assertIs(response.context['inline_admin_formset'].has_add_permission, False) - self.assertIs(response.context['inline_admin_formset'].has_change_permission, False) - self.assertIs(response.context['inline_admin_formset'].has_delete_permission, False) - self.assertContains(response, '<h2>Author-book relationships</h2>') + self.assertIs( + response.context["inline_admin_formset"].has_view_permission, True + ) + self.assertIs( + response.context["inline_admin_formset"].has_add_permission, False + ) + self.assertIs( + response.context["inline_admin_formset"].has_change_permission, False + ) + self.assertIs( + response.context["inline_admin_formset"].has_delete_permission, False + ) + self.assertContains(response, "<h2>Author-book relationships</h2>") self.assertContains( response, '<input type="hidden" name="Author_books-TOTAL_FORMS" value="1" ' @@ -862,7 +1008,7 @@ class TestInlinePermissions(TestCase): html=True, ) # The field in the inline is read-only. - self.assertContains(response, '<p>%s</p>' % self.book) + self.assertContains(response, "<p>%s</p>" % self.book) self.assertNotContains( response, '<input type="checkbox" name="Author_books-0-DELETE" id="id_Author_books-0-DELETE">', @@ -870,70 +1016,91 @@ class TestInlinePermissions(TestCase): ) def test_inline_change_m2m_change_perm(self): - permission = Permission.objects.get(codename='change_book', content_type=self.book_ct) + permission = Permission.objects.get( + codename="change_book", content_type=self.book_ct + ) self.user.user_permissions.add(permission) response = self.client.get(self.author_change_url) # We have change perm on books, so we can add/change/delete inlines - self.assertIs(response.context['inline_admin_formset'].has_view_permission, True) - self.assertIs(response.context['inline_admin_formset'].has_add_permission, True) - self.assertIs(response.context['inline_admin_formset'].has_change_permission, True) - self.assertIs(response.context['inline_admin_formset'].has_delete_permission, True) - self.assertContains(response, '<h2>Author-book relationships</h2>') - self.assertContains(response, 'Add another Author-book relationship') - self.assertContains(response, '<input type="hidden" id="id_Author_books-TOTAL_FORMS" ' - 'value="4" name="Author_books-TOTAL_FORMS">', html=True) + self.assertIs( + response.context["inline_admin_formset"].has_view_permission, True + ) + self.assertIs(response.context["inline_admin_formset"].has_add_permission, True) + self.assertIs( + response.context["inline_admin_formset"].has_change_permission, True + ) + self.assertIs( + response.context["inline_admin_formset"].has_delete_permission, True + ) + self.assertContains(response, "<h2>Author-book relationships</h2>") + self.assertContains(response, "Add another Author-book relationship") + self.assertContains( + response, + '<input type="hidden" id="id_Author_books-TOTAL_FORMS" ' + 'value="4" name="Author_books-TOTAL_FORMS">', + html=True, + ) self.assertContains( response, '<input type="hidden" id="id_Author_books-0-id" value="%i" ' 'name="Author_books-0-id">' % self.author_book_auto_m2m_intermediate_id, - html=True + html=True, ) self.assertContains(response, 'id="id_Author_books-0-DELETE"') def test_inline_change_fk_add_perm(self): - permission = Permission.objects.get(codename='add_inner2', content_type=self.inner_ct) + permission = Permission.objects.get( + codename="add_inner2", content_type=self.inner_ct + ) self.user.user_permissions.add(permission) response = self.client.get(self.holder_change_url) # Add permission on inner2s, so we can add but not modify existing - self.assertContains(response, '<h2>Inner2s</h2>') - self.assertContains(response, 'Add another Inner2') + self.assertContains(response, "<h2>Inner2s</h2>") + self.assertContains(response, "Add another Inner2") # 3 extra forms only, not the existing instance form self.assertContains( response, '<input type="hidden" id="id_inner2_set-TOTAL_FORMS" value="3" ' 'name="inner2_set-TOTAL_FORMS">', - html=True + html=True, ) self.assertNotContains( response, - '<input type="hidden" id="id_inner2_set-0-id" value="%i" name="inner2_set-0-id">' % self.inner2.id, - html=True + '<input type="hidden" id="id_inner2_set-0-id" value="%i" name="inner2_set-0-id">' + % self.inner2.id, + html=True, ) def test_inline_change_fk_change_perm(self): - permission = Permission.objects.get(codename='change_inner2', content_type=self.inner_ct) + permission = Permission.objects.get( + codename="change_inner2", content_type=self.inner_ct + ) self.user.user_permissions.add(permission) response = self.client.get(self.holder_change_url) # Change permission on inner2s, so we can change existing but not add new - self.assertContains(response, '<h2>Inner2s</h2>', count=2) + self.assertContains(response, "<h2>Inner2s</h2>", count=2) # Just the one form for existing instances self.assertContains( - response, '<input type="hidden" id="id_inner2_set-TOTAL_FORMS" value="1" name="inner2_set-TOTAL_FORMS">', - html=True + response, + '<input type="hidden" id="id_inner2_set-TOTAL_FORMS" value="1" name="inner2_set-TOTAL_FORMS">', + html=True, ) self.assertContains( response, - '<input type="hidden" id="id_inner2_set-0-id" value="%i" name="inner2_set-0-id">' % self.inner2.id, - html=True + '<input type="hidden" id="id_inner2_set-0-id" value="%i" name="inner2_set-0-id">' + % self.inner2.id, + html=True, ) # max-num 0 means we can't add new ones self.assertContains( response, '<input type="hidden" id="id_inner2_set-MAX_NUM_FORMS" value="0" name="inner2_set-MAX_NUM_FORMS">', - html=True + html=True, ) # TabularInline - self.assertContains(response, '<th class="column-dummy required">Dummy</th>', html=True) + self.assertContains( + response, '<th class="column-dummy required">Dummy</th>', html=True + ) self.assertContains( response, '<input type="number" name="inner2_set-2-0-dummy" value="%s" ' @@ -942,69 +1109,89 @@ class TestInlinePermissions(TestCase): ) def test_inline_change_fk_add_change_perm(self): - permission = Permission.objects.get(codename='add_inner2', content_type=self.inner_ct) + permission = Permission.objects.get( + codename="add_inner2", content_type=self.inner_ct + ) self.user.user_permissions.add(permission) - permission = Permission.objects.get(codename='change_inner2', content_type=self.inner_ct) + permission = Permission.objects.get( + codename="change_inner2", content_type=self.inner_ct + ) self.user.user_permissions.add(permission) response = self.client.get(self.holder_change_url) # Add/change perm, so we can add new and change existing - self.assertContains(response, '<h2>Inner2s</h2>') + self.assertContains(response, "<h2>Inner2s</h2>") # One form for existing instance and three extra for new self.assertContains( - response, '<input type="hidden" id="id_inner2_set-TOTAL_FORMS" value="4" name="inner2_set-TOTAL_FORMS">', - html=True + response, + '<input type="hidden" id="id_inner2_set-TOTAL_FORMS" value="4" name="inner2_set-TOTAL_FORMS">', + html=True, ) self.assertContains( response, - '<input type="hidden" id="id_inner2_set-0-id" value="%i" name="inner2_set-0-id">' % self.inner2.id, - html=True + '<input type="hidden" id="id_inner2_set-0-id" value="%i" name="inner2_set-0-id">' + % self.inner2.id, + html=True, ) def test_inline_change_fk_change_del_perm(self): - permission = Permission.objects.get(codename='change_inner2', content_type=self.inner_ct) + permission = Permission.objects.get( + codename="change_inner2", content_type=self.inner_ct + ) self.user.user_permissions.add(permission) - permission = Permission.objects.get(codename='delete_inner2', content_type=self.inner_ct) + permission = Permission.objects.get( + codename="delete_inner2", content_type=self.inner_ct + ) self.user.user_permissions.add(permission) response = self.client.get(self.holder_change_url) # Change/delete perm on inner2s, so we can change/delete existing - self.assertContains(response, '<h2>Inner2s</h2>') + self.assertContains(response, "<h2>Inner2s</h2>") # One form for existing instance only, no new self.assertContains( response, '<input type="hidden" id="id_inner2_set-TOTAL_FORMS" value="1" name="inner2_set-TOTAL_FORMS">', - html=True + html=True, ) self.assertContains( response, - '<input type="hidden" id="id_inner2_set-0-id" value="%i" name="inner2_set-0-id">' % self.inner2.id, - html=True + '<input type="hidden" id="id_inner2_set-0-id" value="%i" name="inner2_set-0-id">' + % self.inner2.id, + html=True, ) self.assertContains(response, 'id="id_inner2_set-0-DELETE"') def test_inline_change_fk_all_perms(self): - permission = Permission.objects.get(codename='add_inner2', content_type=self.inner_ct) + permission = Permission.objects.get( + codename="add_inner2", content_type=self.inner_ct + ) self.user.user_permissions.add(permission) - permission = Permission.objects.get(codename='change_inner2', content_type=self.inner_ct) + permission = Permission.objects.get( + codename="change_inner2", content_type=self.inner_ct + ) self.user.user_permissions.add(permission) - permission = Permission.objects.get(codename='delete_inner2', content_type=self.inner_ct) + permission = Permission.objects.get( + codename="delete_inner2", content_type=self.inner_ct + ) self.user.user_permissions.add(permission) response = self.client.get(self.holder_change_url) # All perms on inner2s, so we can add/change/delete - self.assertContains(response, '<h2>Inner2s</h2>', count=2) + self.assertContains(response, "<h2>Inner2s</h2>", count=2) # One form for existing instance only, three for new self.assertContains( response, '<input type="hidden" id="id_inner2_set-TOTAL_FORMS" value="4" name="inner2_set-TOTAL_FORMS">', - html=True + html=True, ) self.assertContains( response, - '<input type="hidden" id="id_inner2_set-0-id" value="%i" name="inner2_set-0-id">' % self.inner2.id, - html=True + '<input type="hidden" id="id_inner2_set-0-id" value="%i" name="inner2_set-0-id">' + % self.inner2.id, + html=True, ) self.assertContains(response, 'id="id_inner2_set-0-DELETE"') # TabularInline - self.assertContains(response, '<th class="column-dummy required">Dummy</th>', html=True) + self.assertContains( + response, '<th class="column-dummy required">Dummy</th>', html=True + ) self.assertContains( response, '<input type="number" name="inner2_set-2-0-dummy" value="%s" ' @@ -1013,24 +1200,29 @@ class TestInlinePermissions(TestCase): ) -@override_settings(ROOT_URLCONF='admin_inlines.urls') +@override_settings(ROOT_URLCONF="admin_inlines.urls") class TestReadOnlyChangeViewInlinePermissions(TestCase): - @classmethod def setUpTestData(cls): - cls.user = User.objects.create_user('testing', password='password', is_staff=True) + cls.user = User.objects.create_user( + "testing", password="password", is_staff=True + ) cls.user.user_permissions.add( - Permission.objects.get(codename='view_poll', content_type=ContentType.objects.get_for_model(Poll)) + Permission.objects.get( + codename="view_poll", + content_type=ContentType.objects.get_for_model(Poll), + ) ) cls.user.user_permissions.add( *Permission.objects.filter( - codename__endswith="question", content_type=ContentType.objects.get_for_model(Question) - ).values_list('pk', flat=True) + codename__endswith="question", + content_type=ContentType.objects.get_for_model(Question), + ).values_list("pk", flat=True) ) cls.poll = Poll.objects.create(name="Survey") - cls.add_url = reverse('admin:admin_inlines_poll_add') - cls.change_url = reverse('admin:admin_inlines_poll_change', args=(cls.poll.id,)) + cls.add_url = reverse("admin:admin_inlines_poll_add") + cls.change_url = reverse("admin:admin_inlines_poll_change", args=(cls.poll.id,)) def setUp(self): self.client.force_login(self.user) @@ -1053,24 +1245,18 @@ class TestReadOnlyChangeViewInlinePermissions(TestCase): def test_main_model_is_rendered_as_read_only(self): response = self.client.get(self.change_url) self.assertContains( - response, - '<div class="readonly">%s</div>' % self.poll.name, - html=True + response, '<div class="readonly">%s</div>' % self.poll.name, html=True ) input = '<input type="text" name="name" value="%s" class="vTextField" maxlength="40" required id="id_name">' - self.assertNotContains( - response, - input % self.poll.name, - html=True - ) + self.assertNotContains(response, input % self.poll.name, html=True) def test_inlines_are_rendered_as_read_only(self): - question = Question.objects.create(text="How will this be rendered?", poll=self.poll) + question = Question.objects.create( + text="How will this be rendered?", poll=self.poll + ) response = self.client.get(self.change_url) self.assertContains( - response, - '<td class="field-text"><p>%s</p></td>' % question.text, - html=True + response, '<td class="field-text"><p>%s</p></td>' % question.text, html=True ) self.assertNotContains(response, 'id="id_question_set-0-text"') self.assertNotContains(response, 'id="id_related_objs-0-DELETE"') @@ -1080,16 +1266,18 @@ class TestReadOnlyChangeViewInlinePermissions(TestCase): self.assertContains( response, '<a href="/admin/admin_inlines/poll/" class="closelink">Close</a>', - html=True + html=True, ) delete_link = '<p class="deletelink-box"><a href="/admin/admin_inlines/poll/%s/delete/" class="deletelink">Delete</a></p>' # noqa + self.assertNotContains(response, delete_link % self.poll.id, html=True) + self.assertNotContains( + response, + '<input type="submit" value="Save and add another" name="_addanother">', + ) self.assertNotContains( response, - delete_link % self.poll.id, - html=True + '<input type="submit" value="Save and continue editing" name="_continue">', ) - self.assertNotContains(response, '<input type="submit" value="Save and add another" name="_addanother">') - self.assertNotContains(response, '<input type="submit" value="Save and continue editing" name="_continue">') def test_inline_delete_buttons_are_not_shown(self): Question.objects.create(text="How will this be rendered?", poll=self.poll) @@ -1097,7 +1285,7 @@ class TestReadOnlyChangeViewInlinePermissions(TestCase): self.assertNotContains( response, '<input type="checkbox" name="question_set-0-DELETE" id="id_question_set-0-DELETE">', - html=True + html=True, ) def test_extra_inlines_are_not_shown(self): @@ -1105,26 +1293,26 @@ class TestReadOnlyChangeViewInlinePermissions(TestCase): self.assertNotContains(response, 'id="id_question_set-0-text"') -@override_settings(ROOT_URLCONF='admin_inlines.urls') +@override_settings(ROOT_URLCONF="admin_inlines.urls") class TestVerboseNameInlineForms(TestDataMixin, TestCase): factory = RequestFactory() def test_verbose_name_inline(self): class NonVerboseProfileInline(TabularInline): model = Profile - verbose_name = 'Non-verbose childs' + verbose_name = "Non-verbose childs" class VerboseNameProfileInline(TabularInline): model = VerboseNameProfile - verbose_name = 'Childs with verbose name' + verbose_name = "Childs with verbose name" class VerboseNamePluralProfileInline(TabularInline): model = VerboseNamePluralProfile - verbose_name = 'Childs with verbose name plural' + verbose_name = "Childs with verbose name plural" class BothVerboseNameProfileInline(TabularInline): model = BothVerboseNameProfile - verbose_name = 'Childs with both verbose names' + verbose_name = "Childs with both verbose names" modeladmin = ModelAdmin(ProfileCollection, admin_site) modeladmin.inlines = [ @@ -1134,46 +1322,46 @@ class TestVerboseNameInlineForms(TestDataMixin, TestCase): BothVerboseNameProfileInline, ] obj = ProfileCollection.objects.create() - url = reverse('admin:admin_inlines_profilecollection_change', args=(obj.pk,)) + url = reverse("admin:admin_inlines_profilecollection_change", args=(obj.pk,)) request = self.factory.get(url) request.user = self.superuser response = modeladmin.changeform_view(request) - self.assertNotContains(response, 'Add another Profile') + self.assertNotContains(response, "Add another Profile") # Non-verbose model. - self.assertContains(response, '<h2>Non-verbose childss</h2>') - self.assertContains(response, 'Add another Non-verbose child') - self.assertNotContains(response, '<h2>Profiles</h2>') + self.assertContains(response, "<h2>Non-verbose childss</h2>") + self.assertContains(response, "Add another Non-verbose child") + self.assertNotContains(response, "<h2>Profiles</h2>") # Model with verbose name. - self.assertContains(response, '<h2>Childs with verbose names</h2>') - self.assertContains(response, 'Add another Childs with verbose name') - self.assertNotContains(response, '<h2>Model with verbose name onlys</h2>') - self.assertNotContains(response, 'Add another Model with verbose name only') + self.assertContains(response, "<h2>Childs with verbose names</h2>") + self.assertContains(response, "Add another Childs with verbose name") + self.assertNotContains(response, "<h2>Model with verbose name onlys</h2>") + self.assertNotContains(response, "Add another Model with verbose name only") # Model with verbose name plural. - self.assertContains(response, '<h2>Childs with verbose name plurals</h2>') - self.assertContains(response, 'Add another Childs with verbose name plural') - self.assertNotContains(response, '<h2>Model with verbose name plural only</h2>') + self.assertContains(response, "<h2>Childs with verbose name plurals</h2>") + self.assertContains(response, "Add another Childs with verbose name plural") + self.assertNotContains(response, "<h2>Model with verbose name plural only</h2>") # Model with both verbose names. - self.assertContains(response, '<h2>Childs with both verbose namess</h2>') - self.assertContains(response, 'Add another Childs with both verbose names') - self.assertNotContains(response, '<h2>Model with both - plural name</h2>') - self.assertNotContains(response, 'Add another Model with both - name') + self.assertContains(response, "<h2>Childs with both verbose namess</h2>") + self.assertContains(response, "Add another Childs with both verbose names") + self.assertNotContains(response, "<h2>Model with both - plural name</h2>") + self.assertNotContains(response, "Add another Model with both - name") def test_verbose_name_plural_inline(self): class NonVerboseProfileInline(TabularInline): model = Profile - verbose_name_plural = 'Non-verbose childs' + verbose_name_plural = "Non-verbose childs" class VerboseNameProfileInline(TabularInline): model = VerboseNameProfile - verbose_name_plural = 'Childs with verbose name' + verbose_name_plural = "Childs with verbose name" class VerboseNamePluralProfileInline(TabularInline): model = VerboseNamePluralProfile - verbose_name_plural = 'Childs with verbose name plural' + verbose_name_plural = "Childs with verbose name plural" class BothVerboseNameProfileInline(TabularInline): model = BothVerboseNameProfile - verbose_name_plural = 'Childs with both verbose names' + verbose_name_plural = "Childs with both verbose names" modeladmin = ModelAdmin(ProfileCollection, admin_site) modeladmin.inlines = [ @@ -1183,47 +1371,47 @@ class TestVerboseNameInlineForms(TestDataMixin, TestCase): BothVerboseNameProfileInline, ] obj = ProfileCollection.objects.create() - url = reverse('admin:admin_inlines_profilecollection_change', args=(obj.pk,)) + url = reverse("admin:admin_inlines_profilecollection_change", args=(obj.pk,)) request = self.factory.get(url) request.user = self.superuser response = modeladmin.changeform_view(request) # Non-verbose model. - self.assertContains(response, '<h2>Non-verbose childs</h2>') - self.assertContains(response, 'Add another Profile') - self.assertNotContains(response, '<h2>Profiles</h2>') + self.assertContains(response, "<h2>Non-verbose childs</h2>") + self.assertContains(response, "Add another Profile") + self.assertNotContains(response, "<h2>Profiles</h2>") # Model with verbose name. - self.assertContains(response, '<h2>Childs with verbose name</h2>') - self.assertContains(response, 'Add another Model with verbose name only') - self.assertNotContains(response, '<h2>Model with verbose name onlys</h2>') + self.assertContains(response, "<h2>Childs with verbose name</h2>") + self.assertContains(response, "Add another Model with verbose name only") + self.assertNotContains(response, "<h2>Model with verbose name onlys</h2>") # Model with verbose name plural. - self.assertContains(response, '<h2>Childs with verbose name plural</h2>') - self.assertContains(response, 'Add another Profile') - self.assertNotContains(response, '<h2>Model with verbose name plural only</h2>') + self.assertContains(response, "<h2>Childs with verbose name plural</h2>") + self.assertContains(response, "Add another Profile") + self.assertNotContains(response, "<h2>Model with verbose name plural only</h2>") # Model with both verbose names. - self.assertContains(response, '<h2>Childs with both verbose names</h2>') - self.assertContains(response, 'Add another Model with both - name') - self.assertNotContains(response, '<h2>Model with both - plural name</h2>') + self.assertContains(response, "<h2>Childs with both verbose names</h2>") + self.assertContains(response, "Add another Model with both - name") + self.assertNotContains(response, "<h2>Model with both - plural name</h2>") def test_both_verbose_names_inline(self): class NonVerboseProfileInline(TabularInline): model = Profile - verbose_name = 'Non-verbose childs - name' - verbose_name_plural = 'Non-verbose childs - plural name' + verbose_name = "Non-verbose childs - name" + verbose_name_plural = "Non-verbose childs - plural name" class VerboseNameProfileInline(TabularInline): model = VerboseNameProfile - verbose_name = 'Childs with verbose name - name' - verbose_name_plural = 'Childs with verbose name - plural name' + verbose_name = "Childs with verbose name - name" + verbose_name_plural = "Childs with verbose name - plural name" class VerboseNamePluralProfileInline(TabularInline): model = VerboseNamePluralProfile - verbose_name = 'Childs with verbose name plural - name' - verbose_name_plural = 'Childs with verbose name plural - plural name' + verbose_name = "Childs with verbose name plural - name" + verbose_name_plural = "Childs with verbose name plural - plural name" class BothVerboseNameProfileInline(TabularInline): model = BothVerboseNameProfile - verbose_name = 'Childs with both - name' - verbose_name_plural = 'Childs with both - plural name' + verbose_name = "Childs with both - name" + verbose_name_plural = "Childs with both - plural name" modeladmin = ModelAdmin(ProfileCollection, admin_site) modeladmin.inlines = [ @@ -1233,76 +1421,90 @@ class TestVerboseNameInlineForms(TestDataMixin, TestCase): BothVerboseNameProfileInline, ] obj = ProfileCollection.objects.create() - url = reverse('admin:admin_inlines_profilecollection_change', args=(obj.pk,)) + url = reverse("admin:admin_inlines_profilecollection_change", args=(obj.pk,)) request = self.factory.get(url) request.user = self.superuser response = modeladmin.changeform_view(request) - self.assertNotContains(response, 'Add another Profile') + self.assertNotContains(response, "Add another Profile") # Non-verbose model. - self.assertContains(response, '<h2>Non-verbose childs - plural name</h2>') - self.assertContains(response, 'Add another Non-verbose childs - name') - self.assertNotContains(response, '<h2>Profiles</h2>') + self.assertContains(response, "<h2>Non-verbose childs - plural name</h2>") + self.assertContains(response, "Add another Non-verbose childs - name") + self.assertNotContains(response, "<h2>Profiles</h2>") # Model with verbose name. - self.assertContains(response, '<h2>Childs with verbose name - plural name</h2>') - self.assertContains(response, 'Add another Childs with verbose name - name') - self.assertNotContains(response, '<h2>Model with verbose name onlys</h2>') + self.assertContains(response, "<h2>Childs with verbose name - plural name</h2>") + self.assertContains(response, "Add another Childs with verbose name - name") + self.assertNotContains(response, "<h2>Model with verbose name onlys</h2>") # Model with verbose name plural. self.assertContains( response, - '<h2>Childs with verbose name plural - plural name</h2>', + "<h2>Childs with verbose name plural - plural name</h2>", ) self.assertContains( response, - 'Add another Childs with verbose name plural - name', + "Add another Childs with verbose name plural - name", ) - self.assertNotContains(response, '<h2>Model with verbose name plural only</h2>') + self.assertNotContains(response, "<h2>Model with verbose name plural only</h2>") # Model with both verbose names. - self.assertContains(response, '<h2>Childs with both - plural name</h2>') - self.assertContains(response, 'Add another Childs with both - name') - self.assertNotContains(response, '<h2>Model with both - plural name</h2>') - self.assertNotContains(response, 'Add another Model with both - name') + self.assertContains(response, "<h2>Childs with both - plural name</h2>") + self.assertContains(response, "Add another Childs with both - name") + self.assertNotContains(response, "<h2>Model with both - plural name</h2>") + self.assertNotContains(response, "Add another Model with both - name") -@override_settings(ROOT_URLCONF='admin_inlines.urls') +@override_settings(ROOT_URLCONF="admin_inlines.urls") class SeleniumTests(AdminSeleniumTestCase): - available_apps = ['admin_inlines'] + AdminSeleniumTestCase.available_apps + available_apps = ["admin_inlines"] + AdminSeleniumTestCase.available_apps def setUp(self): - User.objects.create_superuser(username='super', password='secret', email='super@example.com') + User.objects.create_superuser( + username="super", password="secret", email="super@example.com" + ) def test_add_stackeds(self): """ The "Add another XXX" link correctly adds items to the stacked formset. """ from selenium.webdriver.common.by import By - self.admin_login(username='super', password='secret') - self.selenium.get(self.live_server_url + reverse('admin:admin_inlines_holder4_add')) - inline_id = '#inner4stacked_set-group' - rows_selector = '%s .dynamic-inner4stacked_set' % inline_id + self.admin_login(username="super", password="secret") + self.selenium.get( + self.live_server_url + reverse("admin:admin_inlines_holder4_add") + ) + + inline_id = "#inner4stacked_set-group" + rows_selector = "%s .dynamic-inner4stacked_set" % inline_id self.assertCountSeleniumElements(rows_selector, 3) - add_button = self.selenium.find_element(By.LINK_TEXT, 'Add another Inner4 stacked') + add_button = self.selenium.find_element( + By.LINK_TEXT, "Add another Inner4 stacked" + ) add_button.click() self.assertCountSeleniumElements(rows_selector, 4) def test_delete_stackeds(self): from selenium.webdriver.common.by import By - self.admin_login(username='super', password='secret') - self.selenium.get(self.live_server_url + reverse('admin:admin_inlines_holder4_add')) - inline_id = '#inner4stacked_set-group' - rows_selector = '%s .dynamic-inner4stacked_set' % inline_id + self.admin_login(username="super", password="secret") + self.selenium.get( + self.live_server_url + reverse("admin:admin_inlines_holder4_add") + ) + + inline_id = "#inner4stacked_set-group" + rows_selector = "%s .dynamic-inner4stacked_set" % inline_id self.assertCountSeleniumElements(rows_selector, 3) - add_button = self.selenium.find_element(By.LINK_TEXT, 'Add another Inner4 stacked') + add_button = self.selenium.find_element( + By.LINK_TEXT, "Add another Inner4 stacked" + ) add_button.click() add_button.click() self.assertCountSeleniumElements(rows_selector, 5) - for delete_link in self.selenium.find_elements(By.CSS_SELECTOR, '%s .inline-deletelink' % inline_id): + for delete_link in self.selenium.find_elements( + By.CSS_SELECTOR, "%s .inline-deletelink" % inline_id + ): delete_link.click() with self.disable_implicit_wait(): self.assertCountSeleniumElements(rows_selector, 0) @@ -1310,29 +1512,42 @@ class SeleniumTests(AdminSeleniumTestCase): def test_delete_invalid_stacked_inlines(self): from selenium.common.exceptions import NoSuchElementException from selenium.webdriver.common.by import By - self.admin_login(username='super', password='secret') - self.selenium.get(self.live_server_url + reverse('admin:admin_inlines_holder4_add')) - inline_id = '#inner4stacked_set-group' - rows_selector = '%s .dynamic-inner4stacked_set' % inline_id + self.admin_login(username="super", password="secret") + self.selenium.get( + self.live_server_url + reverse("admin:admin_inlines_holder4_add") + ) + + inline_id = "#inner4stacked_set-group" + rows_selector = "%s .dynamic-inner4stacked_set" % inline_id self.assertCountSeleniumElements(rows_selector, 3) add_button = self.selenium.find_element( By.LINK_TEXT, - 'Add another Inner4 stacked', + "Add another Inner4 stacked", ) add_button.click() add_button.click() - self.assertCountSeleniumElements('#id_inner4stacked_set-4-dummy', 1) + self.assertCountSeleniumElements("#id_inner4stacked_set-4-dummy", 1) # Enter some data and click 'Save'. - self.selenium.find_element(By.NAME, 'dummy').send_keys('1') - self.selenium.find_element(By.NAME, 'inner4stacked_set-0-dummy').send_keys('100') - self.selenium.find_element(By.NAME, 'inner4stacked_set-1-dummy').send_keys('101') - self.selenium.find_element(By.NAME, 'inner4stacked_set-2-dummy').send_keys('222') - self.selenium.find_element(By.NAME, 'inner4stacked_set-3-dummy').send_keys('103') - self.selenium.find_element(By.NAME, 'inner4stacked_set-4-dummy').send_keys('222') + self.selenium.find_element(By.NAME, "dummy").send_keys("1") + self.selenium.find_element(By.NAME, "inner4stacked_set-0-dummy").send_keys( + "100" + ) + self.selenium.find_element(By.NAME, "inner4stacked_set-1-dummy").send_keys( + "101" + ) + self.selenium.find_element(By.NAME, "inner4stacked_set-2-dummy").send_keys( + "222" + ) + self.selenium.find_element(By.NAME, "inner4stacked_set-3-dummy").send_keys( + "103" + ) + self.selenium.find_element(By.NAME, "inner4stacked_set-4-dummy").send_keys( + "222" + ) with self.wait_page_loaded(): self.selenium.find_element(By.XPATH, '//input[@value="Save"]').click() @@ -1340,14 +1555,19 @@ class SeleniumTests(AdminSeleniumTestCase): self.assertCountSeleniumElements(rows_selector, 5) errorlist = self.selenium.find_element( By.CSS_SELECTOR, - '%s .dynamic-inner4stacked_set .errorlist li' % inline_id, + "%s .dynamic-inner4stacked_set .errorlist li" % inline_id, + ) + self.assertEqual("Please correct the duplicate values below.", errorlist.text) + delete_link = self.selenium.find_element( + By.CSS_SELECTOR, "#inner4stacked_set-4 .inline-deletelink" ) - self.assertEqual('Please correct the duplicate values below.', errorlist.text) - delete_link = self.selenium.find_element(By.CSS_SELECTOR, '#inner4stacked_set-4 .inline-deletelink') delete_link.click() self.assertCountSeleniumElements(rows_selector, 4) with self.disable_implicit_wait(), self.assertRaises(NoSuchElementException): - self.selenium.find_element(By.CSS_SELECTOR, '%s .dynamic-inner4stacked_set .errorlist li' % inline_id) + self.selenium.find_element( + By.CSS_SELECTOR, + "%s .dynamic-inner4stacked_set .errorlist li" % inline_id, + ) with self.wait_page_loaded(): self.selenium.find_element(By.XPATH, '//input[@value="Save"]').click() @@ -1358,29 +1578,41 @@ class SeleniumTests(AdminSeleniumTestCase): def test_delete_invalid_tabular_inlines(self): from selenium.common.exceptions import NoSuchElementException from selenium.webdriver.common.by import By - self.admin_login(username='super', password='secret') - self.selenium.get(self.live_server_url + reverse('admin:admin_inlines_holder4_add')) - inline_id = '#inner4tabular_set-group' - rows_selector = '%s .dynamic-inner4tabular_set' % inline_id + self.admin_login(username="super", password="secret") + self.selenium.get( + self.live_server_url + reverse("admin:admin_inlines_holder4_add") + ) + + inline_id = "#inner4tabular_set-group" + rows_selector = "%s .dynamic-inner4tabular_set" % inline_id self.assertCountSeleniumElements(rows_selector, 3) add_button = self.selenium.find_element( - By.LINK_TEXT, - 'Add another Inner4 tabular' + By.LINK_TEXT, "Add another Inner4 tabular" ) add_button.click() add_button.click() - self.assertCountSeleniumElements('#id_inner4tabular_set-4-dummy', 1) + self.assertCountSeleniumElements("#id_inner4tabular_set-4-dummy", 1) # Enter some data and click 'Save'. - self.selenium.find_element(By.NAME, 'dummy').send_keys('1') - self.selenium.find_element(By.NAME, 'inner4tabular_set-0-dummy').send_keys('100') - self.selenium.find_element(By.NAME, 'inner4tabular_set-1-dummy').send_keys('101') - self.selenium.find_element(By.NAME, 'inner4tabular_set-2-dummy').send_keys('222') - self.selenium.find_element(By.NAME, 'inner4tabular_set-3-dummy').send_keys('103') - self.selenium.find_element(By.NAME, 'inner4tabular_set-4-dummy').send_keys('222') + self.selenium.find_element(By.NAME, "dummy").send_keys("1") + self.selenium.find_element(By.NAME, "inner4tabular_set-0-dummy").send_keys( + "100" + ) + self.selenium.find_element(By.NAME, "inner4tabular_set-1-dummy").send_keys( + "101" + ) + self.selenium.find_element(By.NAME, "inner4tabular_set-2-dummy").send_keys( + "222" + ) + self.selenium.find_element(By.NAME, "inner4tabular_set-3-dummy").send_keys( + "103" + ) + self.selenium.find_element(By.NAME, "inner4tabular_set-4-dummy").send_keys( + "222" + ) with self.wait_page_loaded(): self.selenium.find_element(By.XPATH, '//input[@value="Save"]').click() @@ -1391,15 +1623,20 @@ class SeleniumTests(AdminSeleniumTestCase): # tr#inner4tabular_set-3: errorlist = self.selenium.find_element( By.CSS_SELECTOR, - '%s #inner4tabular_set-3 + .row-form-errors .errorlist li' % inline_id + "%s #inner4tabular_set-3 + .row-form-errors .errorlist li" % inline_id, + ) + self.assertEqual("Please correct the duplicate values below.", errorlist.text) + delete_link = self.selenium.find_element( + By.CSS_SELECTOR, "#inner4tabular_set-4 .inline-deletelink" ) - self.assertEqual('Please correct the duplicate values below.', errorlist.text) - delete_link = self.selenium.find_element(By.CSS_SELECTOR, '#inner4tabular_set-4 .inline-deletelink') delete_link.click() self.assertCountSeleniumElements(rows_selector, 4) with self.disable_implicit_wait(), self.assertRaises(NoSuchElementException): - self.selenium.find_element(By.CSS_SELECTOR, '%s .dynamic-inner4tabular_set .errorlist li' % inline_id) + self.selenium.find_element( + By.CSS_SELECTOR, + "%s .dynamic-inner4tabular_set .errorlist li" % inline_id, + ) with self.wait_page_loaded(): self.selenium.find_element(By.XPATH, '//input[@value="Save"]').click() @@ -1412,47 +1649,80 @@ class SeleniumTests(AdminSeleniumTestCase): The "Add another XXX" link correctly adds items to the inline form. """ from selenium.webdriver.common.by import By - self.admin_login(username='super', password='secret') - self.selenium.get(self.live_server_url + reverse('admin:admin_inlines_profilecollection_add')) + + self.admin_login(username="super", password="secret") + self.selenium.get( + self.live_server_url + reverse("admin:admin_inlines_profilecollection_add") + ) # There's only one inline to start with and it has the correct ID. - self.assertCountSeleniumElements('.dynamic-profile_set', 1) + self.assertCountSeleniumElements(".dynamic-profile_set", 1) self.assertEqual( - self.selenium.find_elements(By.CSS_SELECTOR, '.dynamic-profile_set')[0].get_attribute('id'), - 'profile_set-0', + self.selenium.find_elements(By.CSS_SELECTOR, ".dynamic-profile_set")[ + 0 + ].get_attribute("id"), + "profile_set-0", + ) + self.assertCountSeleniumElements( + ".dynamic-profile_set#profile_set-0 input[name=profile_set-0-first_name]", 1 + ) + self.assertCountSeleniumElements( + ".dynamic-profile_set#profile_set-0 input[name=profile_set-0-last_name]", 1 ) - self.assertCountSeleniumElements('.dynamic-profile_set#profile_set-0 input[name=profile_set-0-first_name]', 1) - self.assertCountSeleniumElements('.dynamic-profile_set#profile_set-0 input[name=profile_set-0-last_name]', 1) # Add an inline - self.selenium.find_element(By.LINK_TEXT, 'Add another Profile').click() + self.selenium.find_element(By.LINK_TEXT, "Add another Profile").click() # The inline has been added, it has the right id, and it contains the # correct fields. - self.assertCountSeleniumElements('.dynamic-profile_set', 2) + self.assertCountSeleniumElements(".dynamic-profile_set", 2) self.assertEqual( - self.selenium.find_elements(By.CSS_SELECTOR, '.dynamic-profile_set')[1].get_attribute('id'), - 'profile_set-1', + self.selenium.find_elements(By.CSS_SELECTOR, ".dynamic-profile_set")[ + 1 + ].get_attribute("id"), + "profile_set-1", + ) + self.assertCountSeleniumElements( + ".dynamic-profile_set#profile_set-1 input[name=profile_set-1-first_name]", 1 + ) + self.assertCountSeleniumElements( + ".dynamic-profile_set#profile_set-1 input[name=profile_set-1-last_name]", 1 ) - self.assertCountSeleniumElements('.dynamic-profile_set#profile_set-1 input[name=profile_set-1-first_name]', 1) - self.assertCountSeleniumElements('.dynamic-profile_set#profile_set-1 input[name=profile_set-1-last_name]', 1) # Let's add another one to be sure - self.selenium.find_element(By.LINK_TEXT, 'Add another Profile').click() - self.assertCountSeleniumElements('.dynamic-profile_set', 3) + self.selenium.find_element(By.LINK_TEXT, "Add another Profile").click() + self.assertCountSeleniumElements(".dynamic-profile_set", 3) self.assertEqual( - self.selenium.find_elements(By.CSS_SELECTOR, '.dynamic-profile_set')[2].get_attribute('id'), - 'profile_set-2', + self.selenium.find_elements(By.CSS_SELECTOR, ".dynamic-profile_set")[ + 2 + ].get_attribute("id"), + "profile_set-2", + ) + self.assertCountSeleniumElements( + ".dynamic-profile_set#profile_set-2 input[name=profile_set-2-first_name]", 1 + ) + self.assertCountSeleniumElements( + ".dynamic-profile_set#profile_set-2 input[name=profile_set-2-last_name]", 1 ) - self.assertCountSeleniumElements('.dynamic-profile_set#profile_set-2 input[name=profile_set-2-first_name]', 1) - self.assertCountSeleniumElements('.dynamic-profile_set#profile_set-2 input[name=profile_set-2-last_name]', 1) # Enter some data and click 'Save' - self.selenium.find_element(By.NAME, 'profile_set-0-first_name').send_keys('0 first name 1') - self.selenium.find_element(By.NAME, 'profile_set-0-last_name').send_keys('0 last name 2') - self.selenium.find_element(By.NAME, 'profile_set-1-first_name').send_keys('1 first name 1') - self.selenium.find_element(By.NAME, 'profile_set-1-last_name').send_keys('1 last name 2') - self.selenium.find_element(By.NAME, 'profile_set-2-first_name').send_keys('2 first name 1') - self.selenium.find_element(By.NAME, 'profile_set-2-last_name').send_keys('2 last name 2') + self.selenium.find_element(By.NAME, "profile_set-0-first_name").send_keys( + "0 first name 1" + ) + self.selenium.find_element(By.NAME, "profile_set-0-last_name").send_keys( + "0 last name 2" + ) + self.selenium.find_element(By.NAME, "profile_set-1-first_name").send_keys( + "1 first name 1" + ) + self.selenium.find_element(By.NAME, "profile_set-1-last_name").send_keys( + "1 last name 2" + ) + self.selenium.find_element(By.NAME, "profile_set-2-first_name").send_keys( + "2 first name 1" + ) + self.selenium.find_element(By.NAME, "profile_set-2-last_name").send_keys( + "2 last name 2" + ) with self.wait_page_loaded(): self.selenium.find_element(By.XPATH, '//input[@value="Save"]').click() @@ -1464,69 +1734,102 @@ class SeleniumTests(AdminSeleniumTestCase): def test_add_inline_link_absent_for_view_only_parent_model(self): from selenium.common.exceptions import NoSuchElementException from selenium.webdriver.common.by import By - user = User.objects.create_user('testing', password='password', is_staff=True) + + user = User.objects.create_user("testing", password="password", is_staff=True) user.user_permissions.add( - Permission.objects.get(codename='view_poll', content_type=ContentType.objects.get_for_model(Poll)) + Permission.objects.get( + codename="view_poll", + content_type=ContentType.objects.get_for_model(Poll), + ) ) user.user_permissions.add( *Permission.objects.filter( - codename__endswith="question", content_type=ContentType.objects.get_for_model(Question) - ).values_list('pk', flat=True) + codename__endswith="question", + content_type=ContentType.objects.get_for_model(Question), + ).values_list("pk", flat=True) ) - self.admin_login(username='testing', password='password') + self.admin_login(username="testing", password="password") poll = Poll.objects.create(name="Survey") - change_url = reverse('admin:admin_inlines_poll_change', args=(poll.id,)) + change_url = reverse("admin:admin_inlines_poll_change", args=(poll.id,)) self.selenium.get(self.live_server_url + change_url) with self.disable_implicit_wait(): with self.assertRaises(NoSuchElementException): - self.selenium.find_element(By.LINK_TEXT, 'Add another Question') + self.selenium.find_element(By.LINK_TEXT, "Add another Question") def test_delete_inlines(self): from selenium.webdriver.common.by import By - self.admin_login(username='super', password='secret') - self.selenium.get(self.live_server_url + reverse('admin:admin_inlines_profilecollection_add')) + + self.admin_login(username="super", password="secret") + self.selenium.get( + self.live_server_url + reverse("admin:admin_inlines_profilecollection_add") + ) # Add a few inlines - self.selenium.find_element(By.LINK_TEXT, 'Add another Profile').click() - self.selenium.find_element(By.LINK_TEXT, 'Add another Profile').click() - self.selenium.find_element(By.LINK_TEXT, 'Add another Profile').click() - self.selenium.find_element(By.LINK_TEXT, 'Add another Profile').click() - self.assertCountSeleniumElements('#profile_set-group table tr.dynamic-profile_set', 5) - self.assertCountSeleniumElements('form#profilecollection_form tr.dynamic-profile_set#profile_set-0', 1) - self.assertCountSeleniumElements('form#profilecollection_form tr.dynamic-profile_set#profile_set-1', 1) - self.assertCountSeleniumElements('form#profilecollection_form tr.dynamic-profile_set#profile_set-2', 1) - self.assertCountSeleniumElements('form#profilecollection_form tr.dynamic-profile_set#profile_set-3', 1) - self.assertCountSeleniumElements('form#profilecollection_form tr.dynamic-profile_set#profile_set-4', 1) + self.selenium.find_element(By.LINK_TEXT, "Add another Profile").click() + self.selenium.find_element(By.LINK_TEXT, "Add another Profile").click() + self.selenium.find_element(By.LINK_TEXT, "Add another Profile").click() + self.selenium.find_element(By.LINK_TEXT, "Add another Profile").click() + self.assertCountSeleniumElements( + "#profile_set-group table tr.dynamic-profile_set", 5 + ) + self.assertCountSeleniumElements( + "form#profilecollection_form tr.dynamic-profile_set#profile_set-0", 1 + ) + self.assertCountSeleniumElements( + "form#profilecollection_form tr.dynamic-profile_set#profile_set-1", 1 + ) + self.assertCountSeleniumElements( + "form#profilecollection_form tr.dynamic-profile_set#profile_set-2", 1 + ) + self.assertCountSeleniumElements( + "form#profilecollection_form tr.dynamic-profile_set#profile_set-3", 1 + ) + self.assertCountSeleniumElements( + "form#profilecollection_form tr.dynamic-profile_set#profile_set-4", 1 + ) # Click on a few delete buttons self.selenium.find_element( By.CSS_SELECTOR, - 'form#profilecollection_form tr.dynamic-profile_set#profile_set-1 td.delete a', + "form#profilecollection_form tr.dynamic-profile_set#profile_set-1 td.delete a", ).click() self.selenium.find_element( By.CSS_SELECTOR, - 'form#profilecollection_form tr.dynamic-profile_set#profile_set-2 td.delete a', + "form#profilecollection_form tr.dynamic-profile_set#profile_set-2 td.delete a", ).click() # The rows are gone and the IDs have been re-sequenced - self.assertCountSeleniumElements('#profile_set-group table tr.dynamic-profile_set', 3) - self.assertCountSeleniumElements('form#profilecollection_form tr.dynamic-profile_set#profile_set-0', 1) - self.assertCountSeleniumElements('form#profilecollection_form tr.dynamic-profile_set#profile_set-1', 1) - self.assertCountSeleniumElements('form#profilecollection_form tr.dynamic-profile_set#profile_set-2', 1) + self.assertCountSeleniumElements( + "#profile_set-group table tr.dynamic-profile_set", 3 + ) + self.assertCountSeleniumElements( + "form#profilecollection_form tr.dynamic-profile_set#profile_set-0", 1 + ) + self.assertCountSeleniumElements( + "form#profilecollection_form tr.dynamic-profile_set#profile_set-1", 1 + ) + self.assertCountSeleniumElements( + "form#profilecollection_form tr.dynamic-profile_set#profile_set-2", 1 + ) def test_collapsed_inlines(self): from selenium.webdriver.common.by import By # Collapsed inlines have SHOW/HIDE links. - self.admin_login(username='super', password='secret') - self.selenium.get(self.live_server_url + reverse('admin:admin_inlines_author_add')) + self.admin_login(username="super", password="secret") + self.selenium.get( + self.live_server_url + reverse("admin:admin_inlines_author_add") + ) # One field is in a stacked inline, other in a tabular one. - test_fields = ['#id_nonautopkbook_set-0-title', '#id_nonautopkbook_set-2-0-title'] - show_links = self.selenium.find_elements(By.LINK_TEXT, 'SHOW') + test_fields = [ + "#id_nonautopkbook_set-0-title", + "#id_nonautopkbook_set-2-0-title", + ] + show_links = self.selenium.find_elements(By.LINK_TEXT, "SHOW") self.assertEqual(len(show_links), 3) for show_index, field_name in enumerate(test_fields, 0): self.wait_until_invisible(field_name) show_links[show_index].click() self.wait_until_visible(field_name) - hide_links = self.selenium.find_elements(By.LINK_TEXT, 'HIDE') + hide_links = self.selenium.find_elements(By.LINK_TEXT, "HIDE") self.assertEqual(len(hide_links), 2) for hide_index, field_name in enumerate(test_fields, 0): self.wait_until_visible(field_name) @@ -1535,17 +1838,20 @@ class SeleniumTests(AdminSeleniumTestCase): def test_added_stacked_inline_with_collapsed_fields(self): from selenium.webdriver.common.by import By - self.admin_login(username='super', password='secret') - self.selenium.get(self.live_server_url + reverse('admin:admin_inlines_teacher_add')) - self.selenium.find_element(By.LINK_TEXT, 'Add another Child').click() - test_fields = ['#id_child_set-0-name', '#id_child_set-1-name'] - show_links = self.selenium.find_elements(By.LINK_TEXT, 'SHOW') + + self.admin_login(username="super", password="secret") + self.selenium.get( + self.live_server_url + reverse("admin:admin_inlines_teacher_add") + ) + self.selenium.find_element(By.LINK_TEXT, "Add another Child").click() + test_fields = ["#id_child_set-0-name", "#id_child_set-1-name"] + show_links = self.selenium.find_elements(By.LINK_TEXT, "SHOW") self.assertEqual(len(show_links), 2) for show_index, field_name in enumerate(test_fields, 0): self.wait_until_invisible(field_name) show_links[show_index].click() self.wait_until_visible(field_name) - hide_links = self.selenium.find_elements(By.LINK_TEXT, 'HIDE') + hide_links = self.selenium.find_elements(By.LINK_TEXT, "HIDE") self.assertEqual(len(hide_links), 2) for hide_index, field_name in enumerate(test_fields, 0): self.wait_until_visible(field_name) @@ -1553,101 +1859,123 @@ class SeleniumTests(AdminSeleniumTestCase): self.wait_until_invisible(field_name) def assertBorder(self, element, border): - width, style, color = border.split(' ') + width, style, color = border.split(" ") border_properties = [ - 'border-bottom-%s', - 'border-left-%s', - 'border-right-%s', - 'border-top-%s', + "border-bottom-%s", + "border-left-%s", + "border-right-%s", + "border-top-%s", ] for prop in border_properties: - prop = prop % 'width' + prop = prop % "width" self.assertEqual(element.value_of_css_property(prop), width) for prop in border_properties: - prop = prop % 'style' + prop = prop % "style" self.assertEqual(element.value_of_css_property(prop), style) # Convert hex color to rgb. - self.assertRegex(color, '#[0-9a-f]{6}') + self.assertRegex(color, "#[0-9a-f]{6}") r, g, b = int(color[1:3], 16), int(color[3:5], 16), int(color[5:], 16) # The value may be expressed as either rgb() or rgba() depending on the # browser. colors = [ - 'rgb(%d, %d, %d)' % (r, g, b), - 'rgba(%d, %d, %d, 1)' % (r, g, b), + "rgb(%d, %d, %d)" % (r, g, b), + "rgba(%d, %d, %d, 1)" % (r, g, b), ] for prop in border_properties: - prop = prop % 'color' + prop = prop % "color" self.assertIn(element.value_of_css_property(prop), colors) def test_inline_formset_error_input_border(self): from selenium.webdriver.common.by import By - self.admin_login(username='super', password='secret') - self.selenium.get(self.live_server_url + reverse('admin:admin_inlines_holder5_add')) - self.wait_until_visible('#id_dummy') - self.selenium.find_element(By.ID, 'id_dummy').send_keys(1) - fields = ['id_inner5stacked_set-0-dummy', 'id_inner5tabular_set-0-dummy'] - show_links = self.selenium.find_elements(By.LINK_TEXT, 'SHOW') + + self.admin_login(username="super", password="secret") + self.selenium.get( + self.live_server_url + reverse("admin:admin_inlines_holder5_add") + ) + self.wait_until_visible("#id_dummy") + self.selenium.find_element(By.ID, "id_dummy").send_keys(1) + fields = ["id_inner5stacked_set-0-dummy", "id_inner5tabular_set-0-dummy"] + show_links = self.selenium.find_elements(By.LINK_TEXT, "SHOW") for show_index, field_name in enumerate(fields): show_links[show_index].click() - self.wait_until_visible('#' + field_name) + self.wait_until_visible("#" + field_name) self.selenium.find_element(By.ID, field_name).send_keys(1) # Before save all inputs have default border - for inline in ('stacked', 'tabular'): - for field_name in ('name', 'select', 'text'): - element_id = 'id_inner5%s_set-0-%s' % (inline, field_name) + for inline in ("stacked", "tabular"): + for field_name in ("name", "select", "text"): + element_id = "id_inner5%s_set-0-%s" % (inline, field_name) self.assertBorder( self.selenium.find_element(By.ID, element_id), - '1px solid #cccccc', + "1px solid #cccccc", ) self.selenium.find_element(By.XPATH, '//input[@value="Save"]').click() # Test the red border around inputs by css selectors - stacked_selectors = ['.errors input', '.errors select', '.errors textarea'] + stacked_selectors = [".errors input", ".errors select", ".errors textarea"] for selector in stacked_selectors: self.assertBorder( self.selenium.find_element(By.CSS_SELECTOR, selector), - '1px solid #ba2121', + "1px solid #ba2121", ) tabular_selectors = [ - 'td ul.errorlist + input', 'td ul.errorlist + select', 'td ul.errorlist + textarea' + "td ul.errorlist + input", + "td ul.errorlist + select", + "td ul.errorlist + textarea", ] for selector in tabular_selectors: self.assertBorder( self.selenium.find_element(By.CSS_SELECTOR, selector), - '1px solid #ba2121', + "1px solid #ba2121", ) def test_inline_formset_error(self): from selenium.webdriver.common.by import By - self.admin_login(username='super', password='secret') - self.selenium.get(self.live_server_url + reverse('admin:admin_inlines_holder5_add')) - stacked_inline_formset_selector = 'div#inner5stacked_set-group fieldset.module.collapse' - tabular_inline_formset_selector = 'div#inner5tabular_set-group fieldset.module.collapse' + + self.admin_login(username="super", password="secret") + self.selenium.get( + self.live_server_url + reverse("admin:admin_inlines_holder5_add") + ) + stacked_inline_formset_selector = ( + "div#inner5stacked_set-group fieldset.module.collapse" + ) + tabular_inline_formset_selector = ( + "div#inner5tabular_set-group fieldset.module.collapse" + ) # Inlines without errors, both inlines collapsed self.selenium.find_element(By.XPATH, '//input[@value="Save"]').click() - self.assertCountSeleniumElements(stacked_inline_formset_selector + '.collapsed', 1) - self.assertCountSeleniumElements(tabular_inline_formset_selector + '.collapsed', 1) - show_links = self.selenium.find_elements(By.LINK_TEXT, 'SHOW') + self.assertCountSeleniumElements( + stacked_inline_formset_selector + ".collapsed", 1 + ) + self.assertCountSeleniumElements( + tabular_inline_formset_selector + ".collapsed", 1 + ) + show_links = self.selenium.find_elements(By.LINK_TEXT, "SHOW") self.assertEqual(len(show_links), 2) # Inlines with errors, both inlines expanded - test_fields = ['#id_inner5stacked_set-0-dummy', '#id_inner5tabular_set-0-dummy'] + test_fields = ["#id_inner5stacked_set-0-dummy", "#id_inner5tabular_set-0-dummy"] for show_index, field_name in enumerate(test_fields): show_links[show_index].click() self.wait_until_visible(field_name) self.selenium.find_element(By.ID, field_name[1:]).send_keys(1) - hide_links = self.selenium.find_elements(By.LINK_TEXT, 'HIDE') + hide_links = self.selenium.find_elements(By.LINK_TEXT, "HIDE") self.assertEqual(len(hide_links), 2) for hide_index, field_name in enumerate(test_fields): hide_link = hide_links[hide_index] - self.selenium.execute_script('window.scrollTo(0, %s);' % hide_link.location['y']) + self.selenium.execute_script( + "window.scrollTo(0, %s);" % hide_link.location["y"] + ) hide_link.click() self.wait_until_invisible(field_name) with self.wait_page_loaded(): self.selenium.find_element(By.XPATH, '//input[@value="Save"]').click() with self.disable_implicit_wait(): - self.assertCountSeleniumElements(stacked_inline_formset_selector + '.collapsed', 0) - self.assertCountSeleniumElements(tabular_inline_formset_selector + '.collapsed', 0) + self.assertCountSeleniumElements( + stacked_inline_formset_selector + ".collapsed", 0 + ) + self.assertCountSeleniumElements( + tabular_inline_formset_selector + ".collapsed", 0 + ) self.assertCountSeleniumElements(stacked_inline_formset_selector, 1) self.assertCountSeleniumElements(tabular_inline_formset_selector, 1) @@ -1657,35 +1985,46 @@ class SeleniumTests(AdminSeleniumTestCase): verbose_name in the inline form. """ from selenium.webdriver.common.by import By - self.admin_login(username='super', password='secret') + + self.admin_login(username="super", password="secret") # Hide sidebar. - self.selenium.get(self.live_server_url + reverse('admin:admin_inlines_course_add')) - toggle_button = self.selenium.find_element(By.CSS_SELECTOR, '#toggle-nav-sidebar') + self.selenium.get( + self.live_server_url + reverse("admin:admin_inlines_course_add") + ) + toggle_button = self.selenium.find_element( + By.CSS_SELECTOR, "#toggle-nav-sidebar" + ) toggle_button.click() # Each combination of horizontal/vertical filter with stacked/tabular # inlines. tests = [ - 'admin:admin_inlines_course_add', - 'admin:admin_inlines_courseproxy_add', - 'admin:admin_inlines_courseproxy1_add', - 'admin:admin_inlines_courseproxy2_add', + "admin:admin_inlines_course_add", + "admin:admin_inlines_courseproxy_add", + "admin:admin_inlines_courseproxy1_add", + "admin:admin_inlines_courseproxy2_add", ] - css_selector = '.dynamic-class_set#class_set-%s h2' + css_selector = ".dynamic-class_set#class_set-%s h2" for url_name in tests: with self.subTest(url=url_name): self.selenium.get(self.live_server_url + reverse(url_name)) # First inline shows the verbose_name. - available, chosen = self.selenium.find_elements(By.CSS_SELECTOR, css_selector % 0) - self.assertEqual(available.text, 'AVAILABLE ATTENDANT') - self.assertEqual(chosen.text, 'CHOSEN ATTENDANT') + available, chosen = self.selenium.find_elements( + By.CSS_SELECTOR, css_selector % 0 + ) + self.assertEqual(available.text, "AVAILABLE ATTENDANT") + self.assertEqual(chosen.text, "CHOSEN ATTENDANT") # Added inline should also have the correct verbose_name. - self.selenium.find_element(By.LINK_TEXT, 'Add another Class').click() - available, chosen = self.selenium.find_elements(By.CSS_SELECTOR, css_selector % 1) - self.assertEqual(available.text, 'AVAILABLE ATTENDANT') - self.assertEqual(chosen.text, 'CHOSEN ATTENDANT') + self.selenium.find_element(By.LINK_TEXT, "Add another Class").click() + available, chosen = self.selenium.find_elements( + By.CSS_SELECTOR, css_selector % 1 + ) + self.assertEqual(available.text, "AVAILABLE ATTENDANT") + self.assertEqual(chosen.text, "CHOSEN ATTENDANT") # Third inline should also have the correct verbose_name. - self.selenium.find_element(By.LINK_TEXT, 'Add another Class').click() - available, chosen = self.selenium.find_elements(By.CSS_SELECTOR, css_selector % 2) - self.assertEqual(available.text, 'AVAILABLE ATTENDANT') - self.assertEqual(chosen.text, 'CHOSEN ATTENDANT') + self.selenium.find_element(By.LINK_TEXT, "Add another Class").click() + available, chosen = self.selenium.find_elements( + By.CSS_SELECTOR, css_selector % 2 + ) + self.assertEqual(available.text, "AVAILABLE ATTENDANT") + self.assertEqual(chosen.text, "CHOSEN ATTENDANT") diff --git a/tests/admin_inlines/urls.py b/tests/admin_inlines/urls.py index 5345386917..7427c84bf9 100644 --- a/tests/admin_inlines/urls.py +++ b/tests/admin_inlines/urls.py @@ -3,8 +3,8 @@ from django.urls import path from . import admin urlpatterns = [ - path('admin/', admin.site.urls), - path('admin2/', admin.site2.urls), - path('admin3/', admin.site3.urls), - path('admin4/', admin.site4.urls), + path("admin/", admin.site.urls), + path("admin2/", admin.site2.urls), + path("admin3/", admin.site3.urls), + path("admin4/", admin.site4.urls), ] |