patchtest2: use black to reformat

Signed-off-by: Trevor Gamblin <[email protected]>

Author: threexc

src/patchtest2/mbox.py

@@ -17,6 +17,7 @@
 
 from dataclasses import dataclass
 
+
 # From: https://stackoverflow.com/questions/59681461/read-a-big-mbox-file-with-python
 class MboxReader:
     def __init__(self, filepath):
@@ -56,17 +57,20 @@ def __init__(self, data):
         self.diff = self.split_body[1]
         # get the shortlog, but make sure to exclude bracketed prefixes
         # before the colon, and remove extra whitespace/newlines
-        self.shortlog = self.subject[self.subject.find(']', 0,
-            self.subject.find(':')) + 1:].replace('\n', '').strip()
+        self.shortlog = (
+            self.subject[self.subject.find("]", 0, self.subject.find(":")) + 1 :]
+            .replace("\n", "")
+            .strip()
+        )
+
 
 class PatchSeries:
     def __init__(self, filepath):
         with MboxReader(filepath) as mbox:
             # Keep raw copies of messages in a list
             self.messages = [message for message in mbox]
             # Get a copy of each message's core patch contents
-            self.patchdata = [Patch(message) for message in
-                              self.messages]
+            self.patchdata = [Patch(message) for message in self.messages]
 
         assert self.patchdata
         self.patch_count = len(self.patchdata)
@@ -121,6 +125,7 @@ def valid_branch(branch):
 
         return not invalid
 
+
 class TargetRepo:
     def __init__(self, repodir):
         self.repodir = repodir
@@ -147,13 +152,12 @@ def can_be_merged(self, patchfile):
         except git.exc.GitCommandError as ce:
             result = ce
             self.abort_merge()
-        
+
         return result
 
     def merge_patch(self, patchfile):
         self.repo.git.execute(
-            ["git", "am", "--keep-cr", os.path.abspath(patchfile)],
-             with_exceptions=True
+            ["git", "am", "--keep-cr", os.path.abspath(patchfile)], with_exceptions=True
         )
 
     def abort_merge(self):

src/patchtest2/tests/core.py

@@ -1,41 +1,60 @@
 import patchtest2.patterns as patterns
 import pyparsing
 
+
 class PatchtestResult:
     def __init__(self, patch, testname, result, reason):
         self.patch = patch
         self.testname = testname
         self.result = result
         self.reason = reason
         self.pass_string = f"{self.result}: {self.testname} on {self.patch}"
-        self.skip_or_fail_string = f"{self.result}: {self.testname} on {self.patch} ({self.reason})"
+        self.skip_or_fail_string = (
+            f"{self.result}: {self.testname} on {self.patch} ({self.reason})"
+        )
 
     def __str__(self):
         if self.result == "PASS":
             return self.pass_string
         else:
             return self.skip_or_fail_string
 
+
 class PatchtestResults:
     def __init__(self, target_repo, series):
         self.target_repo = target_repo
         self.series = series
         self.mbox_results = dict(
-                [
-                ('signed_off_by', [test_mbox_signed_off_by_presence(patch) for
-                                    patch in self.series.patchdata]),
-                ('shortlog_format', [test_mbox_shortlog_format(patch) for patch
-                                      in self.series.patchdata]),
-                ('commit_message_presence',
-                  [test_mbox_commit_message_presence(patch) for patch in
-                   self.series.patchdata]),
-                ]
-                )
+            [
+                (
+                    "signed_off_by",
+                    [
+                        test_mbox_signed_off_by_presence(patch)
+                        for patch in self.series.patchdata
+                    ],
+                ),
+                (
+                    "shortlog_format",
+                    [
+                        test_mbox_shortlog_format(patch)
+                        for patch in self.series.patchdata
+                    ],
+                ),
+                (
+                    "commit_message_presence",
+                    [
+                        test_mbox_commit_message_presence(patch)
+                        for patch in self.series.patchdata
+                    ],
+                ),
+            ]
+        )
 
     def print_mbox_results(self, tag):
         for testresult in self.mbox_results[tag]:
             print(testresult)
 
+
 # test_for_pattern()
 # @pattern: a pyparsing regex object
 # @string: a string (patch subject, commit message, author, etc. to
@@ -46,15 +65,13 @@ def test_for_pattern(pattern, string):
     else:
         return "FAIL"
 
+
 def test_mbox_signed_off_by_presence(target):
     test_name = "test_mbox_signed_off_by_presence"
-    result = test_for_pattern(patterns.signed_off_by,
-                              target.commit_message)
+    result = test_for_pattern(patterns.signed_off_by, target.commit_message)
     reason = "mbox was missing a signed-off-by tag"
-    return PatchtestResult(target.subject,
-                           test_name,
-                           result,
-                           reason)
+    return PatchtestResult(target.subject, test_name, result, reason)
+
 
 def test_mbox_shortlog_format(target):
     test_name = "test_mbox_shortlog_format"
@@ -74,10 +91,8 @@ def test_mbox_shortlog_format(target):
         result = "FAIL"
         reason = 'Commit shortlog (first line of commit message) should follow the format "<target>: <summary>"'
 
-    return PatchtestResult(target.subject,
-                           test_name,
-                           result,
-                           reason)
+    return PatchtestResult(target.subject, test_name, result, reason)
+
 
 def test_mbox_commit_message_presence(target):
     test_name = "test_mbox_commit_message_presence"
@@ -86,10 +101,7 @@ def test_mbox_commit_message_presence(target):
 
     # Check to see if there is content before the signoff
     match = patterns.endcommit_messages_regex.search(target.commit_message)
-    if not target.commit_message[:match.start()]:
+    if not target.commit_message[: match.start()]:
         result = "FAIL"
 
-    return PatchtestResult(target.subject,
-                           test_name,
-                           result,
-                           reason)
+    return PatchtestResult(target.subject, test_name, result, reason)

src/patchtest2/tests/patchtest.py

@@ -2,13 +2,14 @@
 from patchtest2.mbox import PatchSeries, TargetRepo
 from patchtest2.tests.core import PatchtestResults
 
+
 def run():
     parser = PatchtestParser.get_parser()
     args = parser.parse_args()
     target_repo = TargetRepo(args.repodir)
     series = PatchSeries(args.patch_path)
     results = PatchtestResults(target_repo, series)
 
-    results.print_mbox_results('signed_off_by')
-    results.print_mbox_results('shortlog_format')
-    results.print_mbox_results('commit_message_presence')
+    results.print_mbox_results("signed_off_by")
+    results.print_mbox_results("shortlog_format")
+    results.print_mbox_results("commit_message_presence")