Refs #33476 -- Reformatted code with Black.

1 files changed, 134 insertions, 80 deletions

diff --git a/tests/admin_utils/test_logentry.py b/tests/admin_utils/test_logentry.py
index 8d8f4ef9ee..c9f080c334 100644
--- a/tests/admin_utils/test_logentry.py
+++ b/tests/admin_utils/test_logentry.py
@@ -13,12 +13,14 @@ from django.utils.html import escape
 from .models import Article, ArticleProxy, Site
 
 
-@override_settings(ROOT_URLCONF='admin_utils.urls')
+@override_settings(ROOT_URLCONF="admin_utils.urls")
 class LogEntryTests(TestCase):
     @classmethod
     def setUpTestData(cls):
-        cls.user = User.objects.create_superuser(username='super', password='secret', email='super@example.com')
-        cls.site = Site.objects.create(domain='example.org')
+        cls.user = User.objects.create_superuser(
+            username="super", password="secret", email="super@example.com"
+        )
+        cls.site = Site.objects.create(domain="example.org")
         cls.a1 = Article.objects.create(
             site=cls.site,
             title="Title",
@@ -26,8 +28,12 @@ class LogEntryTests(TestCase):
         )
         content_type_pk = ContentType.objects.get_for_model(Article).pk
         LogEntry.objects.log_action(
-            cls.user.pk, content_type_pk, cls.a1.pk, repr(cls.a1), CHANGE,
-            change_message='Changed something'
+            cls.user.pk,
+            content_type_pk,
+            cls.a1.pk,
+            repr(cls.a1),
+            CHANGE,
+            change_message="Changed something",
         )
 
     def setUp(self):
@@ -49,29 +55,40 @@ class LogEntryTests(TestCase):
         to get the message dynamically translated at display time.
         """
         post_data = {
-            'site': self.site.pk, 'title': 'Changed', 'hist': 'Some content',
-            'created_0': '2008-03-12', 'created_1': '11:54',
+            "site": self.site.pk,
+            "title": "Changed",
+            "hist": "Some content",
+            "created_0": "2008-03-12",
+            "created_1": "11:54",
         }
-        change_url = reverse('admin:admin_utils_article_change', args=[quote(self.a1.pk)])
+        change_url = reverse(
+            "admin:admin_utils_article_change", args=[quote(self.a1.pk)]
+        )
         response = self.client.post(change_url, post_data)
-        self.assertRedirects(response, reverse('admin:admin_utils_article_changelist'))
-        logentry = LogEntry.objects.filter(content_type__model__iexact='article').latest('id')
-        self.assertEqual(logentry.get_change_message(), 'Changed Title and History.')
-        with translation.override('fr'):
-            self.assertEqual(logentry.get_change_message(), 'Modification de Title et Historique.')
+        self.assertRedirects(response, reverse("admin:admin_utils_article_changelist"))
+        logentry = LogEntry.objects.filter(
+            content_type__model__iexact="article"
+        ).latest("id")
+        self.assertEqual(logentry.get_change_message(), "Changed Title and History.")
+        with translation.override("fr"):
+            self.assertEqual(
+                logentry.get_change_message(), "Modification de Title et Historique."
+            )
 
-        add_url = reverse('admin:admin_utils_article_add')
-        post_data['title'] = 'New'
+        add_url = reverse("admin:admin_utils_article_add")
+        post_data["title"] = "New"
         response = self.client.post(add_url, post_data)
-        self.assertRedirects(response, reverse('admin:admin_utils_article_changelist'))
-        logentry = LogEntry.objects.filter(content_type__model__iexact='article').latest('id')
-        self.assertEqual(logentry.get_change_message(), 'Added.')
-        with translation.override('fr'):
-            self.assertEqual(logentry.get_change_message(), 'Ajout.')
+        self.assertRedirects(response, reverse("admin:admin_utils_article_changelist"))
+        logentry = LogEntry.objects.filter(
+            content_type__model__iexact="article"
+        ).latest("id")
+        self.assertEqual(logentry.get_change_message(), "Added.")
+        with translation.override("fr"):
+            self.assertEqual(logentry.get_change_message(), "Ajout.")
 
     def test_logentry_change_message_not_json(self):
         """LogEntry.change_message was a string before Django 1.10."""
-        logentry = LogEntry(change_message='non-JSON string')
+        logentry = LogEntry(change_message="non-JSON string")
         self.assertEqual(logentry.get_change_message(), logentry.change_message)
 
     def test_logentry_change_message_localized_datetime_input(self):
@@ -79,15 +96,24 @@ class LogEntryTests(TestCase):
         Localized date/time inputs shouldn't affect changed form data detection.
         """
         post_data = {
-            'site': self.site.pk, 'title': 'Changed', 'hist': 'Some content',
-            'created_0': '12/03/2008', 'created_1': '11:54',
+            "site": self.site.pk,
+            "title": "Changed",
+            "hist": "Some content",
+            "created_0": "12/03/2008",
+            "created_1": "11:54",
         }
