> On Feb. 4, 2015, 10:48 a.m., suresh sadhu wrote: > > you didn't remove the below code... as its not required in test03 -- here > > vm deployed in stopped state.. > > > > if str(self.zone.networktype).lower() == "advanced": > > response = VerifyRouterState( > > self.apiclient, > > self.account.name, > > self.account.domainid, > > STOPPED > > )
I have added new patch removing the code snippet and also rebased it against latest master branch. - Ashutosh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25885/#review70943 ----------------------------------------------------------- On Feb. 16, 2015, 5:16 a.m., Ashutosh Kelkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25885/ > ----------------------------------------------------------- > > (Updated Feb. 16, 2015, 5:16 a.m.) > > > Review request for cloudstack, suresh sadhu and SrikanteswaraRao Talluri. > > > Bugs: CLOUDSTACK-7594 > https://issues.apache.org/jira/browse/CLOUDSTACK-7594 > > > Repository: cloudstack-git > > > Description > ------- > > Automation test cases for Stopped VM test path. > > > Diffs > ----- > > test/integration/testpaths/testpath_stopped_vm.py PRE-CREATION > > Diff: https://reviews.apache.org/r/25885/diff/ > > > Testing > ------- > > Yes. > > Advanced zone: > Positive test for stopped VM test path - T1 ... === TestName: > test_01_pt_deploy_vm_without_startvm | Status : SUCCESS === > ok > Positive test for stopped VM test path - T1 variant ... === TestName: > test_02_pt_deploy_vm_with_startvm_true | Status : SUCCESS === > ok > Positive test for stopped VM test path - T2 ... === TestName: > test_03_pt_deploy_vm_with_startvm_false | Status : SUCCESS === > ok > Positive test for stopped VM test path - T3 and variant, T9 ... === TestName: > test_04_pt_startvm_false_attach_disk | Status : SUCCESS === > ok > Positive test for stopped VM test path - T4 ... === TestName: > test_05_pt_startvm_false_attach_disk_change_SO | Status : SUCCESS === > ok > Positive test for stopped VM test path - T5 ... === TestName: > test_06_pt_startvm_false_attach_iso | Status : SUCCESS === > ok > Positive test for stopped VM test path - T5 variant ... === TestName: > test_07_pt_startvm_false_attach_iso_running_vm | Status : SUCCESS === > ok > Positive test for stopped VM test path - T10 ... === TestName: > test_08_pt_startvm_false_password_enabled_template | Status : SUCCESS === > ok > Positive test for stopped VM test path - T11 ... === TestName: > test_09_pt_destroy_stopped_vm | Status : SUCCESS === > ok > Positive test for stopped VM test path - T12 ... === TestName: > test_10_max_account_limit | Status : SUCCESS === > ok > > > Basic zone: > > Positive test for stopped VM test path - T1 ... === TestName: > test_01_pt_deploy_vm_without_startvm | Status : SUCCESS === > ok > Positive test for stopped VM test path - T1 variant ... === TestName: > test_02_pt_deploy_vm_with_startvm_true | Status : SUCCESS === > ok > Positive test for stopped VM test path - T2 ... === TestName: > test_03_pt_deploy_vm_with_startvm_false | Status : SUCCESS === > ok > Positive test for stopped VM test path - T3 and variant, T9 ... === TestName: > test_04_pt_startvm_false_attach_disk | Status : SUCCESS === > ok > Positive test for stopped VM test path - T4 ... === TestName: > test_05_pt_startvm_false_attach_disk_change_SO | Status : SUCCESS === > ok > Positive test for stopped VM test path - T5 ... === TestName: > test_06_pt_startvm_false_attach_iso | Status : SUCCESS === > ok > Positive test for stopped VM test path - T5 variant ... === TestName: > test_07_pt_startvm_false_attach_iso_running_vm | Status : SUCCESS === > ok > Positive test for stopped VM test path - T10 ... === TestName: > test_08_pt_startvm_false_password_enabled_template | Status : SUCCESS === > ok > Positive test for stopped VM test path - T11 ... === TestName: > test_09_pt_destroy_stopped_vm | Status : SUCCESS === > ok > Positive test for stopped VM test path - T12 ... === TestName: > test_10_max_account_limit | Status : SUCCESS === > ok > > ---------------------------------------------------------------------- > Ran 10 tests in 2792.968s > > OK > > > Thanks, > > Ashutosh Kelkar > >
