Simplify show_papi_history 57/18057/3
authorVratko Polak <vrpolak@cisco.com>
Thu, 7 Mar 2019 12:00:22 +0000 (13:00 +0100)
committerVratko Polak <vrpolak@cisco.com>
Thu, 7 Mar 2019 12:00:22 +0000 (13:00 +0100)
Change-Id: Id86f8630ff37e958d90011f421aeb8a75b3e4783
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
resources/libraries/python/PapiHistory.py

index eede0d2..ec5b167 100644 (file)
@@ -84,12 +84,12 @@ class PapiHistory(object):
         :param node: DUT node to show PAPI command history for.
         :type node: dict
         """
-        history = "\nNo PAPI command executed"
-        if DICT__DUTS_PAPI_HISTORY[node['host']]:
-            history = "".join(["\n{}".format(
-                cmd) for cmd in DICT__DUTS_PAPI_HISTORY[node['host']]])
+        history_list = DICT__DUTS_PAPI_HISTORY[node['host']]
+        if not history_list:
+            history_list = ("No PAPI command executed", )
         logger.trace(
-            "{0} PAPI command history:\n{1}\n".format(node['host'], history))
+            "{0} PAPI command history:\n{1}\n".format(
+                node['host'], "\n".join(history_list)))
 
     @staticmethod
     def show_papi_history_on_all_duts(nodes):