commit 8be08ccd8da32b7bbe01cac7e85d50d63ca16b5e
author Masayuki Igawa <igawa@mxs.nes.nec.co.jp> Fri Feb 07 10:39:22 2014 +0900
committer Masayuki Igawa <masayuki.igawa@gmail.com> Fri Feb 21 22:13:12 2014 +0900
tree ccb01aec27aed196cbd907d604008ba2ab519e09
parent 0155496ecd1a4ec89f683a58b35552c2be8f6fd2

Clarify admin_delete_server_of_others test case

create_test_server() doesn't wait for server 'ACTIVE' by default now.
But this test case verifies 'Can a admin delete servers of others?'. So
we should wait until 'ACTIVE' here.

Change-Id: Ibb9e65166035c686c0b222033cb08f32a63f8b23

tempest/api/compute/admin/test_servers.py

diff --git a/tempest/api/compute/admin/test_servers.py b/tempest/api/compute/admin/test_servers.py
index 2cee78a..39d69bf 100644
--- a/tempest/api/compute/admin/test_servers.py
+++ b/tempest/api/compute/admin/test_servers.py
@@ -93,7 +93,7 @@
     @attr(type='gate')
     def test_admin_delete_servers_of_others(self):
         # Administrator can delete servers of others
-        _, server = self.create_test_server()
+        _, server = self.create_test_server(wait_until='ACTIVE')
         resp, _ = self.client.delete_server(server['id'])
         self.assertEqual('204', resp['status'])
         self.servers_client.wait_for_server_termination(server['id'])

tempest/api/compute/v3/admin/test_servers.py

diff --git a/tempest/api/compute/v3/admin/test_servers.py b/tempest/api/compute/v3/admin/test_servers.py
index ef9eedc..b5076c3 100644
--- a/tempest/api/compute/v3/admin/test_servers.py
+++ b/tempest/api/compute/v3/admin/test_servers.py
@@ -79,7 +79,7 @@
     @attr(type='gate')
     def test_admin_delete_servers_of_others(self):
         # Administrator can delete servers of others
-        _, server = self.create_test_server()
+        _, server = self.create_test_server(wait_until='ACTIVE')
         resp, _ = self.client.delete_server(server['id'])
         self.assertEqual('204', resp['status'])
         self.servers_client.wait_for_server_termination(server['id'])