-        with translation.override('fr'):
-            change_url = reverse('admin:admin_utils_article_change', args=[quote(self.a1.pk)])
+        with translation.override("fr"):
+            change_url = reverse(
+                "admin:admin_utils_article_change", args=[quote(self.a1.pk)]
+            )
             response = self.client.post(change_url, post_data)
-            self.assertRedirects(response, reverse('admin:admin_utils_article_changelist'))
-        logentry = LogEntry.objects.filter(content_type__model__iexact='article').latest('id')
-        self.assertEqual(logentry.get_change_message(), 'Changed Title and History.')
+            self.assertRedirects(
+                response, reverse("admin:admin_utils_article_changelist")
+            )
+        logentry = LogEntry.objects.filter(
+            content_type__model__iexact="article"
+        ).latest("id")
+        self.assertEqual(logentry.get_change_message(), "Changed Title and History.")
 
     def test_logentry_change_message_formsets(self):
         """
@@ -99,52 +125,62 @@ class LogEntryTests(TestCase):
             created=datetime(2012, 3, 18, 11, 54),
         )
         post_data = {
-            'domain': 'example.com',  # domain changed
-            'admin_articles-TOTAL_FORMS': '5',
-            'admin_articles-INITIAL_FORMS': '2',
-            'admin_articles-MIN_NUM_FORMS': '0',
-            'admin_articles-MAX_NUM_FORMS': '1000',
+            "domain": "example.com",  # domain changed
+            "admin_articles-TOTAL_FORMS": "5",
+            "admin_articles-INITIAL_FORMS": "2",
+            "admin_articles-MIN_NUM_FORMS": "0",
+            "admin_articles-MAX_NUM_FORMS": "1000",
             # Changed title for 1st article
-            'admin_articles-0-id': str(self.a1.pk),
-            'admin_articles-0-site': str(self.site.pk),
-            'admin_articles-0-title': 'Changed Title',
+            "admin_articles-0-id": str(self.a1.pk),
+            "admin_articles-0-site": str(self.site.pk),
+            "admin_articles-0-title": "Changed Title",
             # Second article is deleted
-            'admin_articles-1-id': str(a2.pk),
-            'admin_articles-1-site': str(self.site.pk),
-            'admin_articles-1-title': 'Title second article',
-            'admin_articles-1-DELETE': 'on',
+            "admin_articles-1-id": str(a2.pk),
+            "admin_articles-1-site": str(self.site.pk),
+            "admin_articles-1-title": "Title second article",
+            "admin_articles-1-DELETE": "on",
             # A new article is added
-            'admin_articles-2-site': str(self.site.pk),
-            'admin_articles-2-title': 'Added article',
+            "admin_articles-2-site": str(self.site.pk),
+            "admin_articles-2-title": "Added article",
         }
-        change_url = reverse('admin:admin_utils_site_change', args=[quote(self.site.pk)])
+        change_url = reverse(
+            "admin:admin_utils_site_change", args=[quote(self.site.pk)]
+        )
         response = self.client.post(change_url, post_data)
-        self.assertRedirects(response, reverse('admin:admin_utils_site_changelist'))
+        self.assertRedirects(response, reverse("admin:admin_utils_site_changelist"))
         self.assertQuerysetEqual(Article.objects.filter(pk=a2.pk), [])
-        logentry = LogEntry.objects.filter(content_type__model__iexact='site').latest('action_time')
+        logentry = LogEntry.objects.filter(content_type__model__iexact="site").latest(
+            "action_time"
+        )
         self.assertEqual(
             json.loads(logentry.change_message),
             [
                 {"changed": {"fields": ["Domain"]}},
                 {"added": {"object": "Added article", "name": "article"}},
-                {"changed": {"fields": ["Title", "not_a_form_field"], "object": "Changed Title", "name": "article"}},
+                {
+                    "changed": {
+                        "fields": ["Title", "not_a_form_field"],
+                        "object": "Changed Title",
+                        "name": "article",
+                    }
+                },
                 {"deleted": {"object": "Title second article", "name": "article"}},
-            ]
+            ],
         )
         self.assertEqual(
             logentry.get_change_message(),
-            'Changed Domain. Added article “Added article”. '
-            'Changed Title and not_a_form_field for article “Changed Title”. '
-            'Deleted article “Title second article”.'
+            "Changed Domain. Added article “Added article”. "
+            "Changed Title and not_a_form_field for article “Changed Title”. "
+            "Deleted article “Title second article”.",
         )
 
-        with translation.override('fr'):
+        with translation.override("fr"):
             self.assertEqual(
                 logentry.get_change_message(),
                 "Modification de Domain. Ajout de article « Added article ». "
                 "Modification de Title et not_a_form_field pour l'objet "
                 "article « Changed Title ». "
-                "Suppression de article « Title second article »."
+                "Suppression de article « Title second article ».",
             )
 
     def test_logentry_get_edited_object(self):
@@ -161,10 +197,12 @@ class LogEntryTests(TestCase):
         LogEntry.get_admin_url returns a URL to edit the entry's object or
         None for nonexistent (possibly deleted) models.
         """
