diff options
| author | Tang Chen <chen.tang@easystack.cn> | 2016-02-14 18:43:19 +0800 |
|---|---|---|
| committer | Tang Chen <chen.tang@easystack.cn> | 2016-02-14 18:43:19 +0800 |
| commit | cfcb750a97af1ab82b425532437456c22dcebad9 (patch) | |
| tree | 5064b1fbe9b6853ff4e353d3eef56dfaeb16ccc3 /openstackclient/tests/compute | |
| parent | d8abec33ada8b2b028d52eb8bfad2640812b9af8 (diff) | |
| download | python-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.py | 9 |
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 = [ |
