Change in vdsm[master]: network: tests: add error code checking

phoracek at redhat.com phoracek at redhat.com
Wed Mar 18 10:12:46 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/60/38860/1

diff --git a/tests/functional/networkTests.py b/tests/functional/networkTests.py
index db22006..8671244 100644
--- a/tests/functional/networkTests.py
+++ b/tests/functional/networkTests.py
@@ -2369,8 +2369,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/38860
To unsubscribe, visit https://gerrit.ovirt.org/settings

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


More information about the vdsm-patches mailing list