tests: fix black formatting on tests

tests/test_pagure_flask_api_issue.py

@@ -1579,7 +1579,7 @@ class PagureFlaskApiIssuetests(tests.SimplePagureTest):
         data = {
             "title": "test issue",
             "issue_content": "This issue needs attention",
-            "private": True
+            "private": True,
         }
 
         # Create an issue

tests/test_pagure_flask_api_user.py

@@ -427,7 +427,9 @@ class PagureFlaskApiUSertests(tests.Modeltests):
         data = json.loads(output.get_data(as_text=True))
         exp = {
             "activities": [
-                {"description_mk": '<div class="markdown"><p>pingou committed on test#githash</p></div>'}
+                {
+                    "description_mk": '<div class="markdown"><p>pingou committed on test#githash</p></div>'
+                }
             ],
             "date": date,
         }
@@ -532,7 +534,9 @@ class PagureFlaskApiUSertests(tests.Modeltests):
         data = json.loads(output.get_data(as_text=True))
         exp = {
             "activities": [
-                {"description_mk": '<div class="markdown"><p>pingou committed on test#githash</p></div>'}
+                {
+                    "description_mk": '<div class="markdown"><p>pingou committed on test#githash</p></div>'
+                }
             ],
             "date": utcdate,
         }
@@ -641,7 +645,9 @@ class PagureFlaskApiUSertests(tests.Modeltests):
         data = json.loads(output.get_data(as_text=True))
         exp = {
             "activities": [
-                {"description_mk": '<div class="markdown"><p>pingou committed on test#githash</p></div>'}
+                {
+                    "description_mk": '<div class="markdown"><p>pingou committed on test#githash</p></div>'
+                }
             ],
             "date": utcdate,
         }

tests/test_pagure_flask_ui_app.py

@@ -1440,7 +1440,9 @@ class PagureFlaskApptests(tests.Modeltests):
         text = "Cf commit %s" % first_commit.oid.hex
         exp = (
             '<div class="markdown"><p>Cf commit <a href="/test/c/{0}" title="Commit {0}">{1}'
-            "</a></p></div>".format(first_commit.oid.hex, first_commit.oid.hex[:7])
+            "</a></p></div>".format(
+                first_commit.oid.hex, first_commit.oid.hex[:7]
+            )
         )
 
         with self.app.application.app_context():

tests/test_pagure_flask_ui_fork.py

@@ -2564,7 +2564,7 @@ More information</textarea>
             self.assertIn(
                 '<a class="dropdown-item branch_from_item pointer" '
                 'data-value="master"><span class="fa fa-random">'
-                '</span> master</a>',
+                "</span> master</a>",
                 output_text,
             )
 
@@ -2789,7 +2789,7 @@ More information</textarea>
             self.assertIn(
                 '<a class="dropdown-item branch_from_item pointer" '
                 'data-value="master"><span class="fa fa-random">'
-                '</span> master</a>',
+                "</span> master</a>",
                 output_text,
             )
 
@@ -3106,13 +3106,13 @@ More information</textarea>
             self.assertIn(
                 '<a class="dropdown-item branch_from_item pointer" '
                 'data-value="master"><span class="fa fa-random">'
-                '</span> master</a>',
+                "</span> master</a>",
                 output_text,
             )
             self.assertIn(
                 '<a class="dropdown-item branch_from_item pointer" '
                 'data-value="random_branch"><span class="fa fa-random">'
-                '</span> random_branch</a>',
+                "</span> random_branch</a>",
                 output_text,
             )
 
@@ -3208,13 +3208,13 @@ More information</textarea>
             self.assertIn(
                 '<a class="dropdown-item branch_from_item pointer" '
                 'data-value="master"><span class="fa fa-random">'
-                '</span> master</a>',
+                "</span> master</a>",
                 output_text,
             )
             self.assertIn(
                 '<a class="dropdown-item branch_from_item pointer" '
                 'data-value="random_branch"><span class="fa fa-random">'
-                '</span> random_branch</a>',
+                "</span> random_branch</a>",
                 output_text,
             )
 
