From: Vratko Polak <vrpolak@cisco.com>
Date: Wed, 24 Jul 2019 07:49:05 +0000 (+0200)
Subject: PapiHistory: Use repr strings for argument values
X-Git-Url: https://gerrit.fd.io/r/gitweb?p=csit.git;a=commitdiff_plain;h=fb3aba8bd6418418d9b3741839c4b305d8c72a4e

PapiHistory: Use repr strings for argument values

Composite arguments frequently do not have valid __str__ method
so the values were not visible.

Change-Id: If302934cb227f212580bb2f5c277a755ace51bcf
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
---

diff --git a/resources/libraries/python/PapiHistory.py b/resources/libraries/python/PapiHistory.py
index 6a4f913d03..c4ee4ee495 100644
--- a/resources/libraries/python/PapiHistory.py
+++ b/resources/libraries/python/PapiHistory.py
@@ -51,6 +51,8 @@ class PapiHistory(object):
     def add_to_papi_history(node, csit_papi_command, papi=True, **kwargs):
         """Add command to PAPI command history on DUT node.
 
+        Repr strings are used for argument values.
+
         The argument name 'csit_papi_command' must be unique enough as it cannot
         be repeated in kwargs.
 
@@ -84,7 +86,7 @@ class PapiHistory(object):
         if papi:
             args = list()
             for key, val in kwargs.iteritems():
-                args.append("{key}={val}".format(key=key, val=val))
+                args.append("{key}={val!r}".format(key=key, val=val))
             item = "{cmd}({args})".format(cmd=csit_papi_command,
                                           args=",".join(args))
         else: