[PATCHv2 3/3] lnst-ctl: use the new print_summary method

olichtne at redhat.com olichtne at redhat.com
Fri May 31 09:06:40 UTC 2013


From: Ondrej Lichtner <olichtne at redhat.com>

This commit removes the old function that was used to print the summary
of test results. Instead the summary is printed via the use of the
NetTestResultSerializer method print_summary that was added in the
previous commit.

Signed-off-by: Ondrej Lichtner <olichtne at redhat.com>
---
 lnst-ctl | 12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)

diff --git a/lnst-ctl b/lnst-ctl
index 1ed3515..ce75d68 100755
--- a/lnst-ctl
+++ b/lnst-ctl
@@ -60,16 +60,6 @@ def process_recipe(action, file_path, cleanup, res_serializer,
         logging.error("Unknown action \"%s\"" % action)
         usage();
 
-def print_summary(summary):
-    logging.info("====================== SUMMARY ======================")
-    for recipe_file, res in summary:
-        if res:
-            res = "PASS"
-        else:
-            res = "FAIL"
-        logging.info("*%s* %s" % (res, recipe_file))
-    logging.info("=====================================================")
-
 def get_recipe_result(args, file_path, cleanup, res_serializer, packet_capture,
                       config, log_ctl, pool_checks):
     res_serializer.add_recipe(file_path)
@@ -178,7 +168,7 @@ def main():
 
     log_ctl.set_recipe("", clean=False)
 
-    print_summary(summary)
+    res_serializer.print_summary()
 
     if result_path:
         result_path = os.path.expanduser(result_path)
-- 
1.8.1.4



More information about the LNST-developers mailing list