update to black 20.8b1 (#676)

Some trailing commas were removed because of some black changes.

1 files changed, 27 insertions, 42 deletions

diff --git a/ihatemoney/tests/tests.py b/ihatemoney/tests/tests.py
index cecc4b0..8f7f747 100644
--- a/ihatemoney/tests/tests.py
+++ b/ihatemoney/tests/tests.py
@@ -186,8 +186,7 @@ class BudgetTestCase(IhatemoneyTestCase):
             self.assertEqual(len(outbox), 0)
 
     def test_invite(self):
-        """Test that invitation e-mails are sent properly
-        """
+        """Test that invitation e-mails are sent properly"""
         self.login("raclette")
         self.post_project("raclette")
         with self.app.mail.record_messages() as outbox:
@@ -979,10 +978,10 @@ class BudgetTestCase(IhatemoneyTestCase):
             regex.format("fred", r"\$20\.00", r"\$5\.83"),
         )
         self.assertRegex(
-            response.data.decode("utf-8"), regex.format("tata", r"\$0\.00", r"\$2\.50"),
+            response.data.decode("utf-8"), regex.format("tata", r"\$0\.00", r"\$2\.50")
         )
         self.assertRegex(
-            response.data.decode("utf-8"), regex.format("pépé", r"\$0\.00", r"\$0\.00"),
+            response.data.decode("utf-8"), regex.format("pépé", r"\$0\.00", r"\$0\.00")
         )
 
         # Check that the order of participants in the sidebar table is the
@@ -1443,8 +1442,7 @@ class BudgetTestCase(IhatemoneyTestCase):
             self.fail("ExpectedException not raised")
 
     def test_access_other_projects(self):
-        """Test that accessing or editing bills and members from another project fails
-        """
+        """Test that accessing or editing bills and members from another project fails"""
         # Create project
         self.post_project("raclette")
 
@@ -1744,8 +1742,7 @@ class APITestCase(IhatemoneyTestCase):
         self.assertEqual(401, resp.status_code)
 
     def test_token_creation(self):
-        """Test that token of project is generated
-        """
+        """Test that token of project is generated"""
 
         # Create project
         resp = self.api_create("raclette")
@@ -2300,7 +2297,7 @@ class APITestCase(IhatemoneyTestCase):
             f"Participant {em_surround('zorglub')} added", resp.data.decode("utf-8")
         )
         self.assertIn(
-            f"Project {em_surround('raclette')} added", resp.data.decode("utf-8"),
+            f"Project {em_surround('raclette')} added", resp.data.decode("utf-8")
         )
         self.assertEqual(resp.data.decode("utf-8").count("<td> -- </td>"), 2)
         self.assertNotIn("127.0.0.1", resp.data.decode("utf-8"))
@@ -2326,7 +2323,7 @@ class ServerTestCase(IhatemoneyTestCase):
 
 class CommandTestCase(BaseTestCase):
     def test_generate_config(self):
