Bug fix: vping_userdata test status was not properly reported 11/43911/2
authorMorgan Richomme <morgan.richomme@orange.com>
Mon, 2 Oct 2017 15:11:13 +0000 (17:11 +0200)
committerMorgan Richomme <morgan.richomme@orange.com>
Mon, 2 Oct 2017 15:30:26 +0000 (17:30 +0200)
vping_base is calling vping_userdata
vping_user data could return only EX_TESTCASE_FAILED or EX_OK
by default vping_base always returns EX_OK (based on self.result=100..)
it returned EX_RUN_ERROR only if upper class report EX_RUN_ERROR

that is why vping_userdata was always PASS even if the trace "Ping OK"
cannot be retrieved from the Nova console

it seems that when running the vping_userdata, we systematically got
 Failed to read iid from metadata

Change-Id: If72441dd93bffada5c7a5399c87db68d7ebeebed
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>
functest/opnfv_tests/openstack/vping/vping_base.py

index c93d2f2..8b622e1 100644 (file)
@@ -165,7 +165,7 @@ class VPingBase(testcase.TestCase):
         else:
             raise Exception('VMs never became active')
 
-        if result == testcase.TestCase.EX_RUN_ERROR:
+        if result != testcase.TestCase.EX_OK:
             return testcase.TestCase.EX_RUN_ERROR
 
         self.stop_time = time.time()