----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22712/#review52760 -----------------------------------------------------------
Please change method comments to use one docstrings style instead of mixing them with single-line comments. - Min Chen On Sept. 8, 2014, 7:52 p.m., sangeetha hariharan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22712/ > ----------------------------------------------------------- > > (Updated Sept. 8, 2014, 7:52 p.m.) > > > Review request for cloudstack, Min Chen, Prachi Damle, and Santhosh Edukulla. > > > Repository: cloudstack-git > > > Description > ------- > > This Test suite has test cases relating to acess checks for deleteNetwork() > for Admin, domain admin and regular users > > > Diffs > ----- > > test/integration/component/test_acl_isolatednetwork_delete.py PRE-CREATION > > Diff: https://reviews.apache.org/r/22712/diff/ > > > Testing > ------- > > Test Suite was executed against a management server built from 4.4-forward > branch using a simulator set up: > test_deleteNetwork_admin > (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete) > ... === TestName: test_deleteNetwork_admin | Status : SUCCESS === > ok > test_deleteNetwork_admin_foruserinotherdomain > (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete) > ... === TestName: test_deleteNetwork_admin_foruserinotherdomain | Status : > SUCCESS === > ok > test_deleteNetwork_admin_foruserinsamedomain > (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete) > ... === TestName: test_deleteNetwork_admin_foruserinsamedomain | Status : > SUCCESS === > ok > test_deleteNetwork_domaindmin > (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete) > ... === TestName: test_deleteNetwork_domaindmin | Status : SUCCESS === > ok > test_deleteNetwork_domaindmin_forcrossdomainuser > (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete) > ... === TestName: test_deleteNetwork_domaindmin_forcrossdomainuser | Status > : SUCCESS === > ok > test_deleteNetwork_domaindmin_foruserinsamedomain > (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete) > ... === TestName: test_deleteNetwork_domaindmin_foruserinsamedomain | Status > : SUCCESS === > ok > test_deleteNetwork_domaindmin_foruserinsubdomain > (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete) > ... === TestName: test_deleteNetwork_domaindmin_foruserinsubdomain | Status > : SUCCESS === > ok > test_deleteNetwork_user > (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete) > ... === TestName: test_deleteNetwork_user | Status : SUCCESS === > ok > test_deleteNetwork_user_foruserinotherdomain > (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete) > ... === TestName: test_deleteNetwork_user_foruserinotherdomain | Status : > SUCCESS === > ok > test_deleteNetwork_user_foruserinsamedomain > (integration.component.test_acl_isolatednetwork_delete.TestIsolatedNetworkDelete) > ... === TestName: test_deleteNetwork_user_foruserinsamedomain | Status : > SUCCESS === > ok > > ---------------------------------------------------------------------- > Ran 10 tests in 61.766s > > OK > > > Thanks, > > sangeetha hariharan > >