-        """ Simply checks that all config file generation
+        """Simply checks that all config file generation
         - raise no exception
         - produce something non-empty
         """
@@ -2487,7 +2484,7 @@ class EmailFailureTestCase(IhatemoneyTestCase):
             )
             # Check that we are still on the same page (no redirection)
             self.assertIn(
-                "Invite people to join this project", resp.data.decode("utf-8"),
+                "Invite people to join this project", resp.data.decode("utf-8")
             )
 
 
@@ -2507,9 +2504,7 @@ class HistoryTestCase(IhatemoneyTestCase):
     def test_simple_create_logentry_no_ip(self):
         resp = self.client.get("/demo/history")
         self.assertEqual(resp.status_code, 200)
-        self.assertIn(
-            f"Project {em_surround('demo')} added", resp.data.decode("utf-8"),
-        )
+        self.assertIn(f"Project {em_surround('demo')} added", resp.data.decode("utf-8"))
         self.assertEqual(resp.data.decode("utf-8").count("<td> -- </td>"), 1)
         self.assertNotIn("127.0.0.1", resp.data.decode("utf-8"))
 
@@ -2552,15 +2547,13 @@ class HistoryTestCase(IhatemoneyTestCase):
             "This project has history disabled. New actions won't appear below. ",
             resp.data.decode("utf-8"),
         )
-        self.assertIn(
-            "Nothing to list", resp.data.decode("utf-8"),
-        )
+        self.assertIn("Nothing to list", resp.data.decode("utf-8"))
         self.assertNotIn(
             "The table below reflects actions recorded prior to disabling project history.",
             resp.data.decode("utf-8"),
         )
         self.assertNotIn(
-            "Some entries below contain IP addresses,", resp.data.decode("utf-8"),
+            "Some entries below contain IP addresses,", resp.data.decode("utf-8")
         )
         self.assertNotIn("127.0.0.1", resp.data.decode("utf-8"))
         self.assertNotIn("<td> -- </td>", resp.data.decode("utf-8"))
@@ -2587,11 +2580,9 @@ class HistoryTestCase(IhatemoneyTestCase):
             f"Project contact email changed to {em_surround('demo2@notmyidea.org')}",
             resp.data.decode("utf-8"),
         )
+        self.assertIn("Project private code changed", resp.data.decode("utf-8"))
         self.assertIn(
-            "Project private code changed", resp.data.decode("utf-8"),
-        )
-        self.assertIn(
-            f"Project renamed to {em_surround('demo2')}", resp.data.decode("utf-8"),
+            f"Project renamed to {em_surround('demo2')}", resp.data.decode("utf-8")
         )
         self.assertLess(
             resp.data.decode("utf-8").index("Project renamed "),
@@ -2743,11 +2734,9 @@ class HistoryTestCase(IhatemoneyTestCase):
             "The table below reflects actions recorded prior to disabling project history.",
             resp.data.decode("utf-8"),
         )
+        self.assertNotIn("Nothing to list", resp.data.decode("utf-8"))
         self.assertNotIn(
-            "Nothing to list", resp.data.decode("utf-8"),
-        )
-        self.assertNotIn(
-            "Some entries below contain IP addresses,", resp.data.decode("utf-8"),
+            "Some entries below contain IP addresses,", resp.data.decode("utf-8")
         )
 
         # Clear Existing Entries
@@ -2780,11 +2769,9 @@ class HistoryTestCase(IhatemoneyTestCase):
             "The table below reflects actions recorded prior to disabling project history.",
             resp.data.decode("utf-8"),
         )
-        self.assertNotIn(
-            "Nothing to list", resp.data.decode("utf-8"),
-        )
+        self.assertNotIn("Nothing to list", resp.data.decode("utf-8"))
         self.assertIn(
-            "Some entries below contain IP addresses,", resp.data.decode("utf-8"),
+            "Some entries below contain IP addresses,", resp.data.decode("utf-8")
         )
         self.assertEqual(resp.data.decode("utf-8").count("127.0.0.1"), 10)
         self.assertEqual(resp.data.decode("utf-8").count("<td> -- </td>"), 1)
@@ -2808,11 +2795,9 @@ class HistoryTestCase(IhatemoneyTestCase):
             "The table below reflects actions recorded prior to disabling project history.",
             resp.data.decode("utf-8"),
         )
+        self.assertNotIn("Nothing to list", resp.data.decode("utf-8"))
         self.assertNotIn(
-            "Nothing to list", resp.data.decode("utf-8"),
-        )
-        self.assertNotIn(
-            "Some entries below contain IP addresses,", resp.data.decode("utf-8"),
+            "Some entries below contain IP addresses,", resp.data.decode("utf-8")
         )
         self.assertEqual(resp.data.decode("utf-8").count("127.0.0.1"), 0)
         self.assertEqual(resp.data.decode("utf-8").count("<td> -- </td>"), 16)
@@ -2882,7 +2867,7 @@ class HistoryTestCase(IhatemoneyTestCase):
         )
         self.assertIn(
             "Bill %s renamed to %s"
-            % (em_surround("fromage à raclette"), em_surround("new thing"),),
+            % (em_surround("fromage à raclette"), em_surround("new thing")),
             resp.data.decode("utf-8"),
         )
         self.assertLess(
@@ -2899,7 +2884,7 @@ class HistoryTestCase(IhatemoneyTestCase):
         resp = self.client.get("/demo/history")
         self.assertEqual(resp.status_code, 200)
         self.assertIn(
-            f"Bill {em_surround('new thing')} removed", resp.data.decode("utf-8"),
+            f"Bill {em_surround('new thing')} removed", resp.data.decode("utf-8")
         )
 
         # edit user
@@ -2923,7 +2908,7 @@ class HistoryTestCase(IhatemoneyTestCase):
         )
         self.assertIn(
             "Participant %s renamed to %s"
-            % (em_surround("zorglub"), em_surround("new name"),),
+            % (em_surround("zorglub"), em_surround("new name")),
             resp.data.decode("utf-8"),
         )
         self.assertLess(
@@ -3039,7 +3024,7 @@ class HistoryTestCase(IhatemoneyTestCase):
         self.assertNotIn("127.0.0.1", resp.data.decode("utf-8"))
         self.assertIn(f"Bill {em_surround('Bill 1')} added", resp.data.decode("utf-8"))
         self.assertIn(
-            f"Bill {em_surround('Bill 1')} removed", resp.data.decode("utf-8"),
+            f"Bill {em_surround('Bill 1')} removed", resp.data.decode("utf-8")
         )
 
         # Add a new bill
@@ -3060,11 +3045,11 @@ class HistoryTestCase(IhatemoneyTestCase):
         self.assertNotIn("127.0.0.1", resp.data.decode("utf-8"))
         self.assertIn(f"Bill {em_surround('Bill 1')} added", resp.data.decode("utf-8"))
         self.assertEqual(
-            resp.data.decode("utf-8").count(f"Bill {em_surround('Bill 1')} added"), 1,
+            resp.data.decode("utf-8").count(f"Bill {em_surround('Bill 1')} added"), 1
         )
         self.assertIn(f"Bill {em_surround('Bill 2')} added", resp.data.decode("utf-8"))
         self.assertIn(
-            f"Bill {em_surround('Bill 1')} removed", resp.data.decode("utf-8"),
+            f"Bill {em_surround('Bill 1')} removed", resp.data.decode("utf-8")
         )
 
     def test_double_bill_double_person_edit_second_no_web(self):
@@ -3075,8 +3060,8 @@ class HistoryTestCase(IhatemoneyTestCase):
         models.db.session.add(u2)
         models.db.session.commit()
 
-        b1 = models.Bill(what="Bill 1", payer_id=u1.id, owers=[u2], amount=10,)
-        b2 = models.Bill(what="Bill 2", payer_id=u2.id, owers=[u2], amount=11,)
+        b1 = models.Bill(what="Bill 1", payer_id=u1.id, owers=[u2], amount=10)
+        b2 = models.Bill(what="Bill 2", payer_id=u2.id, owers=[u2], amount=11)
 
         # This db commit exposes the "spurious owers edit" bug
         models.db.session.add(b1)