Updated Branches:
  refs/heads/master 2d6644d96 -> 22cb5be8b

CLOUDSTACK-2273: cleanup using user api client for domain admins

Signed-off-by: Prasanna Santhanam <t...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/9f9510be
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/9f9510be
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/9f9510be

Branch: refs/heads/master
Commit: 9f9510be9189346ed72ff92705d284de91d53c6f
Parents: cfd892f
Author: ashutoshkelkar <ashut...@clogeny.com>
Authored: Wed Jul 17 15:26:34 2013 +0530
Committer: Prasanna Santhanam <t...@apache.org>
Committed: Wed Jul 17 19:23:57 2013 +0530

----------------------------------------------------------------------
 test/integration/component/test_assign_vm.py | 29 +++++++++++++++++++----
 1 file changed, 24 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/9f9510be/test/integration/component/test_assign_vm.py
----------------------------------------------------------------------
diff --git a/test/integration/component/test_assign_vm.py 
b/test/integration/component/test_assign_vm.py
index 8bc98fe..55c174b 100644
--- a/test/integration/component/test_assign_vm.py
+++ b/test/integration/component/test_assign_vm.py
@@ -353,8 +353,11 @@ class TestVMOwnership(cloudstackTestCase):
         # 1. deploy VM in sub subdomain1
         # 2. stop VM in sub subdomain1
         # 3. assignVirtualMachine to subdomain2
+        userapiclient = 
self.testClient.getUserApiClient(account=self.sdomain_account_user1['account'].name,
+                                                         
domain=self.sdomain_account_user1['domain'].name,
+                                                         type=2)
         self.create_vm(self.sdomain_account_user1['account'], 
self.sdomain_account_user1['domain'])
-        self.assertRaises(Exception, 
self.virtual_machine.assign_virtual_machine, self.apiclient, 
self.sdomain_account_user2['account'].name 
,self.sdomain_account_user2['domain'].id)
+        self.assertRaises(Exception, 
self.virtual_machine.assign_virtual_machine, userapiclient, 
self.sdomain_account_user2['account'].name 
,self.sdomain_account_user2['domain'].id)
 
     @attr(tags = ["advanced"])
     @log_test_exceptions
@@ -386,8 +389,11 @@ class TestVMOwnership(cloudstackTestCase):
         # Validate the following:
         # 1. deploy VM in sub subdomain1 with volumes.
         # 3. assignVirtualMachine to subdomain2
+        userapiclient = 
self.testClient.getUserApiClient(account=self.sdomain_account_user1['account'].name,
+                                                         
domain=self.sdomain_account_user1['domain'].name,
+                                                         type=2)
         self.create_vm(self.sdomain_account_user1['account'], 
self.sdomain_account_user1['domain'],volume=self.sdomain_account_user1['volume'])
-        self.assertRaises(Exception, 
self.virtual_machine.assign_virtual_machine, self.apiclient, 
self.sdomain_account_user2['account'].name 
,self.sdomain_account_user2['domain'].id)
+        self.assertRaises(Exception, 
self.virtual_machine.assign_virtual_machine, userapiclient, 
self.sdomain_account_user2['account'].name 
,self.sdomain_account_user2['domain'].id)
          # Check all volumes attached to same VM
         list_volume_response = list_volumes(self.apiclient,
                                             
virtualmachineid=self.virtual_machine.id,
@@ -398,6 +404,8 @@ class TestVMOwnership(cloudstackTestCase):
                          "Check list volumes response for valid list")
 
         self.assertNotEqual(list_volume_response[0].domainid, 
self.sdomain_account_user2['domain'].id, "Volume ownership not changed.")
+        self.virtual_machine.detach_volume(self.apiclient,
+                                           
self.sdomain_account_user1['volume'])
 
     @attr(tags = ["advanced"])
     @log_test_exceptions
@@ -423,8 +431,11 @@ class TestVMOwnership(cloudstackTestCase):
         # Validate the following:
         # 1. deploy VM in sub subdomain1 with snapshot.
         # 3. assignVirtualMachine to subdomain2
+        userapiclient = 
self.testClient.getUserApiClient(account=self.sdomain_account_user1['account'].name,
+                                                         
domain=self.sdomain_account_user1['domain'].name,
+                                                         type=2)
         self.create_vm(self.sdomain_account_user1['account'], 
self.sdomain_account_user1['domain'], 
project=self.sdomain_account_user1['project'])
-        self.assertRaises(Exception, 
self.virtual_machine.assign_virtual_machine, self.apiclient, 
self.sdomain_account_user2['account'].name 
,self.sdomain_account_user2['domain'].id)
+        self.assertRaises(Exception, 
self.virtual_machine.assign_virtual_machine, userapiclient, 
self.sdomain_account_user2['account'].name 
,self.sdomain_account_user2['domain'].id)
 
     @attr(tags = ["advanced"])
     @log_test_exceptions
@@ -439,8 +450,11 @@ class TestVMOwnership(cloudstackTestCase):
                               
account=self.sdomain_account_user2['account'].name,
                               domainid=self.sdomain_account_user2['domain'].id,
                               max=0)
+        userapiclient = 
self.testClient.getUserApiClient(account=self.sdomain_account_user1['account'].name,
+                                                         
domain=self.sdomain_account_user1['domain'].name,
+                                                         type=2)
         self.create_vm(self.sdomain_account_user1['account'], 
self.sdomain_account_user1['domain'], snapshot=True)
-        self.assertRaises(Exception, 
self.virtual_machine.assign_virtual_machine, self.apiclient, 
self.sdomain_account_user2['account'].name 
,self.sdomain_account_user2['domain'].id)
+        self.assertRaises(Exception, 
self.virtual_machine.assign_virtual_machine, userapiclient, 
self.sdomain_account_user2['account'].name 
,self.sdomain_account_user2['domain'].id)
 
     @attr(tags = ["advanced"])
     @log_test_exceptions
@@ -456,5 +470,10 @@ class TestVMOwnership(cloudstackTestCase):
                               
account=self.sdomain_account_user2['account'].name,
                               domainid=self.sdomain_account_user2['domain'].id,
                               max=0)
+        userapiclient = 
self.testClient.getUserApiClient(account=self.sdomain_account_user1['account'].name,
+                                                         
domain=self.sdomain_account_user1['domain'].name,
+                                                         type=2)
         self.create_vm(self.sdomain_account_user1['account'], 
self.sdomain_account_user1['domain'], snapshot=True, 
volume=self.sdomain_account_user1['volume'])
-        self.assertRaises(Exception, 
self.virtual_machine.assign_virtual_machine, self.apiclient, 
self.sdomain_account_user2['account'].name 
,self.sdomain_account_user2['domain'].id)
+        self.assertRaises(Exception, 
self.virtual_machine.assign_virtual_machine, userapiclient, 
self.sdomain_account_user2['account'].name 
,self.sdomain_account_user2['domain'].id)
+        self.virtual_machine.detach_volume(self.apiclient,
+                                            
self.sdomain_account_user1['volume'])

Reply via email to