----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13014/#review24125 -----------------------------------------------------------
Commit 67c96d419d08f5a2702eee0aa64958e486f74703 in branch refs/heads/4.2 from Gaurav Aradhye [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=67c96d4 ] CLOUDSTACK-3841: Fixed test case issue. The exception raised earlier was valid behaviour, added missing assertRaiss statement Signed-off-by: Prasanna Santhanam <t...@apache.org> - ASF Subversion and Git Services On July 29, 2013, 12:39 p.m., Gaurav Aradhye wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13014/ > ----------------------------------------------------------- > > (Updated July 29, 2013, 12:39 p.m.) > > > Review request for cloudstack and Prasanna Santhanam. > > > Repository: cloudstack-git > > > Description > ------- > > The exception raised is valid behavior. It attempts to delete security group > without destroying vm in it, hence expecting exception in the operation. > Added self.assertRaises for the security group delete operation. Works fin > now. > > > Diffs > ----- > > test/integration/component/test_security_groups.py 3c25e25 > > Diff: https://reviews.apache.org/r/13014/diff/ > > > Testing > ------- > > Tested on KVM basic setup. > > > Thanks, > > Gaurav Aradhye > >