Merge pull request #101 from treyhunner/code-cleanup

Basic code style cleanup

Author: treyhunner

simple_history/tests/tests/test_admin.py

@@ -9,7 +9,7 @@
     from django.contrib.auth.models import User
 from django.contrib.admin.util import quote
 
-from ..models import Book, Document, Person, Poll
+from ..models import Book, Person, Poll
 
 
 today = datetime(2021, 1, 1, 10, 0)
@@ -139,7 +139,8 @@ def test_underscore_in_pk(self):
         book._history_user = self.user
         book.save()
         response = self.app.get(get_history_url(book))
-        self.assertIn(book.history.all()[0].revert_url(), response.unicode_normal_body)
+        self.assertIn(book.history.all()[0].revert_url(),
+                      response.unicode_normal_body)
 
     def test_historical_user_no_setter(self):
         """Demonstrate admin error without `_historical_user` setter.

simple_history/tests/tests/test_models.py

@@ -358,24 +358,24 @@ def test_register_app_label(self):
                          'tests_historicalexternalmodel4')
 
     def test_get_model(self):
-        self.assertEqual(get_model('external','ExternalModel1'),
+        self.assertEqual(get_model('external', 'ExternalModel1'),
                          ExternalModel1)
-        self.assertEqual(get_model('external','HistoricalExternalModel1'),
+        self.assertEqual(get_model('external', 'HistoricalExternalModel1'),
                          ExternalModel1.history.model)
 
-        self.assertEqual(get_model('external','ExternalModel2'),
+        self.assertEqual(get_model('external', 'ExternalModel2'),
                          ExternalModel2)
-        self.assertEqual(get_model('external','HistoricalExternalModel2'),
+        self.assertEqual(get_model('external', 'HistoricalExternalModel2'),
                          ExternalModel2.history.model)
 
-        self.assertEqual(get_model('tests','ExternalModel3'),
+        self.assertEqual(get_model('tests', 'ExternalModel3'),
                          ExternalModel3)
-        self.assertEqual(get_model('external','HistoricalExternalModel3'),
+        self.assertEqual(get_model('external', 'HistoricalExternalModel3'),
                          ExternalModel3.histories.model)
 
-        self.assertEqual(get_model('external','ExternalModel4'),
+        self.assertEqual(get_model('external', 'ExternalModel4'),
                          ExternalModel4)
-        self.assertEqual(get_model('tests','HistoricalExternalModel4'),
+        self.assertEqual(get_model('tests', 'HistoricalExternalModel4'),
                          ExternalModel4.histories.model)
 
 
@@ -392,9 +392,9 @@ def test_most_recent(self):
         self.assertEqual(most_recent.question, "why?")
 
     def test_get_model(self):
-        self.assertEqual(get_model('tests','poll'),
+        self.assertEqual(get_model('tests', 'poll'),
                          Poll)
-        self.assertEqual(get_model('tests','historicalpoll'),
+        self.assertEqual(get_model('tests', 'historicalpoll'),
                          HistoricalPoll)
 
     def test_most_recent_on_model_class(self):