diff --git a/nova/tests/test_db_api.py b/nova/tests/test_db_api.py index eb0b8bed..76bd1f03 100644 --- a/nova/tests/test_db_api.py +++ b/nova/tests/test_db_api.py @@ -1182,8 +1182,7 @@ class SMVolumeDBApiTestCase(test.TestCase): def test_sm_backend_conf_delete_nonexisting(self): ctxt = context.get_admin_context() - self.assertNotRaises(None, db.sm_backend_conf_delete, - ctxt, "FA15E-1D") + db.sm_backend_conf_delete(ctxt, "FA15E-1D") def test_sm_flavor_create(self): ctxt = context.get_admin_context() @@ -1231,8 +1230,7 @@ class SMVolumeDBApiTestCase(test.TestCase): def test_sm_flavor_delete_nonexisting(self): ctxt = context.get_admin_context() - self.assertNotRaises(None, db.sm_flavor_delete, - ctxt, 7) + db.sm_flavor_delete(ctxt, 7) def test_sm_flavor_get(self): ctxt = context.get_admin_context() diff --git a/nova/tests/test_xenapi.py b/nova/tests/test_xenapi.py index f8bf253c..0449f3da 100644 --- a/nova/tests/test_xenapi.py +++ b/nova/tests/test_xenapi.py @@ -2437,11 +2437,9 @@ class XenAPILiveMigrateTestCase(stubs.XenAPITestBase): 'destination_sr_ref': None, 'migrate_send_data': None }} - self.assertNotRaises(None, - self.conn.check_can_live_migrate_source, - self.context, - {'host': 'host'}, - dest_check_data) + self.conn.check_can_live_migrate_source(self.context, + {'host': 'host'}, + dest_check_data) def test_check_can_live_migrate_source_with_block_migrate_fails(self): stubs.stubout_session(self.stubs, diff --git a/nova/tests/test_xensm.py b/nova/tests/test_xensm.py index c1748567..25d09a2e 100644 --- a/nova/tests/test_xensm.py +++ b/nova/tests/test_xensm.py @@ -89,11 +89,8 @@ class XenSMTestCase(stubs.XenAPITestBase): ctxt = context.get_admin_context() beconf = self._setup_step(ctxt) volume = self._create_volume() - self.assertNotRaises(None, self.driver.create_volume, volume) - self.assertNotRaises(None, - db.sm_volume_get, - ctxt, - volume['id']) + self.driver.create_volume(volume) + db.sm_volume_get(ctxt, volume['id']) def test_local_path(self): ctxt = context.get_admin_context() @@ -106,7 +103,7 @@ class XenSMTestCase(stubs.XenAPITestBase): beconf = self._setup_step(ctxt) volume = self._create_volume() self.driver.create_volume(volume) - self.assertNotRaises(None, self.driver.delete_volume, volume) + self.driver.delete_volume(volume) self.assertRaises(exception.NotFound, db.sm_volume_get, ctxt,