I just tried this with the current devstack and it worked. But if the intended behavior is to accept anything and return an empty list then perhaps this case should just be removed.
diff --git a/tempest/tests/compute/servers/test_list_servers_negative.py b/tempest/tests/compute/servers/test_list_servers_
index 1be7480..340b28c 100644
--- a/tempest/tests/compute/servers/test_list_servers_negative.py
+++ b/tempest/tests/compute/servers/test_list_servers_negative.py
@@ -142,8 +142,9 @@ class ListServersNegativeTest(BaseComputeTest):
def test_list_servers_pass_numeric_status(self):
"""Return an error when a numeric value for status is specified"""
- self.assertRaises(exceptions.BadRequest, self.client.list_servers,
- {'status': 1})
+ resp, body = self.client.list_servers({'status': 1})
+ self.assertEqual('200', resp['status'])
+ self.assertEqual(0, len(body['servers']))
def test_list_servers_by_limits(self):
"""List servers by specifying limits"""
I just tried this with the current devstack and it worked. But if the intended behavior is to accept anything and return an empty list then perhaps this case should just be removed.
diff --git a/tempest/ tests/compute/ servers/ test_list_ servers_ negative. py b/tempest/ tests/compute/ servers/ test_list_ servers_ tests/compute/ servers/ test_list_ servers_ negative. py tests/compute/ servers/ test_list_ servers_ negative. py tiveTest( BaseComputeTest ):
index 1be7480..340b28c 100644
--- a/tempest/
+++ b/tempest/
@@ -142,8 +142,9 @@ class ListServersNega
def test_list_ servers_ pass_numeric_ status( self): es(exceptions. BadRequest, self.client. list_servers, list_servers( {'status' : 1}) l('200' , resp['status']) 'servers' ]))
"""Return an error when a numeric value for status is specified"""
- self.assertRais
- {'status': 1})
+ resp, body = self.client.
+ self.assertEqua
+ self.assertEqual(0, len(body[
def test_list_ servers_ by_limits( self):
"""List servers by specifying limits"""