-        logentry = LogEntry.objects.get(content_type__model__iexact='article')
-        expected_url = reverse('admin:admin_utils_article_change', args=(quote(self.a1.pk),))
+        logentry = LogEntry.objects.get(content_type__model__iexact="article")
+        expected_url = reverse(
+            "admin:admin_utils_article_change", args=(quote(self.a1.pk),)
+        )
         self.assertEqual(logentry.get_admin_url(), expected_url)
-        self.assertIn('article/%d/change/' % self.a1.pk, logentry.get_admin_url())
+        self.assertIn("article/%d/change/" % self.a1.pk, logentry.get_admin_url())
 
         logentry.content_type.model = "nonexistent"
         self.assertIsNone(logentry.get_admin_url())
@@ -173,17 +211,17 @@ class LogEntryTests(TestCase):
         log_entry = LogEntry()
 
         log_entry.action_flag = ADDITION
-        self.assertTrue(str(log_entry).startswith('Added '))
+        self.assertTrue(str(log_entry).startswith("Added "))
 
         log_entry.action_flag = CHANGE
-        self.assertTrue(str(log_entry).startswith('Changed '))
+        self.assertTrue(str(log_entry).startswith("Changed "))
 
         log_entry.action_flag = DELETION
-        self.assertTrue(str(log_entry).startswith('Deleted '))
+        self.assertTrue(str(log_entry).startswith("Deleted "))
 
         # Make sure custom action_flags works
         log_entry.action_flag = 4
-        self.assertEqual(str(log_entry), 'LogEntry Object')
+        self.assertEqual(str(log_entry), "LogEntry Object")
 
     def test_logentry_repr(self):
         logentry = LogEntry.objects.first()
@@ -192,23 +230,30 @@ class LogEntryTests(TestCase):
     def test_log_action(self):
         content_type_pk = ContentType.objects.get_for_model(Article).pk
         log_entry = LogEntry.objects.log_action(
-            self.user.pk, content_type_pk, self.a1.pk, repr(self.a1), CHANGE,
-            change_message='Changed something else',
+            self.user.pk,
+            content_type_pk,
+            self.a1.pk,
+            repr(self.a1),
+            CHANGE,
+            change_message="Changed something else",
         )
-        self.assertEqual(log_entry, LogEntry.objects.latest('id'))
+        self.assertEqual(log_entry, LogEntry.objects.latest("id"))
 
     def test_recentactions_without_content_type(self):
         """
         If a LogEntry is missing content_type it will not display it in span
         tag under the hyperlink.
         """
