From: Trevor Gamblin <tgamblin@baylibre.com>
Date: Thu, 18 Jun 2026 20:36:30 +0000 (-0400)
Subject: scripts/patchtest: simplify run()
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=11c046d9db7d5f10fce53eb068b095975a814855;p=thirdparty%2Fopenembedded%2Fopenembedded-core.git

scripts/patchtest: simplify run()

Set premerge_result and postmerge_result directly by calling
getResult(), instead of doing it in multiple stages with confusing
variable names.

Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
Signed-off-by: Mathieu Dubois-Briand <mathieu.dubois-briand@bootlin.com>
Signed-off-by: Ross Burton <ross.burton@arm.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---

diff --git a/scripts/patchtest b/scripts/patchtest
index aef80a86bb..d2f6302743 100755
--- a/scripts/patchtest
+++ b/scripts/patchtest
@@ -141,13 +141,8 @@ def run(patch, logfile=None):
     # Get the result class and install the control-c handler
     unittest.installHandler()
 
-    # run pre-merge tests, meaning those methods with 'pretest' as prefix
-    premerge_resultklass = getResult(patch, False, logfile)
-    premerge_result = _runner(premerge_resultklass, 'pretest')
-
-    # run post-merge tests, meaning those methods with 'test' as prefix
-    postmerge_resultklass = getResult(patch, True, logfile)
-    postmerge_result = _runner(postmerge_resultklass, 'test')
+    premerge_result = _runner(getResult(patch, False, logfile), 'pretest')
+    postmerge_result = _runner(getResult(patch, True, logfile), 'test')
 
     print_result_message(premerge_result, postmerge_result)
     return premerge_result or postmerge_result