nit: PEP 8 (mdn#3976)

Alicja-code · web-flow · commit cb7802d401c0 · 2021-04-09T16:09:23.000-07:00
Hi,
Below several NIT ideas for small improvements in Django Tutorial Part 10 'Testing a Django web application' considering PEP 8 guidelines.

In section 'LocalLibrary tests'/'Forms' there is:
self.assertTrue(form.fields['renewal_date'].label == None or form.fields['renewal_date'].label == 'renewal date')
While I believe there should be:
self.assertTrue(form.fields['renewal_date'].label is None or form.fields['renewal_date'].label == 'renewal date')
Explaination:
Comparison with None performed with equality operator. That type of compression should be done with 'is'. PEP 8: comparison to None should be 'if cond is None'.

In section 'LocalLibrary tests'/'Models' there is:
   def test_date_of_death_label(self):
        author=Author.objects.get(id=1)
While I believe there should be:
   def test_date_of_death_label(self):
        author = Author.objects.get(id=1)
Explaination:
PEP 8: Missing whitespace around operator (=)
diff --git a/files/en-us/learn/server-side/django/testing/index.html b/files/en-us/learn/server-side/django/testing/index.html
@@ -321,7 +321,7 @@ <h3 id="Models">Models</h3>
         self.assertEqual(field_label, 'first name')
 
     def test_date_of_death_label(self):
-        author=Author.objects.get(id=1)
+        author = Author.objects.get(id=1)
         field_label = author._meta.get_field('date_of_death').verbose_name
         self.assertEqual(field_label, 'died')
 
@@ -436,7 +436,7 @@ <h3 id="Forms">Forms</h3>
 class RenewBookFormTest(TestCase):
     def test_renew_form_date_field_label(self):
         form = RenewBookForm()
-        self.assertTrue(form.fields['renewal_date'].label == None or form.fields['renewal_date'].label == 'renewal date')
+        self.assertTrue(form.fields['renewal_date'].label is None or form.fields['renewal_date'].label == 'renewal date')
 
     def test_renew_form_date_field_help_text(self):
         form = RenewBookForm()
