X-Git-Url: https://gerrit.fd.io/r/gitweb?p=csit.git;a=blobdiff_plain;f=resources%2Flibraries%2Fpython%2FPapiExecutor.py;h=a51cb76654a256826e4b899d3171a78d851b559d;hp=03132844059aa1dc6e6ee2d6efa0a960f56d20fe;hb=86af0b9a49b5513e328359241a11a1ca44075bc5;hpb=935734b04269b8fe5348e1c2b168dd5c6cd9339a

diff --git a/resources/libraries/python/PapiExecutor.py b/resources/libraries/python/PapiExecutor.py
index 0313284405..a51cb76654 100644
--- a/resources/libraries/python/PapiExecutor.py
+++ b/resources/libraries/python/PapiExecutor.py
@@ -20,12 +20,10 @@ from robot.api import logger
 
 from resources.libraries.python.Constants import Constants
 from resources.libraries.python.ssh import SSH, SSHTimeout
+from resources.libraries.python.PapiHistory import PapiHistory
 
 __all__ = ["PapiExecutor", "PapiResponse"]
 
-# TODO: Implement Papi History
-# from resources.libraries.python.PapiHistory import PapiHistory
-
 
 class PapiResponse(object):
     """Class for metadata specifying the Papi reply, stdout, stderr and return
@@ -138,17 +136,22 @@ class PapiExecutor(object):
         self._api_command_list = list()
         return self
 
-    def add(self, command, **kwargs):
+    def add(self, csit_papi_command, **kwargs):
         """Add next command to internal command list; return self.
 
-        :param command: VPP API command.
+        The argument name 'csit_papi_command' must be unique enough as it cannot
+        be repeated in kwargs.
+
+        :param csit_papi_command: VPP API command.
         :param kwargs: Optional key-value arguments.
-        :type command: str
+        :type csit_papi_command: str
         :type kwargs: dict
         :returns: self, so that method chaining is possible.
         :rtype: PapiExecutor
         """
-        self._api_command_list.append(dict(api_name=command, api_args=kwargs))
+        PapiHistory.add_to_papi_history(self._node, csit_papi_command, **kwargs)
+        self._api_command_list.append(dict(api_name=csit_papi_command,
+                                           api_args=kwargs))
         return self
 
     def execute(self, process_reply=True, ignore_errors=False, timeout=120):