aboutsummaryrefslogtreecommitdiffstats
path: root/resources/libraries/python/PapiHistory.py
diff options
context:
space:
mode:
authorTibor Frank <tifrank@cisco.com>2019-03-07 09:27:46 +0100
committerTibor Frank <tifrank@cisco.com>2019-03-07 09:30:32 +0100
commit86af0b9a49b5513e328359241a11a1ca44075bc5 (patch)
tree6fd0b93444772ea75ae1c879810780aee88dd21f /resources/libraries/python/PapiHistory.py
parent4096508ece27f9576d5aab3ed06acfb93925e2d4 (diff)
Papi: Rename methods argument
The argument name 'csit_papi_command' must be unique enough as it cannot be repeated in kwargs. - PapiExecutor.add(self, csit_papi_command, **kwargs) - PapiHistory.add_to_papi_history(node, csit_papi_command, papi=True, **kwargs) Change-Id: I243a20d6f03befb149891c37a88e0e6359ea2940 Signed-off-by: Tibor Frank <tifrank@cisco.com>
Diffstat (limited to 'resources/libraries/python/PapiHistory.py')
-rw-r--r--resources/libraries/python/PapiHistory.py14
1 files changed, 9 insertions, 5 deletions
diff --git a/resources/libraries/python/PapiHistory.py b/resources/libraries/python/PapiHistory.py
index 13c483b4bc..eede0d250f 100644
--- a/resources/libraries/python/PapiHistory.py
+++ b/resources/libraries/python/PapiHistory.py
@@ -48,16 +48,19 @@ class PapiHistory(object):
PapiHistory.reset_papi_history(node)
@staticmethod
- def add_to_papi_history(node, cmd, papi=True, **kwargs):
+ def add_to_papi_history(node, csit_papi_command, papi=True, **kwargs):
"""Add command to PAPI command history on DUT node.
+ The argument name 'csit_papi_command' must be unique enough as it cannot
+ be repeated in kwargs.
+
:param node: DUT node to add command to PAPI command history for.
- :param cmd: Command to be added to PAPI command history.
+ :param csit_papi_command: Command to be added to PAPI command history.
:param papi: Says if the command to store is PAPi or VAT. Remove when
VAT executor is completely removed.
:param kwargs: Optional key-value arguments.
:type node: dict
- :type cmd: str
+ :type csit_papi_command: str
:type papi: bool
:type kwargs: dict
"""
@@ -65,12 +68,13 @@ class PapiHistory(object):
args = list()
for key, val in kwargs.iteritems():
args.append("{key}={val}".format(key=key, val=val))
- item = "{cmd}({args})".format(cmd=cmd, args=",".join(args))
+ item = "{cmd}({args})".format(cmd=csit_papi_command,
+ args=",".join(args))
else:
# This else part is here to store VAT commands.
# VAT history is not used.
# TODO: Remove when VatExecutor is completely removed.
- item = "{cmd}".format(cmd=cmd)
+ item = "{cmd}".format(cmd=csit_papi_command)
DICT__DUTS_PAPI_HISTORY[node['host']].append(item)
@staticmethod