summaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorNick Pope <nick@nickpope.me.uk>2022-01-28 20:15:53 +0000
committerMariusz Felisiak <felisiak.mariusz@gmail.com>2022-10-31 12:30:13 +0100
commitd3cb91db87b78629c0d2682630e90a048275179e (patch)
tree92476354cbb4df1bcff3cd1514a1bac6cbc98676 /tests
parenta320aab5129f4019b3c1d28b7a3b509582bc56f9 (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.py9
-rw-r--r--tests/forms_tests/tests/test_media.py2
-rw-r--r--tests/migrations/test_loader.py10
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"),