diff mbox series

[2/7] scripts/patchtest: clean up startTestRun()

Message ID 20260618203632.131125-3-tgamblin@baylibre.com
State New
Headers show
Series patchtest: cleanups and fixes, part 1 | expand

Commit Message

Trevor Gamblin June 18, 2026, 8:36 p.m. UTC 
- Initialize self.repo directly instead of in two steps
- Remove extra whitespace when setting error and failure members

Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
---
 scripts/patchtest | 21 +++++++++------------
 1 file changed, 9 insertions(+), 12 deletions(-)
diff mbox series

Patch

diff --git a/scripts/patchtest b/scripts/patchtest
index 07c3ada6f7..ff8e8b11bd 100755
--- a/scripts/patchtest
+++ b/scripts/patchtest
@@ -61,20 +61,17 @@  def getResult(patch, mergepatch, logfile=None):
         skip        = 'SKIP'
 
         def startTestRun(self):
-            # let's create the repo already, it can be used later on
-            repoargs = {
-                "repodir": PatchtestParser.repodir,
-                "commit": PatchtestParser.basecommit,
-                "branch": PatchtestParser.basebranch,
-                "patch": patch,
-            }
-
-            self.repo_error    = False
-            self.test_error    = False
-            self.test_failure  = False
+            self.repo_error   = False
+            self.test_error   = False
+            self.test_failure = False
 
             try:
-                self.repo = PatchtestParser.repo = PatchTestRepo(**repoargs)
+                self.repo = PatchtestParser.repo = PatchTestRepo(
+                    patch=patch,
+                    repodir=PatchtestParser.repodir,
+                    commit=PatchtestParser.basecommit,
+                    branch=PatchtestParser.basebranch,
+                )
             except:
                 logger.error(traceback.print_exc())
                 self.repo_error = True