Change in vdsm[ovirt-3.5]: network: tests: add error code checking

phoracek at redhat.com phoracek at redhat.com
Wed Mar 18 11:25:04 UTC 2015


Petr Horáček has uploaded a new change for review.

Change subject: network: tests: add error code checking
......................................................................

network: tests: add error code checking

Change-Id: I017a6307eab44d57b456535f7bcd7779f91ed570
Signed-off-by: Petr Horáček <phoracek at redhat.com>
---
M tests/functional/networkTests.py
1 file changed, 4 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/65/38865/1

diff --git a/tests/functional/networkTests.py b/tests/functional/networkTests.py
index 8af47e4..075fac4 100644
--- a/tests/functional/networkTests.py
+++ b/tests/functional/networkTests.py
@@ -2213,8 +2213,10 @@
 NM_CONTROLLED=no
 HOTPLUG=no""" % (BONDING_NAME, VLAN_ID))
 
-            rc, _, _ = execCmd([EXT_IFUP, BONDING_NAME])
-            rc, _, _ = execCmd([EXT_IFUP, BONDING_NAME + '.' + VLAN_ID])
+            rc, _, err = execCmd([EXT_IFUP, BONDING_NAME])
+            self.assertEqual(rc, SUCCESS, err)
+            rc, _, err = execCmd([EXT_IFUP, BONDING_NAME + '.' + VLAN_ID])
+            self.assertEqual(rc, SUCCESS, err)
 
             status, msg = self.vdsm_net.setupNetworks(
                 {NETWORK_NAME: {'bonding': BONDING_NAME, 'bridged': True,


-- 
To view, visit https://gerrit.ovirt.org/38865
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I017a6307eab44d57b456535f7bcd7779f91ed570
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>


More information about the vdsm-patches mailing list