Dan Kenigsberg has uploaded a new change for review.
Change subject: py3: make ovs_test pass
......................................................................
py3: make ovs_test pass
Change-Id: Id812fb7c37573ed68dd4be9f982b8ae892532c9c
Signed-off-by: Dan Kenigsberg <danken(a)redhat.com>
---
M tests/Makefile.am
M tests/network/ovs_test.py
2 files changed, 4 insertions(+), 5 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/99/65599/1
diff --git a/tests/Makefile.am b/tests/Makefile.am
index be213c0..e96f226 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -195,7 +195,6 @@
network/netinfo_test.py \
network/ovs_driver_test.py \
network/ovs_info_test.py \
- network/ovs_test.py \
network/tc_test.py \
network/unified_persistence_test.py \
numaUtilsTests.py \
diff --git a/tests/network/ovs_test.py b/tests/network/ovs_test.py
index 000811d..96eaf85 100644
--- a/tests/network/ovs_test.py
+++ b/tests/network/ovs_test.py
@@ -68,7 +68,7 @@
ovs_validator.validate_net_configuration(
'net1', {'nic': 'eth0', 'switch':
'ovs'},
fake_to_be_added_bonds, fake_running_bonds, fake_kernel_nics)
- self.assertEquals(e.exception[0], ne.ERR_BAD_NIC)
+ self.assertEquals(e.exception.args[0], ne.ERR_BAD_NIC)
def test_add_network_with_non_existing_bond(self):
fake_running_bonds = {}
@@ -78,7 +78,7 @@
ovs_validator.validate_net_configuration(
'net1', {'bonding': 'bond1', 'switch':
'ovs'},
fake_to_be_added_bonds, fake_running_bonds, fake_kernel_nics)
- self.assertEquals(e.exception[0], ne.ERR_BAD_BONDING)
+ self.assertEquals(e.exception.args[0], ne.ERR_BAD_BONDING)
def test_add_network_with_to_be_added_bond(self):
fake_running_bonds = {}
@@ -169,7 +169,7 @@
ovs_validator.validate_bond_configuration(
'bond1', {'remove': True}, nets, running_nets,
fake_kernel_nics)
- self.assertEquals(e.exception[0], ne.ERR_USED_BOND)
+ self.assertEquals(e.exception.args[0], ne.ERR_USED_BOND)
def test_remove_bond_attached_to_network_that_will_use_nic(self):
fake_kernel_nics = ['eth0', 'eth1']
@@ -188,7 +188,7 @@
ovs_validator.validate_bond_configuration(
'bond1', {'remove': True}, nets, running_nets,
fake_kernel_nics)
- self.assertEquals(e.exception[0], ne.ERR_USED_BOND)
+ self.assertEquals(e.exception.args[0], ne.ERR_USED_BOND)
class MockedOvsInfo(ovs_info.OvsInfo):
--
To view, visit
https://gerrit.ovirt.org/65599
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id812fb7c37573ed68dd4be9f982b8ae892532c9c
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com>