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/update_only_fields | |
| parent | f68fa8b45dfac545cfc4111d4e52804c86db68d3 (diff) | |
| download | django-9c19aff7c7561e3a82978a272ecdaad40dda5c00.tar.gz | |
Refs #33476 -- Reformatted code with Black.
Diffstat (limited to 'tests/update_only_fields')
| -rw-r--r-- | tests/update_only_fields/models.py | 11 | ||||
| -rw-r--r-- | tests/update_only_fields/tests.py | 194 |
2 files changed, 113 insertions, 92 deletions
diff --git a/tests/update_only_fields/models.py b/tests/update_only_fields/models.py index 3bba684d21..4810d5b191 100644 --- a/tests/update_only_fields/models.py +++ b/tests/update_only_fields/models.py @@ -1,4 +1,3 @@ - from django.db import models @@ -8,8 +7,8 @@ class Account(models.Model): class Person(models.Model): GENDER_CHOICES = ( - ('M', 'Male'), - ('F', 'Female'), + ("M", "Male"), + ("F", "Female"), ) name = models.CharField(max_length=20) gender = models.CharField(max_length=1, choices=GENDER_CHOICES) @@ -18,8 +17,10 @@ class Person(models.Model): class Employee(Person): employee_num = models.IntegerField(default=0) - profile = models.ForeignKey('Profile', models.SET_NULL, related_name='profiles', null=True) - accounts = models.ManyToManyField('Account', related_name='employees', blank=True) + profile = models.ForeignKey( + "Profile", models.SET_NULL, related_name="profiles", null=True + ) + accounts = models.ManyToManyField("Account", related_name="employees", blank=True) class NonConcreteField(models.IntegerField): diff --git a/tests/update_only_fields/tests.py b/tests/update_only_fields/tests.py index 65a260338c..6c23ae27d8 100644 --- a/tests/update_only_fields/tests.py +++ b/tests/update_only_fields/tests.py @@ -6,25 +6,25 @@ from .models import Account, Employee, Person, Profile, ProxyEmployee class UpdateOnlyFieldsTests(TestCase): msg = ( - 'The following fields do not exist in this model, are m2m fields, or ' - 'are non-concrete fields: %s' + "The following fields do not exist in this model, are m2m fields, or " + "are non-concrete fields: %s" ) def test_update_fields_basic(self): - s = Person.objects.create(name='Sara', gender='F') - self.assertEqual(s.gender, 'F') + s = Person.objects.create(name="Sara", gender="F") + self.assertEqual(s.gender, "F") - s.gender = 'M' - s.name = 'Ian' - s.save(update_fields=['name']) + s.gender = "M" + s.name = "Ian" + s.save(update_fields=["name"]) s = Person.objects.get(pk=s.pk) - self.assertEqual(s.gender, 'F') - self.assertEqual(s.name, 'Ian') + self.assertEqual(s.gender, "F") + self.assertEqual(s.name, "Ian") def test_update_fields_deferred(self): - s = Person.objects.create(name='Sara', gender='F', pid=22) - self.assertEqual(s.gender, 'F') + s = Person.objects.create(name="Sara", gender="F", pid=22) + self.assertEqual(s.gender, "F") s1 = Person.objects.defer("gender", "pid").get(pk=s.pk) s1.name = "Emily" @@ -38,10 +38,10 @@ class UpdateOnlyFieldsTests(TestCase): self.assertEqual(s2.gender, "M") def test_update_fields_only_1(self): - s = Person.objects.create(name='Sara', gender='F') - self.assertEqual(s.gender, 'F') + s = Person.objects.create(name="Sara", gender="F") + self.assertEqual(s.gender, "F") - s1 = Person.objects.only('name').get(pk=s.pk) + s1 = Person.objects.only("name").get(pk=s.pk) s1.name = "Emily" s1.gender = "M" @@ -53,47 +53,51 @@ class UpdateOnlyFieldsTests(TestCase): self.assertEqual(s2.gender, "M") def test_update_fields_only_2(self): - s = Person.objects.create(name='Sara', gender='F', pid=22) - self.assertEqual(s.gender, 'F') + s = Person.objects.create(name="Sara", gender="F", pid=22) + self.assertEqual(s.gender, "F") - s1 = Person.objects.only('name').get(pk=s.pk) + s1 = Person.objects.only("name").get(pk=s.pk) s1.name = "Emily" s1.gender = "M" with self.assertNumQueries(2): - s1.save(update_fields=['pid']) + s1.save(update_fields=["pid"]) s2 = Person.objects.get(pk=s1.pk) self.assertEqual(s2.name, "Sara") self.assertEqual(s2.gender, "F") def test_update_fields_only_repeated(self): - s = Person.objects.create(name='Sara', gender='F') - self.assertEqual(s.gender, 'F') + s = Person.objects.create(name="Sara", gender="F") + self.assertEqual(s.gender, "F") - s1 = Person.objects.only('name').get(pk=s.pk) - s1.gender = 'M' + s1 = Person.objects.only("name").get(pk=s.pk) + s1.gender = "M" with self.assertNumQueries(1): s1.save() # save() should not fetch deferred fields - s1 = Person.objects.only('name').get(pk=s.pk) + s1 = Person.objects.only("name").get(pk=s.pk) with self.assertNumQueries(1): s1.save() def test_update_fields_inheritance_defer(self): - profile_boss = Profile.objects.create(name='Boss', salary=3000) - e1 = Employee.objects.create(name='Sara', gender='F', employee_num=1, profile=profile_boss) - e1 = Employee.objects.only('name').get(pk=e1.pk) - e1.name = 'Linda' + profile_boss = Profile.objects.create(name="Boss", salary=3000) + e1 = Employee.objects.create( + name="Sara", gender="F", employee_num=1, profile=profile_boss + ) + e1 = Employee.objects.only("name").get(pk=e1.pk) + e1.name = "Linda" with self.assertNumQueries(1): e1.save() - self.assertEqual(Employee.objects.get(pk=e1.pk).name, 'Linda') + self.assertEqual(Employee.objects.get(pk=e1.pk).name, "Linda") def test_update_fields_fk_defer(self): - profile_boss = Profile.objects.create(name='Boss', salary=3000) - profile_receptionist = Profile.objects.create(name='Receptionist', salary=1000) - e1 = Employee.objects.create(name='Sara', gender='F', employee_num=1, profile=profile_boss) - e1 = Employee.objects.only('profile').get(pk=e1.pk) + profile_boss = Profile.objects.create(name="Boss", salary=3000) + profile_receptionist = Profile.objects.create(name="Receptionist", salary=1000) + e1 = Employee.objects.create( + name="Sara", gender="F", employee_num=1, profile=profile_boss + ) + e1 = Employee.objects.only("profile").get(pk=e1.pk) e1.profile = profile_receptionist with self.assertNumQueries(1): e1.save() @@ -104,10 +108,16 @@ class UpdateOnlyFieldsTests(TestCase): self.assertEqual(Employee.objects.get(pk=e1.pk).profile, profile_boss) def test_select_related_only_interaction(self): - profile_boss = Profile.objects.create(name='Boss', salary=3000) - e1 = Employee.objects.create(name='Sara', gender='F', employee_num=1, profile=profile_boss) - e1 = Employee.objects.only('profile__salary').select_related('profile').get(pk=e1.pk) - profile_boss.name = 'Clerk' + profile_boss = Profile.objects.create(name="Boss", salary=3000) + e1 = Employee.objects.create( + name="Sara", gender="F", employee_num=1, profile=profile_boss + ) + e1 = ( + Employee.objects.only("profile__salary") + .select_related("profile") + .get(pk=e1.pk) + ) + profile_boss.name = "Clerk" profile_boss.salary = 1000 profile_boss.save() # The loaded salary of 3000 gets saved, the name of 'Clerk' isn't @@ -119,112 +129,122 @@ class UpdateOnlyFieldsTests(TestCase): self.assertEqual(reloaded_profile.salary, 3000) def test_update_fields_m2m(self): - profile_boss = Profile.objects.create(name='Boss', salary=3000) - e1 = Employee.objects.create(name='Sara', gender='F', employee_num=1, profile=profile_boss) + profile_boss = Profile.objects.create(name="Boss", salary=3000) + e1 = Employee.objects.create( + name="Sara", gender="F", employee_num=1, profile=profile_boss + ) a1 = Account.objects.create(num=1) a2 = Account.objects.create(num=2) e1.accounts.set([a1, a2]) - with self.assertRaisesMessage(ValueError, self.msg % 'accounts'): - e1.save(update_fields=['accounts']) + with self.assertRaisesMessage(ValueError, self.msg % "accounts"): + e1.save(update_fields=["accounts"]) def test_update_fields_inheritance(self): - profile_boss = Profile.objects.create(name='Boss', salary=3000) - profile_receptionist = Profile.objects.create(name='Receptionist', salary=1000) - e1 = Employee.objects.create(name='Sara', gender='F', employee_num=1, profile=profile_boss) + profile_boss = Profile.objects.create(name="Boss", salary=3000) + profile_receptionist = Profile.objects.create(name="Receptionist", salary=1000) + e1 = Employee.objects.create( + name="Sara", gender="F", employee_num=1, profile=profile_boss + ) - e1.name = 'Ian' - e1.gender = 'M' - e1.save(update_fields=['name']) + e1.name = "Ian" + e1.gender = "M" + e1.save(update_fields=["name"]) e2 = Employee.objects.get(pk=e1.pk) - self.assertEqual(e2.name, 'Ian') - self.assertEqual(e2.gender, 'F') + self.assertEqual(e2.name, "Ian") + self.assertEqual(e2.gender, "F") self.assertEqual(e2.profile, profile_boss) e2.profile = profile_receptionist - e2.name = 'Sara' - e2.save(update_fields=['profile']) + e2.name = "Sara" + e2.save(update_fields=["profile"]) e3 = Employee.objects.get(pk=e1.pk) - self.assertEqual(e3.name, 'Ian') + self.assertEqual(e3.name, "Ian") self.assertEqual(e3.profile, profile_receptionist) with self.assertNumQueries(1): e3.profile = profile_boss - e3.save(update_fields=['profile_id']) + e3.save(update_fields=["profile_id"]) e4 = Employee.objects.get(pk=e3.pk) self.assertEqual(e4.profile, profile_boss) self.assertEqual(e4.profile_id, profile_boss.pk) def test_update_fields_inheritance_with_proxy_model(self): - profile_boss = Profile.objects.create(name='Boss', salary=3000) - profile_receptionist = Profile.objects.create(name='Receptionist', salary=1000) - e1 = ProxyEmployee.objects.create(name='Sara', gender='F', employee_num=1, profile=profile_boss) + profile_boss = Profile.objects.create(name="Boss", salary=3000) + profile_receptionist = Profile.objects.create(name="Receptionist", salary=1000) + e1 = ProxyEmployee.objects.create( + name="Sara", gender="F", employee_num=1, profile=profile_boss + ) - e1.name = 'Ian' - e1.gender = 'M' - e1.save(update_fields=['name']) + e1.name = "Ian" + e1.gender = "M" + e1.save(update_fields=["name"]) e2 = ProxyEmployee.objects.get(pk=e1.pk) - self.assertEqual(e2.name, 'Ian') - self.assertEqual(e2.gender, 'F') + self.assertEqual(e2.name, "Ian") + self.assertEqual(e2.gender, "F") self.assertEqual(e2.profile, profile_boss) e2.profile = profile_receptionist - e2.name = 'Sara' - e2.save(update_fields=['profile']) + e2.name = "Sara" + e2.save(update_fields=["profile"]) e3 = ProxyEmployee.objects.get(pk=e1.pk) - self.assertEqual(e3.name, 'Ian') + self.assertEqual(e3.name, "Ian") self.assertEqual(e3.profile, profile_receptionist) def test_update_fields_signals(self): - p = Person.objects.create(name='Sara', gender='F') + p = Person.objects.create(name="Sara", gender="F") pre_save_data = [] def pre_save_receiver(**kwargs): - pre_save_data.append(kwargs['update_fields']) + pre_save_data.append(kwargs["update_fields"]) + pre_save.connect(pre_save_receiver) post_save_data = [] def post_save_receiver(**kwargs): - post_save_data.append(kwargs['update_fields']) + post_save_data.append(kwargs["update_fields"]) + post_save.connect(post_save_receiver) - p.save(update_fields=['name']) + p.save(update_fields=["name"]) self.assertEqual(len(pre_save_data), 1) self.assertEqual(len(pre_save_data[0]), 1) - self.assertIn('name', pre_save_data[0]) + self.assertIn("name", pre_save_data[0]) self.assertEqual(len(post_save_data), 1) self.assertEqual(len(post_save_data[0]), 1) - self.assertIn('name', post_save_data[0]) + self.assertIn("name", post_save_data[0]) pre_save.disconnect(pre_save_receiver) post_save.disconnect(post_save_receiver) def test_update_fields_incorrect_params(self): - s = Person.objects.create(name='Sara', gender='F') + s = Person.objects.create(name="Sara", gender="F") - with self.assertRaisesMessage(ValueError, self.msg % 'first_name'): - s.save(update_fields=['first_name']) + with self.assertRaisesMessage(ValueError, self.msg % "first_name"): + s.save(update_fields=["first_name"]) # "name" is treated as an iterable so the output is something like # "n, a, m, e" but the order isn't deterministic. - with self.assertRaisesMessage(ValueError, self.msg % ''): + with self.assertRaisesMessage(ValueError, self.msg % ""): s.save(update_fields="name") def test_empty_update_fields(self): - s = Person.objects.create(name='Sara', gender='F') + s = Person.objects.create(name="Sara", gender="F") pre_save_data = [] def pre_save_receiver(**kwargs): - pre_save_data.append(kwargs['update_fields']) + pre_save_data.append(kwargs["update_fields"]) + pre_save.connect(pre_save_receiver) post_save_data = [] def post_save_receiver(**kwargs): - post_save_data.append(kwargs['update_fields']) + post_save_data.append(kwargs["update_fields"]) + post_save.connect(post_save_receiver) # Save is skipped. with self.assertNumQueries(0): @@ -237,28 +257,28 @@ class UpdateOnlyFieldsTests(TestCase): post_save.disconnect(post_save_receiver) def test_num_queries_inheritance(self): - s = Employee.objects.create(name='Sara', gender='F') + s = Employee.objects.create(name="Sara", gender="F") s.employee_num = 1 - s.name = 'Emily' + s.name = "Emily" with self.assertNumQueries(1): - s.save(update_fields=['employee_num']) + s.save(update_fields=["employee_num"]) s = Employee.objects.get(pk=s.pk) self.assertEqual(s.employee_num, 1) - self.assertEqual(s.name, 'Sara') + self.assertEqual(s.name, "Sara") s.employee_num = 2 - s.name = 'Emily' + s.name = "Emily" with self.assertNumQueries(1): - s.save(update_fields=['name']) + s.save(update_fields=["name"]) s = Employee.objects.get(pk=s.pk) - self.assertEqual(s.name, 'Emily') + self.assertEqual(s.name, "Emily") self.assertEqual(s.employee_num, 1) # A little sanity check that we actually did updates... self.assertEqual(Employee.objects.count(), 1) self.assertEqual(Person.objects.count(), 1) with self.assertNumQueries(2): - s.save(update_fields=['name', 'employee_num']) + s.save(update_fields=["name", "employee_num"]) def test_update_non_concrete_field(self): - profile_boss = Profile.objects.create(name='Boss', salary=3000) - with self.assertRaisesMessage(ValueError, self.msg % 'non_concrete'): - profile_boss.save(update_fields=['non_concrete']) + profile_boss = Profile.objects.create(name="Boss", salary=3000) + with self.assertRaisesMessage(ValueError, self.msg % "non_concrete"): + profile_boss.save(update_fields=["non_concrete"]) |
