diff options
| author | Nick Pope <nick@nickpope.me.uk> | 2022-01-28 20:15:53 +0000 |
|---|---|---|
| committer | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2022-10-31 12:30:13 +0100 |
| commit | d3cb91db87b78629c0d2682630e90a048275179e (patch) | |
| tree | 92476354cbb4df1bcff3cd1514a1bac6cbc98676 /tests | |
| parent | a320aab5129f4019b3c1d28b7a3b509582bc56f9 (diff) | |
Used more augmented assignment statements.
Identified using the following command:
$ git grep -I '\(\<[_a-zA-Z0-9]\+\>\) *= *\1 *[-+/*^%&|<>@]'
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/admin_inlines/tests.py | 9 | ||||
| -rw-r--r-- | tests/forms_tests/tests/test_media.py | 2 | ||||
| -rw-r--r-- | tests/migrations/test_loader.py | 10 |
3 files changed, 9 insertions, 12 deletions
diff --git a/tests/admin_inlines/tests.py b/tests/admin_inlines/tests.py index ad7446e1ca..73f888d9d3 100644 --- a/tests/admin_inlines/tests.py +++ b/tests/admin_inlines/tests.py @@ -1908,11 +1908,9 @@ class SeleniumTests(AdminSeleniumTestCase): "border-top-%s", ] for prop in border_properties: - prop = prop % "width" - self.assertEqual(element.value_of_css_property(prop), width) + self.assertEqual(element.value_of_css_property(prop % "width"), width) for prop in border_properties: - prop = prop % "style" - self.assertEqual(element.value_of_css_property(prop), style) + self.assertEqual(element.value_of_css_property(prop % "style"), style) # Convert hex color to rgb. self.assertRegex(color, "#[0-9a-f]{6}") r, g, b = int(color[1:3], 16), int(color[3:5], 16), int(color[5:], 16) @@ -1923,8 +1921,7 @@ class SeleniumTests(AdminSeleniumTestCase): "rgba(%d, %d, %d, 1)" % (r, g, b), ] for prop in border_properties: - prop = prop % "color" - self.assertIn(element.value_of_css_property(prop), colors) + self.assertIn(element.value_of_css_property(prop % "color"), colors) def test_inline_formset_error_input_border(self): from selenium.webdriver.common.by import By diff --git a/tests/forms_tests/tests/test_media.py b/tests/forms_tests/tests/test_media.py index 171b70e508..aa508cb415 100644 --- a/tests/forms_tests/tests/test_media.py +++ b/tests/forms_tests/tests/test_media.py @@ -643,7 +643,7 @@ class FormsMediaTestCase(SimpleTestCase): self.assertEqual( merged._css, {"screen": ["c.css", "a.css"], "all": ["d.css", "e.css"]} ) - merged = merged + widget3 + merged += widget3 # widget3 contains an explicit ordering of c.css and a.css. self.assertEqual( merged._css, diff --git a/tests/migrations/test_loader.py b/tests/migrations/test_loader.py index 847b937d5a..800d715bc9 100644 --- a/tests/migrations/test_loader.py +++ b/tests/migrations/test_loader.py @@ -390,7 +390,7 @@ class LoaderTests(TestCase): loader.build_graph() plan = set(loader.graph.forwards_plan(("app1", "4_auto"))) - plan = plan - loader.applied_migrations.keys() + plan -= loader.applied_migrations.keys() expected_plan = { ("app2", "1_squashed_2"), ("app1", "3_auto"), @@ -529,7 +529,7 @@ class LoaderTests(TestCase): # Load with nothing applied: both migrations squashed. loader.build_graph() plan = set(loader.graph.forwards_plan(("app1", "4_auto"))) - plan = plan - loader.applied_migrations.keys() + plan -= loader.applied_migrations.keys() expected_plan = { ("app1", "1_auto"), ("app2", "1_squashed_2"), @@ -548,7 +548,7 @@ class LoaderTests(TestCase): loader.replace_migrations = False loader.build_graph() plan = set(loader.graph.forwards_plan(("app1", "3_auto"))) - plan = plan - loader.applied_migrations.keys() + plan -= loader.applied_migrations.keys() expected_plan = { ("app1", "1_auto"), ("app2", "1_auto"), @@ -564,7 +564,7 @@ class LoaderTests(TestCase): self.record_applied(recorder, "app1", "2_auto") loader.build_graph() plan = set(loader.graph.forwards_plan(("app1", "4_auto"))) - plan = plan - loader.applied_migrations.keys() + plan -= loader.applied_migrations.keys() expected_plan = { ("app2", "1_squashed_2"), ("app1", "3_auto"), @@ -576,7 +576,7 @@ class LoaderTests(TestCase): self.record_applied(recorder, "app2", "1_auto") loader.build_graph() plan = set(loader.graph.forwards_plan(("app1", "4_auto"))) - plan = plan - loader.applied_migrations.keys() + plan -= loader.applied_migrations.keys() expected_plan = { ("app2", "2_auto"), ("app1", "3_auto"), |
