Skip to content

Commit

Permalink
Add missing tests for iscsi_* methods
Browse files Browse the repository at this point in the history
There was no tests in test_db_api for iscsi_*
methods.

Bug 1187367 found.

blueprint db-api-tests

Change-Id: Ia554e6030307a0c4f0510ab74fe0881ee160f743
  • Loading branch information
yuriy-z committed Jun 4, 2013
1 parent a003e4a commit 34e649e
Showing 1 changed file with 29 additions and 0 deletions.
29 changes: 29 additions & 0 deletions cinder/tests/test_db_api.py
Expand Up @@ -539,3 +539,32 @@ def test_quota_usage_get_all_by_project(self):
'res2': {'in_use': 2, 'reserved': 2}}
self.assertEqual(expected, db.quota_usage_get_all_by_project(
self.ctxt, 'p1'))


class DBAPIIscsiTargetTestCase(BaseTest):

"""Unit tests for cinder.db.api.iscsi_target_*."""

def _get_base_values(self):
return {'target_num': 10, 'host': 'fake_host'}

def test_iscsi_target_create_safe(self):
target = db.iscsi_target_create_safe(self.ctxt,
self._get_base_values())
self.assertTrue(target['id'])
self.assertEqual(target['host'], 'fake_host')
self.assertEqual(target['target_num'], 10)

def test_iscsi_target_count_by_host(self):
for i in range(3):
values = self._get_base_values()
values['target_num'] += i
db.iscsi_target_create_safe(self.ctxt, values)
self.assertEqual(db.iscsi_target_count_by_host(self.ctxt, 'fake_host'),
3)

@test.testtools.skip("bug 1187367")
def test_integrity_error(self):
db.iscsi_target_create_safe(self.ctxt, self._get_base_values())
self.assertFalse(db.iscsi_target_create_safe(self.ctxt,
self._get_base_values()))

0 comments on commit 34e649e

Please sign in to comment.