Add a minor refactor for the result output.
Signed-off-by: Trevor Gamblin <[email protected]>
---
scripts/patchtest | 23 +++++++++++++++--------
1 file changed, 15 insertions(+), 8 deletions(-)
diff --git a/scripts/patchtest b/scripts/patchtest
index 278fc4e6e09..9218db232a7 100755
--- a/scripts/patchtest
+++ b/scripts/patchtest
@@ -165,16 +165,23 @@ def run(patch, logfile=None):
postmerge_resultklass = getResult(patch, True, logfile)
postmerge_result = _runner(postmerge_resultklass, 'test')
-
print('----------------------------------------------------------------------\n')
- if premerge_result == 2 and postmerge_result == 2:
- logger.error('patchtest: No test cases found - did you specify the
correct suite directory?')
- if premerge_result == 1 or postmerge_result == 1:
- logger.error('WARNING: patchtest: At least one patchtest caused a
failure or an error - please check https://wiki.yoctoproject.org/wiki/Patchtest
for further guidance')
- else:
- logger.info('OK: patchtest: All patchtests passed')
-
print('----------------------------------------------------------------------\n')
+ print_result_message(premerge_result, postmerge_result)
return premerge_result or postmerge_result
+def print_result_message(preresult, postresult):
+
print("----------------------------------------------------------------------\n")
+ if preresult == 2 and postresult == 2:
+ logger.error(
+ "patchtest: No test cases found - did you specify the correct
suite directory?"
+ )
+ if preresult == 1 or postresult == 1:
+ logger.error(
+ "WARNING: patchtest: At least one patchtest caused a failure or an
error - please check https://wiki.yoctoproject.org/wiki/Patchtest for further
guidance"
+ )
+ else:
+ logger.info("OK: patchtest: All patchtests passed")
+
print("----------------------------------------------------------------------\n")
+
def main():
tmp_patch = False
patch_path = PatchtestParser.patch_path
--
2.39.5
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#204855):
https://lists.openembedded.org/g/openembedded-core/message/204855
Mute This Topic: https://lists.openembedded.org/mt/108626676/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