@@ -4776,12 +4776,12 @@ class TestTicketAccessEditPRMetadata(tests.Modeltests):
             output_text,
         )
         self.assertIn(
-                '<i class="fa fa-fw fa-pencil"></i></a>',
-                '<a class="btn btn-outline-primary border-0 btn-sm '
-                'issue-metadata-display editmetadatatoggle pointer inline-block'
-                '"><i class="fa fa-fw fa-pencil"></i></a>',
-                output_text,
-            )
+            '<i class="fa fa-fw fa-pencil"></i></a>',
+            '<a class="btn btn-outline-primary border-0 btn-sm '
+            "issue-metadata-display editmetadatatoggle pointer inline-block"
+            '"><i class="fa fa-fw fa-pencil"></i></a>',
+            output_text,
+        )
         self.assertNotIn(
             '<form method="POST" action="/test/pull-request/1/update">',
             output_text,
@@ -4804,7 +4804,7 @@ class TestTicketAccessEditPRMetadata(tests.Modeltests):
             self.assertIn(
                 '<i class="fa fa-fw fa-pencil"></i></a>',
                 '<a class="btn btn-outline-primary border-0 btn-sm '
-                'issue-metadata-display editmetadatatoggle pointer inline-block'
+                "issue-metadata-display editmetadatatoggle pointer inline-block"
                 '"><i class="fa fa-fw fa-pencil"></i></a>',
                 output_text,
             )
@@ -4832,7 +4832,7 @@ class TestTicketAccessEditPRMetadata(tests.Modeltests):
             self.assertIn(
                 '<i class="fa fa-fw fa-pencil"></i></a>',
                 '<a class="btn btn-outline-primary border-0 btn-sm '
-                'issue-metadata-display editmetadatatoggle pointer inline-block'
+                "issue-metadata-display editmetadatatoggle pointer inline-block"
                 '"><i class="fa fa-fw fa-pencil"></i></a>',
                 output_text,
             )
@@ -4863,7 +4863,7 @@ class TestTicketAccessEditPRMetadata(tests.Modeltests):
             self.assertIn(
                 '<i class="fa fa-fw fa-pencil"></i></a>',
                 '<a class="btn btn-outline-primary border-0 btn-sm '
-                'issue-metadata-display editmetadatatoggle pointer inline-block'
+                "issue-metadata-display editmetadatatoggle pointer inline-block"
                 '"><i class="fa fa-fw fa-pencil"></i></a>',
                 output_text,
             )
@@ -4893,7 +4893,7 @@ class TestTicketAccessEditPRMetadata(tests.Modeltests):
             self.assertIn(
                 '<i class="fa fa-fw fa-pencil"></i></a>',
                 '<a class="btn btn-outline-primary border-0 btn-sm '
-                'issue-metadata-display editmetadatatoggle pointer inline-block'
+                "issue-metadata-display editmetadatatoggle pointer inline-block"
                 '"><i class="fa fa-fw fa-pencil"></i></a>',
                 output_text,
             )

tests/test_pagure_flask_ui_issues.py

@@ -1355,9 +1355,7 @@ class PagureFlaskIssuestests(tests.Modeltests):
             # Not author nor admin = No take
             self.assertNotIn("function take_issue(){", output_text)
             self.assertNotIn("function drop_issue(){", output_text)
-            self.assertNotIn(
-                '<a class="pointer" id="take-btn"\n', output_text
-            )
+            self.assertNotIn('<a class="pointer" id="take-btn"\n', output_text)
 
         user.username = "pingou"
         with tests.user_set(self.app.application, user):
@@ -1507,9 +1505,7 @@ class PagureFlaskIssuestests(tests.Modeltests):
             # author admin = take
             self.assertIn("function take_issue(){", output_text)
             self.assertIn("function drop_issue(){", output_text)
-            self.assertIn(
-                '<a class="pointer" id="take-btn"\n', output_text
-            )
+            self.assertIn('<a class="pointer" id="take-btn"\n', output_text)
 
     @patch("pagure.lib.git.update_git")
     @patch("pagure.lib.notify.send_email")
@@ -1586,9 +1582,7 @@ class PagureFlaskIssuestests(tests.Modeltests):
             # user has ticket = take ok
             self.assertIn("function take_issue(){", output_text)
             self.assertIn("function drop_issue(){", output_text)
-            self.assertIn(
-                '<a class="pointer" id="take-btn"\n', output_text
-            )
+            self.assertIn('<a class="pointer" id="take-btn"\n', output_text)
 
     @patch("pagure.lib.git.update_git")
     @patch("pagure.lib.notify.send_email")
@@ -1658,9 +1652,7 @@ class PagureFlaskIssuestests(tests.Modeltests):
             # user no ACLs = no take action/button
             self.assertNotIn("function take_issue(){", output_text)
             self.assertNotIn("function drop_issue(){", output_text)
-            self.assertNotIn(
-                '<a class="pointer" id="take-btn"\n', output_text
-            )
+            self.assertNotIn('<a class="pointer" id="take-btn"\n', output_text)
 
             # user no ACLs = no metadata form
             self.assertNotIn(
@@ -1701,9 +1693,7 @@ class PagureFlaskIssuestests(tests.Modeltests):
             # user has ticket = take ok
             self.assertIn("function take_issue(){", output_text)
             self.assertIn("function drop_issue(){", output_text)
-            self.assertIn(
-                '<a class="pointer" id="take-btn"\n', output_text
-            )
+            self.assertIn('<a class="pointer" id="take-btn"\n', output_text)
 
             # user has ticket == Sees the metadata
             self.assertIn(
@@ -3766,7 +3756,7 @@ class PagureFlaskIssuestests(tests.Modeltests):
             self.assertTrue('<section class="edit_comment">' in output_text)
             self.assertIn(
                 '<textarea class="form-control width-100per" id="update_comment"',
-                output_text
+                output_text,
             )
 
             csrf_token = self.get_csrf(output=output)

tests/test_pagure_flask_ui_issues_acl_checks.py

@@ -463,7 +463,7 @@ class PagureFlaskIssuesACLtests(tests.Modeltests):
             # edit metadata
             self.assertIn(
                 '<a class="btn btn-outline-primary border-0 btn-sm '
-                'issue-metadata-display editmetadatatoggle pointer inline-block'
+                "issue-metadata-display editmetadatatoggle pointer inline-block"
                 '"><i class="fa fa-fw fa-pencil"></i></a>',
                 output_text,
             )
@@ -718,7 +718,7 @@ class PagureFlaskIssuesACLtests(tests.Modeltests):
             # edit metadata
             self.assertIn(
                 '<a class="btn btn-outline-primary border-0 btn-sm '
-                'issue-metadata-display editmetadatatoggle pointer inline-block'
+                "issue-metadata-display editmetadatatoggle pointer inline-block"
                 '"><i class="fa fa-fw fa-pencil"></i></a>',
                 output_text,
             )
@@ -970,7 +970,7 @@ class PagureFlaskIssuesACLtests(tests.Modeltests):
 
             self.assertIn(
                 '<a class="btn btn-outline-primary border-0 btn-sm '
-                'issue-metadata-display editmetadatatoggle pointer inline-block'
+                "issue-metadata-display editmetadatatoggle pointer inline-block"
                 '"><i class="fa fa-fw fa-pencil"></i></a>',
                 output_text,
             )

tests/test_pagure_flask_ui_issues_open_access.py

@@ -241,9 +241,7 @@ class PagureFlaskIssuesOpenAccesstests(tests.Modeltests):
             # Not author nor admin but open_access = take
             self.assertIn("function take_issue(){", output_text)
             self.assertNotIn("function drop_issue(){", output_text)
-            self.assertIn(
-                '<a class="pointer" id="take-btn"\n', output_text
-            )
+            self.assertIn('<a class="pointer" id="take-btn"\n', output_text)
 
             csrf_token = self.get_csrf(output=output)
 
@@ -352,9 +350,7 @@ class PagureFlaskIssuesOpenAccesstests(tests.Modeltests):
             # user has ticket = take ok
             self.assertIn("function take_issue(){", output_text)
             self.assertIn("function drop_issue(){", output_text)
-            self.assertIn(
-                '<a class="pointer" id="take-btn"\n', output_text
-            )
+            self.assertIn('<a class="pointer" id="take-btn"\n', output_text)
 
     @patch("pagure.lib.git.update_git", MagicMock(return_value=True))
     @patch("pagure.lib.notify.send_email", MagicMock(return_value=True))
@@ -415,9 +411,7 @@ class PagureFlaskIssuesOpenAccesstests(tests.Modeltests):
             # user no ACLs but open_access = take action/button - no drop
             self.assertIn("function take_issue(){", output_text)
             self.assertNotIn("function drop_issue(){", output_text)
-            self.assertIn(
-                '<a class="pointer" id="take-btn"\n', output_text
-            )
+            self.assertIn('<a class="pointer" id="take-btn"\n', output_text)
 
             # user no ACLs = no metadata form
             self.assertNotIn(
@@ -458,9 +452,7 @@ class PagureFlaskIssuesOpenAccesstests(tests.Modeltests):
             # user has ticket = take ok
             self.assertIn("function take_issue(){", output_text)
             self.assertIn("function drop_issue(){", output_text)
-            self.assertIn(
-                '<a class="pointer" id="take-btn"\n', output_text
-            )
+            self.assertIn('<a class="pointer" id="take-btn"\n', output_text)
 
             # user has ticket == Sees the metadata
             self.assertIn(
@@ -1159,7 +1151,7 @@ class PagureFlaskIssuesOpenAccesstests(tests.Modeltests):
             self.assertTrue('<section class="edit_comment">' in output_text)
             self.assertIn(
                 '<textarea class="form-control width-100per" id="update_comment"',
-                output_text
+                output_text,
             )
 
             csrf_token = self.get_csrf(output=output)
@@ -1231,7 +1223,7 @@ class PagureFlaskIssuesOpenAccesstests(tests.Modeltests):
             self.assertIn("Comment added", output_text)
             self.assertIn(
                 '<a class="btn btn-outline-primary border-0 btn-sm '
-                'issue-metadata-display editmetadatatoggle pointer inline-block'
+                "issue-metadata-display editmetadatatoggle pointer inline-block"
                 '"><i class="fa fa-fw fa-pencil"></i></a>',
                 output_text,
             )
@@ -1258,7 +1250,7 @@ class PagureFlaskIssuesOpenAccesstests(tests.Modeltests):
             )
             self.assertIn(
                 '<a class="btn btn-outline-primary border-0 btn-sm '
-                'issue-metadata-display editmetadatatoggle pointer inline-block'
+                "issue-metadata-display editmetadatatoggle pointer inline-block"
                 '"><i class="fa fa-fw fa-pencil"></i></a>',
                 output_text,
             )

tests/test_pagure_flask_ui_repo.py

@@ -2351,9 +2351,7 @@ class PagureFlaskRepotests(tests.Modeltests):
                 ),
                 2,
             )
-            self.assertIn(
-                '<a class="pointer">1 more commits...', output_text
-            )
+            self.assertIn('<a class="pointer">1 more commits...', output_text)
             self.assertIn(
                 'title="View file as of 4829cf">Šource</a>', output_text
             )
@@ -2385,9 +2383,7 @@ class PagureFlaskRepotests(tests.Modeltests):
                 '<pre class="alert-danger"><code>- Row 0</code></pre>',
                 output_text,
             )
-            self.assertIn(
-                '<a class="pointer">1 more commits...', output_text
-            )
+            self.assertIn('<a class="pointer">1 more commits...', output_text)
             self.assertIn(
                 'title="View file as of 000000">Šource</a>', output_text
             )
@@ -2880,8 +2876,7 @@ class PagureFlaskRepotests(tests.Modeltests):
         self.assertTrue(
             '<tr><td class="cell1"><a id="1" href="#1" '
             'data-line-number="1"></a></td>' in output_text
-            or
-            '<tr><td class="cell1"><a data-line-number="1" '
+            or '<tr><td class="cell1"><a data-line-number="1" '
             'href="#1" id="1"></a></td>' in output_text
         )
         self.assertIn(
@@ -2904,8 +2899,7 @@ class PagureFlaskRepotests(tests.Modeltests):
         self.assertTrue(
             '<tr><td class="cell1"><a id="1" href="#1" '
             'data-line-number="1"></a></td>' in output_text
-            or
-            '<tr><td class="cell1"><a data-line-number="1" '
+            or '<tr><td class="cell1"><a data-line-number="1" '
             'href="#1" id="1"></a></td>' in output_text
         )
         self.assertIn(
@@ -2923,8 +2917,7 @@ class PagureFlaskRepotests(tests.Modeltests):
         self.assertTrue(
             '<tr><td class="cell1"><a id="1" href="#1" '
             'data-line-number="1"></a></td>' in output_text
-            or
-            '<tr><td class="cell1"><a data-line-number="1" '
+            or '<tr><td class="cell1"><a data-line-number="1" '
             'href="#1" id="1"></a></td>' in output_text
         )
         self.assertIn(
@@ -2975,8 +2968,7 @@ class PagureFlaskRepotests(tests.Modeltests):
         self.assertTrue(
             '<tr><td class="cell1"><a id="1" href="#1" '
             'data-line-number="1"></a></td>' in output_text
-            or
-            '<tr><td class="cell1"><a data-line-number="1" '
+            or '<tr><td class="cell1"><a data-line-number="1" '
             'href="#1" id="1"></a></td>' in output_text
         )
         self.assertIn(
@@ -3017,8 +3009,7 @@ class PagureFlaskRepotests(tests.Modeltests):
         self.assertTrue(
             '<tr><td class="cell1"><a id="1" href="#1" '
             'data-line-number="1"></a></td>' in output_text
-            or
-            '<tr><td class="cell1"><a data-line-number="1" '
+            or '<tr><td class="cell1"><a data-line-number="1" '
             'href="#1" id="1"></a></td>' in output_text
         )
         self.assertIn(
@@ -3058,8 +3049,7 @@ class PagureFlaskRepotests(tests.Modeltests):
         self.assertTrue(
             '<tr><td class="cell1"><a id="1" href="#1" '
             'data-line-number="1"></a></td>' in output_text
-            or
-            '<tr><td class="cell1"><a data-line-number="1" '
+            or '<tr><td class="cell1"><a data-line-number="1" '
             'href="#1" id="1"></a></td>' in output_text
         )
         self.assertIn(

tests/test_pagure_flask_ui_repo_view_file.py

@@ -244,7 +244,7 @@ class PagureFlaskRepoViewFiletests(LocalBasetests):
         )
         self.assertIn(
             '<button class="btn btn-sm btn-secondary fork_project_btn">'
-                '\n                            Fork and Edit',
+            "\n                            Fork and Edit",
             output_text,
         )
 
@@ -267,7 +267,7 @@ class PagureFlaskRepoViewFiletests(LocalBasetests):
             )
             self.assertIn(
                 '<button class="btn btn-sm btn-secondary fork_project_btn">'
-                '\n                            Fork and Edit',
+                "\n                            Fork and Edit",
                 output_text,
             )
 
@@ -360,7 +360,7 @@ class PagureFlaskRepoViewFileForktests(LocalBasetests):
         )
         self.assertIn(
             '<button class="btn btn-sm btn-secondary fork_project_btn">'
-                '\n                            Fork and Edit',
+            "\n                            Fork and Edit",
             output_text,
         )
 
@@ -382,7 +382,7 @@ class PagureFlaskRepoViewFileForktests(LocalBasetests):
             )
             self.assertNotIn(
                 '<button class="btn btn-sm btn-secondary fork_project_btn">'
-                '\n                            Fork and Edit',
+                "\n                            Fork and Edit",
                 output_text,
             )
 
@@ -406,7 +406,7 @@ class PagureFlaskRepoViewFileForktests(LocalBasetests):
             )
             self.assertIn(
                 '<button class="btn btn-sm btn-secondary fork_project_btn">'
-                '\n                            Fork and Edit',
+                "\n                            Fork and Edit",
                 output_text,
             )
 
@@ -429,7 +429,7 @@ class PagureFlaskRepoViewFileForktests(LocalBasetests):
             )
             self.assertIn(
                 '<button class="btn btn-sm btn-secondary fork_project_btn">'
-                '\n                            Edit in your fork',
+                "\n                            Edit in your fork",
                 output_text,
             )