From fb3aba8bd6418418d9b3741839c4b305d8c72a4e Mon Sep 17 00:00:00 2001 From: Vratko Polak Date: Wed, 24 Jul 2019 09:49:05 +0200 Subject: 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 --- resources/libraries/python/PapiHistory.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) 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: -- cgit 1.2.3-korg