Fixed the incorrect assertion in noncontiguous_vlan test The assertion fails if the VLAN is found in which case find returns a positive number. So here the assertion should infact assert < 0 result.
Signed-off-by: Prasanna Santhanam <[email protected]> Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/4c0eecc1 Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/4c0eecc1 Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/4c0eecc1 Branch: refs/heads/ui-cisco-asa1000v-support Commit: 4c0eecc195ba4a8bbed90fd1a0478eb746e7d538 Parents: c388f7f Author: Prasanna Santhanam <[email protected]> Authored: Mon May 6 20:39:36 2013 +0530 Committer: Prasanna Santhanam <[email protected]> Committed: Mon May 6 20:39:36 2013 +0530 ---------------------------------------------------------------------- test/integration/smoke/test_non_contigiousvlan.py | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/4c0eecc1/test/integration/smoke/test_non_contigiousvlan.py ---------------------------------------------------------------------- diff --git a/test/integration/smoke/test_non_contigiousvlan.py b/test/integration/smoke/test_non_contigiousvlan.py index 91b6325..4e130d9 100644 --- a/test/integration/smoke/test_non_contigiousvlan.py +++ b/test/integration/smoke/test_non_contigiousvlan.py @@ -81,6 +81,6 @@ class TestUpdatePhysicalNetwork(cloudstackTestCase): self.network = phy_networks[0] self.networkid = phy_networks[0].id updateResponse = self.network.update(self.apiClient, id = self.networkid, removevlan = self.vlan["full"]) - self.assert_(updateResponse.vlan.find(self.vlan["full"]) > 0, + self.assert_(updateResponse.vlan.find(self.vlan["full"]) < 0, "VLAN was not removed successfully")
