aboutsummaryrefslogtreecommitdiffstats
path: root/resources/libraries/python/PapiExecutor.py
diff options
context:
space:
mode:
authorVratko Polak <vrpolak@cisco.com>2021-11-23 17:03:16 +0100
committerVratko Polak <vrpolak@cisco.com>2021-11-25 09:14:49 +0000
commit7f8933997cc1a6c9a20657f3d84fce8935363cc9 (patch)
treea6ae1091cf1f6aca5ac2625fbc4ef8a6a64911ca /resources/libraries/python/PapiExecutor.py
parent8822e6ed5801da972e66055d23f05892eddafc34 (diff)
PAPI: Include more details in PAPI fail message
This helps to distinguish different symptoms for PAL failure alerting. Change-Id: Idd8e8f4b427641fa055516bdf17715f6b967ca72 Signed-off-by: Vratko Polak <vrpolak@cisco.com>
Diffstat (limited to 'resources/libraries/python/PapiExecutor.py')
-rw-r--r--resources/libraries/python/PapiExecutor.py7
1 files changed, 4 insertions, 3 deletions
diff --git a/resources/libraries/python/PapiExecutor.py b/resources/libraries/python/PapiExecutor.py
index 7bfc8614a5..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