From Dan Kenigsberg <dan...@redhat.com>:
Dan Kenigsberg has uploaded a new change for review.
Change subject: net tests: add a slightly more meaningful message on exception
......................................................................
net tests: add a slightly more meaningful message on exception
Seeing a list of offending rules or routes is more legible than a
generic
AssertionError: 2 != 3
Change-Id: I9ede150d5c35be4eddd14bf68e423c0477f747ca
Signed-off-by: Dan Kenigsberg <dan...@redhat.com>
---
M tests/network/sourceroute_test.py
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/31/77731/1
diff --git a/tests/network/sourceroute_test.py
b/tests/network/sourceroute_test.py
index 4f8a387..74ebe5f 100644
--- a/tests/network/sourceroute_test.py
+++ b/tests/network/sourceroute_test.py
@@ -76,8 +76,8 @@
dsroute = DynamicSourceRoute(nic, None, None, None)
routes, rules = dsroute.current_srconfig()
- self.assertEqual(2, len(routes))
- self.assertEqual(2, len(rules))
+ self.assertEqual(2, len(routes), routes)
+ self.assertEqual(2, len(rules), rules)
self.assertEqual('0.0.0.0/0', routes[0].to)
self.assertEqual(nic, routes[0].device)
--
To view, visit https://gerrit.ovirt.org/77731
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9ede150d5c35be4eddd14bf68e423c0477f747ca
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <dan...@redhat.com>
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org
