diff options
author | Jan Gelety <jgelety@cisco.com> | 2017-05-29 12:07:58 +0200 |
---|---|---|
committer | Jan Gelety <jgelety@cisco.com> | 2017-05-30 10:01:08 +0000 |
commit | 43e7440a3fea2125ecb8d6d1f4ee7f69bbf9f3ac (patch) | |
tree | 0e2156f882bd6fd03f8a40a9253fd65a02e2256f /resources/libraries/robot | |
parent | 39311c05e6a056a3f33ec2cf050a00665c8f4540 (diff) |
Do not fail test case when no vpp pid detected in the tear down
- currently when no pid (or more pids) detected in the test case
tear down of successfully run test case not only the WARN message
is logged but the whole test case is failed
- to change this behaviour but keep test case failure if no vpp pid
(or more vpp pids) detected in test case setup the low level KW
"Get VPP PID" behaviour has been changed to return one or more
pids or None w/o raising an error and check has been moved to
higher level KW "Save VPP PIDs"
- there was added suite name and test case name to WARN message when
no pid or more pids detected in the test case tear down
Change-Id: I04f8d81c1ca48a1e4a45bd0e58f00d36d51c6933
Signed-off-by: Jan Gelety <jgelety@cisco.com>
Diffstat (limited to 'resources/libraries/robot')
-rw-r--r-- | resources/libraries/robot/default.robot | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/resources/libraries/robot/default.robot b/resources/libraries/robot/default.robot index 05314d6a15..fd1b280b94 100644 --- a/resources/libraries/robot/default.robot +++ b/resources/libraries/robot/default.robot @@ -350,6 +350,12 @@ | | ... | where the key is the host and the value is the PID. | | ... | | ${setup_vpp_pids}= | Get VPP PIDs | ${nodes} +| | ${keys}= | Get Dictionary Keys | ${setup_vpp_pids} +| | :FOR | ${key} | IN | @{keys} +| | | ${pid}= | Get From Dictionary | ${setup_vpp_pids} | ${key} +| | | Run Keyword If | $pid is None | FAIL | No VPP PID found on node ${key} +| | | Run Keyword If | ',' in '${pid}' +| | | ... | FAIL | More then one VPP PID found on node ${key}: ${pid} | | Set Test Variable | ${setup_vpp_pids} | Check VPP PID in Teardown @@ -358,7 +364,8 @@ | | ... | is printed on console and to log. The test will not fail. | | ... | | ${teardown_vpp_pids}= | Get VPP PIDs | ${nodes} -| | ${err_msg}= | Catenate | \nThe VPP PIDs are not equal!\nTest Setup VPP PIDs: +| | ${err_msg}= | Catenate | ${SUITE NAME} - ${TEST NAME} +| | ... | \nThe VPP PIDs are not equal!\nTest Setup VPP PIDs: | | ... | ${setup_vpp_pids}\nTest Teardown VPP PIDs: ${teardown_vpp_pids} | | ${rc} | ${msg}= | Run keyword and ignore error | | ... | Dictionaries Should Be Equal |