X-Git-Url: https://gerrit.fd.io/r/gitweb?p=csit.git;a=blobdiff_plain;f=resources%2Flibraries%2Fpython%2FPapiExecutor.py;h=ecee70c9c5e14ee284c9e65652145bb31a73273b;hp=76d7daf3ed2bde2edbb64f814a9b7b0d58d70a84;hb=b6606e7625e308a66bdfb9d5a9c065b58e429a99;hpb=96c5f5ef45cf039691404a4451b1c6d9260d6ea0 diff --git a/resources/libraries/python/PapiExecutor.py b/resources/libraries/python/PapiExecutor.py index 76d7daf3ed..ecee70c9c5 100644 --- a/resources/libraries/python/PapiExecutor.py +++ b/resources/libraries/python/PapiExecutor.py @@ -794,16 +794,17 @@ class PapiSocketExecutor: if not isinstance(reply, list): reply = [reply] for item in reply: - self.crc_checker.check_api_name(item.__class__.__name__) + message_name = item.__class__.__name__ + self.crc_checker.check_api_name(message_name) dict_item = dictize(item) if u"retval" in dict_item.keys(): # *_details messages do not contain retval. retval = dict_item[u"retval"] if retval != exp_rv: - # TODO: What exactly to log and raise here? raise AssertionError( f"Retval {retval!r} does not match expected " - f"retval {exp_rv!r}" + f"retval {exp_rv!r} in message {message_name} " + f"for command {command}." ) replies.append(dict_item) return replies @@ -872,6 +873,8 @@ class PapiExecutor: is "stats". - the second parameter must be 'path' as it is used by PapiExecutor method 'add'. + - even if the parameter contains multiple paths, there is only one + reply item (for each .add). """ def __init__(self, node):