Repository: libcloud Updated Branches: refs/heads/trunk f90b673a3 -> a2a36f030
Use assertRaises for asserting if subject threw an exception Closes #674 This change ensures that, should a regression happen, the test will print a more helpful failure message than "True is not False" Project: http://git-wip-us.apache.org/repos/asf/libcloud/repo Commit: http://git-wip-us.apache.org/repos/asf/libcloud/commit/a2a36f03 Tree: http://git-wip-us.apache.org/repos/asf/libcloud/tree/a2a36f03 Diff: http://git-wip-us.apache.org/repos/asf/libcloud/diff/a2a36f03 Branch: refs/heads/trunk Commit: a2a36f030247365db31efc73a776a0c324164ca9 Parents: f90b673 Author: Mark S. Maglana <[email protected]> Authored: Wed Jan 6 13:54:41 2016 -0800 Committer: anthony-shaw <[email protected]> Committed: Thu Jan 7 20:38:33 2016 +1100 ---------------------------------------------------------------------- libcloud/test/compute/test_dimensiondata.py | 73 +++++++++--------------- 1 file changed, 26 insertions(+), 47 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/libcloud/blob/a2a36f03/libcloud/test/compute/test_dimensiondata.py ---------------------------------------------------------------------- diff --git a/libcloud/test/compute/test_dimensiondata.py b/libcloud/test/compute/test_dimensiondata.py index 02eb663..d819334 100644 --- a/libcloud/test/compute/test_dimensiondata.py +++ b/libcloud/test/compute/test_dimensiondata.py @@ -43,19 +43,14 @@ class DimensionDataTests(unittest.TestCase, TestCaseMixin): self.driver = DimensionData(*DIMENSIONDATA_PARAMS) def test_invalid_region(self): - try: - self.driver = DimensionData(*DIMENSIONDATA_PARAMS, region='blah') - except ValueError: - pass + self.assertRaises(ValueError, + DimensionData, + *DIMENSIONDATA_PARAMS, + region='blah') def test_invalid_creds(self): DimensionDataMockHttp.type = 'UNAUTHORIZED' - try: - self.driver.list_nodes() - self.assertTrue( - False) # Above command should have thrown an InvalidCredsException - except InvalidCredsError: - pass + self.assertRaises(InvalidCredsError, self.driver.list_nodes) def test_list_locations_response(self): DimensionDataMockHttp.type = None @@ -88,12 +83,7 @@ class DimensionDataTests(unittest.TestCase, TestCaseMixin): DimensionDataMockHttp.type = 'INPROGRESS' node = Node(id='11', name=None, state=None, public_ips=None, private_ips=None, driver=self.driver) - try: - node.reboot() - self.assertTrue( - False) # above command should have thrown DimensionDataAPIException - except DimensionDataAPIException: - pass + self.assertRaises(DimensionDataAPIException, node.reboot) def test_destroy_node_response(self): node = Node(id='11', name=None, state=None, @@ -105,12 +95,7 @@ class DimensionDataTests(unittest.TestCase, TestCaseMixin): DimensionDataMockHttp.type = 'INPROGRESS' node = Node(id='11', name=None, state=None, public_ips=None, private_ips=None, driver=self.driver) - try: - node.destroy() - self.assertTrue( - False) # above command should have thrown DimensionDataAPIException - except DimensionDataAPIException: - pass + self.assertRaises(DimensionDataAPIException, node.destroy) def test_list_images(self): images = self.driver.list_images() @@ -163,12 +148,15 @@ class DimensionDataTests(unittest.TestCase, TestCaseMixin): def test_create_node_no_network(self): rootPw = NodeAuthPassword('pass123') image = self.driver.list_images()[0] - try: - self.driver.create_node(name='test2', image=image, auth=rootPw, - ex_description='test2 node', ex_network=None, - ex_isStarted=False) - except ValueError: - pass + self.assertRaises( + ValueError, + self.driver.create_node, + name='test2', + image=image, + auth=rootPw, + ex_description='test2 node', + ex_network=None, + ex_isStarted=False) def test_ex_shutdown_graceful(self): node = Node(id='11', name=None, state=None, @@ -180,12 +168,9 @@ class DimensionDataTests(unittest.TestCase, TestCaseMixin): DimensionDataMockHttp.type = 'INPROGRESS' node = Node(id='11', name=None, state=None, public_ips=None, private_ips=None, driver=self.driver) - try: - self.driver.ex_shutdown_graceful(node) - self.assertTrue( - False) # above command should have thrown DimensionDataAPIException - except DimensionDataAPIException: - pass + self.assertRaises(DimensionDataAPIException, + self.driver.ex_shutdown_graceful, + node) def test_ex_start_node(self): node = Node(id='11', name=None, state=None, @@ -197,12 +182,9 @@ class DimensionDataTests(unittest.TestCase, TestCaseMixin): DimensionDataMockHttp.type = 'INPROGRESS' node = Node(id='11', name=None, state=None, public_ips=None, private_ips=None, driver=self.driver) - try: - self.driver.ex_start_node(node) - self.assertTrue( - False) # above command should have thrown DimensionDataAPIException - except DimensionDataAPIException: - pass + self.assertRaises(DimensionDataAPIException, + self.driver.ex_start_node, + node) def test_ex_power_off(self): node = Node(id='11', name=None, state=None, @@ -220,12 +202,9 @@ class DimensionDataTests(unittest.TestCase, TestCaseMixin): DimensionDataMockHttp.type = 'INPROGRESS' node = Node(id='11', name=None, state=None, public_ips=None, private_ips=None, driver=self.driver) - try: - self.driver.ex_power_off(node) - self.assertTrue( - False) # above command should have thrown DimensionDataAPIException - except DimensionDataAPIException: - pass + self.assertRaises(DimensionDataAPIException, + self.driver.ex_power_off, + node) def test_ex_reset(self): node = Node(id='11', name=None, state=None, @@ -237,7 +216,7 @@ class DimensionDataTests(unittest.TestCase, TestCaseMixin): node = self.driver.ex_get_node_by_id('e75ead52-692f-4314-8725-c8a4f4d13a87') vlan = self.driver.ex_get_vlan('0e56433f-d808-4669-821d-812769517ff8') ret = self.driver.ex_attach_node_to_vlan(node, vlan) - self.assertTrue(ret) + self.assertTrue(ret is True) def test_ex_destroy_nic(self): node = self.driver.ex_destroy_nic('a202e51b-41c0-4cfc-add0-b1c62fc0ecf6')
