[lnst] lnst-ctl: use the new print_summary method

Jiří Pírko jirka at fedoraproject.org
Fri May 31 11:33:44 UTC 2013


commit 33d1bfb2e360738bbf5e90198f4d871138897e02
Author: Ondrej Lichtner <olichtne at redhat.com>
Date:   Fri May 31 11:06:40 2013 +0200

    lnst-ctl: use the new print_summary method
    
    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>
    Signed-off-by: Jiri Pirko <jiri at resnulli.us>

 lnst-ctl |   12 +-----------
 1 files changed, 1 insertions(+), 11 deletions(-)
---
diff --git a/lnst-ctl b/lnst-ctl
index 5701518..05f17c1 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)


More information about the LNST-developers mailing list