summaryrefslogtreecommitdiff
path: root/openstackclient/tests/compute
diff options
context:
space:
mode:
authorTang Chen <chen.tang@easystack.cn>2016-02-14 18:43:19 +0800
committerTang Chen <chen.tang@easystack.cn>2016-02-14 18:43:19 +0800
commitcfcb750a97af1ab82b425532437456c22dcebad9 (patch)
tree5064b1fbe9b6853ff4e353d3eef56dfaeb16ccc3 /openstackclient/tests/compute
parentd8abec33ada8b2b028d52eb8bfad2640812b9af8 (diff)
downloadpython-openstackclient-cfcb750a97af1ab82b425532437456c22dcebad9.tar.gz
Use assertRaises() to check if an exception is raised
In some test cases, try/except is used to check if an exception has been raised. We should use assertRaises() instead. Change-Id: I15c8e757dcab77fd6f895feb018184e1eb7e617b
Diffstat (limited to 'openstackclient/tests/compute')
-rw-r--r--openstackclient/tests/compute/v2/test_server.py9
1 files changed, 4 insertions, 5 deletions
diff --git a/openstackclient/tests/compute/v2/test_server.py b/openstackclient/tests/compute/v2/test_server.py
index a80eaf51..95188522 100644
--- a/openstackclient/tests/compute/v2/test_server.py
+++ b/openstackclient/tests/compute/v2/test_server.py
@@ -143,11 +143,10 @@ class TestServerCreate(TestServer):
verifylist = [
('server_name', self.new_server.name),
]
- try:
- # Missing required args should bail here
- self.check_parser(self.cmd, arglist, verifylist)
- except utils.ParserException:
- pass
+
+ # Missing required args should bail here
+ self.assertRaises(utils.ParserException, self.check_parser,
+ self.cmd, arglist, verifylist)
def test_server_create_minimal(self):
arglist = [