diff options
author | Tibor Frank <tifrank@cisco.com> | 2019-03-07 09:27:46 +0100 |
---|---|---|
committer | Tibor Frank <tifrank@cisco.com> | 2019-03-07 09:30:32 +0100 |
commit | 86af0b9a49b5513e328359241a11a1ca44075bc5 (patch) | |
tree | 6fd0b93444772ea75ae1c879810780aee88dd21f /resources/libraries/python/PapiHistory.py | |
parent | 4096508ece27f9576d5aab3ed06acfb93925e2d4 (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.py | 14 |
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 |