-        response = self.client.get(reverse('admin:index'))
-        link = reverse('admin:admin_utils_article_change', args=(quote(self.a1.pk),))
-        should_contain = """<a href="%s">%s</a>""" % (escape(link), escape(repr(self.a1)))
+        response = self.client.get(reverse("admin:index"))
+        link = reverse("admin:admin_utils_article_change", args=(quote(self.a1.pk),))
+        should_contain = """<a href="%s">%s</a>""" % (
+            escape(link),
+            escape(repr(self.a1)),
+        )
         self.assertContains(response, should_contain)
         should_contain = "Article"
         self.assertContains(response, should_contain)
-        logentry = LogEntry.objects.get(content_type__model__iexact='article')
+        logentry = LogEntry.objects.get(content_type__model__iexact="article")
         # If the log entry doesn't have a content type it should still be
         # possible to view the Recent Actions part (#10275).
         logentry.content_type = None
@@ -216,7 +261,7 @@ class LogEntryTests(TestCase):
 
         should_contain = should_contain.encode()
         counted_presence_before = response.content.count(should_contain)
-        response = self.client.get(reverse('admin:index'))
+        response = self.client.get(reverse("admin:index"))
         counted_presence_after = response.content.count(should_contain)
         self.assertEqual(counted_presence_before - 1, counted_presence_after)
 
@@ -225,41 +270,50 @@ class LogEntryTests(TestCase):
         Log entries for proxy models should have the proxy model's contenttype
         (#21084).
         """
-        proxy_content_type = ContentType.objects.get_for_model(ArticleProxy, for_concrete_model=False)
+        proxy_content_type = ContentType.objects.get_for_model(
+            ArticleProxy, for_concrete_model=False
+        )
         post_data = {
-            'site': self.site.pk, 'title': "Foo", 'hist': "Bar",
-            'created_0': '2015-12-25', 'created_1': '00:00',
+            "site": self.site.pk,
+            "title": "Foo",
+            "hist": "Bar",
+            "created_0": "2015-12-25",
+            "created_1": "00:00",
         }
-        changelist_url = reverse('admin:admin_utils_articleproxy_changelist')
+        changelist_url = reverse("admin:admin_utils_articleproxy_changelist")
 
         # add
-        proxy_add_url = reverse('admin:admin_utils_articleproxy_add')
+        proxy_add_url = reverse("admin:admin_utils_articleproxy_add")
         response = self.client.post(proxy_add_url, post_data)
         self.assertRedirects(response, changelist_url)
-        proxy_addition_log = LogEntry.objects.latest('id')
+        proxy_addition_log = LogEntry.objects.latest("id")
         self.assertEqual(proxy_addition_log.action_flag, ADDITION)
         self.assertEqual(proxy_addition_log.content_type, proxy_content_type)
 
         # change
         article_id = proxy_addition_log.object_id
-        proxy_change_url = reverse('admin:admin_utils_articleproxy_change', args=(article_id,))
-        post_data['title'] = 'New'
+        proxy_change_url = reverse(
+            "admin:admin_utils_articleproxy_change", args=(article_id,)
+        )
+        post_data["title"] = "New"
         response = self.client.post(proxy_change_url, post_data)
         self.assertRedirects(response, changelist_url)
-        proxy_change_log = LogEntry.objects.latest('id')
+        proxy_change_log = LogEntry.objects.latest("id")
         self.assertEqual(proxy_change_log.action_flag, CHANGE)
         self.assertEqual(proxy_change_log.content_type, proxy_content_type)
 
         # delete
-        proxy_delete_url = reverse('admin:admin_utils_articleproxy_delete', args=(article_id,))
-        response = self.client.post(proxy_delete_url, {'post': 'yes'})
+        proxy_delete_url = reverse(
+            "admin:admin_utils_articleproxy_delete", args=(article_id,)
+        )
+        response = self.client.post(proxy_delete_url, {"post": "yes"})
         self.assertRedirects(response, changelist_url)
-        proxy_delete_log = LogEntry.objects.latest('id')
+        proxy_delete_log = LogEntry.objects.latest("id")
         self.assertEqual(proxy_delete_log.action_flag, DELETION)
         self.assertEqual(proxy_delete_log.content_type, proxy_content_type)
 
     def test_action_flag_choices(self):
-        tests = ((1, 'Addition'), (2, 'Change'), (3, 'Deletion'))
+        tests = ((1, "Addition"), (2, "Change"), (3, "Deletion"))
         for action_flag, display_name in tests:
             with self.subTest(action_flag=action_flag):
                 log = LogEntry(action_flag=action_flag)