shaba/lvm2
1
0
Fork 0
mirror of git://sourceware.org/git/lvm2.git synced 2024-12-21 13:34:40 +03:00
Code Releases Activity

python_lvm_unit.py: Clean-up method names & scope

Browse Source 
Changed naming of methods from camel case to all lower case with
underscores per guidelines.  Changed any methods that can be
static methods to static.

Signed-off-by: Tony Asleson <tasleson@redhat.com>
This commit is contained in:
Tony Asleson 2013-11-18 15:07:32 -06:00
parent 12d5e53953
commit 370520a310
1 changed files with 158 additions and 142 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

300
test/api/python_lvm_unit.py
View File

@ -39,11 +39,12 @@ def l(txt):
fh.flush()
def rs(l=10):
def rs(rand_len=10):
"""
Generate a random string
"""
return ''.join(random.choice(string.ascii_uppercase) for x in range(l))
return ''.join(random.choice(string.ascii_uppercase)
for x in range(rand_len))
def _get_allowed_devices():
@ -54,9 +55,9 @@ def _get_allowed_devices():
return rc
def compare_pv(r, l):
r_name = r.getName()
l_name = l.getName()
def compare_pv(right, left):
r_name = right.getName()
l_name = left.getName()
if r_name > l_name:
return 1
@ -74,6 +75,7 @@ class AllowedPVS(object):
def __init__(self):
self.handle = None
self.pvs_all = None
def __enter__(self):
rc = []
@ -102,24 +104,27 @@ class TestLvm(unittest.TestCase):
VG_P = 'py_unit_test_'
def _get_pv_device_names(self):
@staticmethod
def _get_pv_device_names():
rc = []
with AllowedPVS() as pvs:
for p in pvs:
rc.append(p.getName())
return rc
def _createThickLV(self, device_list, name):
@staticmethod
def _create_thick_lv(device_list, name):
vg = lvm.vgCreate(TestLvm.VG_P + "_" + name)
for d in device_list:
vg.extend(d)
vg.createLvLinear(name, vg.getSize()/2)
vg.createLvLinear(name, vg.getSize() / 2)
vg.close()
vg = None
def _createThinPool(self, device_list, pool_name):
@staticmethod
def _create_thin_pool(device_list, pool_name):
vg = lvm.vgCreate(TestLvm.VG_P + "_" + pool_name)
for d in device_list:
@ -129,14 +134,16 @@ class TestLvm(unittest.TestCase):
lvm.THIN_DISCARDS_PASSDOWN, 1)
return vg
def _createThinLV(self, pv_devices, name):
@staticmethod
def _create_thin_lv(pv_devices, name):
thin_pool_name = 'thin_vg_pool_' + rs(4)
vg = self._createThinPool(pv_devices, thin_pool_name)
vg = TestLvm._create_thin_pool(pv_devices, thin_pool_name)
vg.createLvThin(thin_pool_name, name, vg.getSize()/8)
vg.close()
vg = None
def _vg_names(self):
@staticmethod
def _vg_names():
rc = []
vg_names = lvm.listVgNames()
@ -146,25 +153,27 @@ class TestLvm(unittest.TestCase):
return rc
def _get_lv(self, lv_vol_type=None, lv_name=None):
vg_name_list = self._vg_names()
@staticmethod
def _get_lv(lv_vol_type=None, lv_name=None):
vg_name_list = TestLvm._vg_names()
for vg_name in vg_name_list:
vg = lvm.vgOpen(vg_name, "w")
lvs = vg.listLVs()
for l in lvs:
attr = l.getAttr()
for lv in lvs:
attr = lv.getAttr()
if lv_vol_type or lv_name:
if lv_vol_type is not None and attr[0] == lv_vol_type:
return l, vg
elif lv_name is not None and lv_name == l.getName():
return l, vg
return lv, vg
elif lv_name is not None and lv_name == lv.getName():
return lv, vg
else:
return l, vg
return lv, vg
vg.close()
return None, None
def _remove_VG(self, vg_name):
@staticmethod
def _remove_vg(vg_name):
vg = lvm.vgOpen(vg_name, 'w')
pvs = vg.listPVs()
@ -172,26 +181,26 @@ class TestLvm(unittest.TestCase):
pe_devices = []
#Remove old snapshots first, then lv
for l in vg.listLVs():
attr = l.getAttr()
for lv in vg.listLVs():
attr = lv.getAttr()
if attr[0] == 's':
l.remove()
lv.remove()
lvs = vg.listLVs()
#Now remove any thin lVs
for l in vg.listLVs():
attr = l.getAttr()
for lv in vg.listLVs():
attr = lv.getAttr()
if attr[0] == 'V':
l.remove()
lv.remove()
#now remove the rest
for l in vg.listLVs():
name = l.getName()
for lv in vg.listLVs():
name = lv.getName()
#Don't remove the hidden ones
if 'tmeta' not in name and 'tdata' not in name:
l.remove()
if '_tmeta' not in name and '_tdata' not in name:
lv.remove()
for p in pvs:
pe_devices.append(p.getName())
@ -202,23 +211,28 @@ class TestLvm(unittest.TestCase):
vg.remove()
vg.close()
def _clean_up(self):
@staticmethod
def _clean_up():
#Clear out the testing PVs, but only if they contain stuff
#this unit test created
for vg_n in self._vg_names():
self._remove_VG(vg_n)
for vg_n in TestLvm._vg_names():
TestLvm._remove_vg(vg_n)
for d in TestLvm._get_pv_device_names():
lvm.pvRemove(d)
lvm.pvCreate(d)
def setUp(self):
device_list = self._get_pv_device_names()
device_list = TestLvm._get_pv_device_names()
#Make sure we have an adequate number of PVs to use
self.assertTrue(len(device_list) >= 4)
self._clean_up()
TestLvm._clean_up()
def tearDown(self):
self._clean_up()
TestLvm._clean_up()
def testPVresize(self):
def test_pv_resize(self):
with AllowedPVS() as pvs:
pv = pvs[0]
curr_size = pv.getSize()
@ -231,7 +245,7 @@ class TestLvm(unittest.TestCase):
self.assertTrue(resized_size != curr_size)
pv.resize(dev_size)
def testPVlifecycle(self):
def test_pv_life_cycle(self):
"""
Test removing and re-creating a PV
"""
@ -256,7 +270,8 @@ class TestLvm(unittest.TestCase):
self.assertTrue(found)
def testPvMethods(self):
@staticmethod
def test_pv_methods():
with AllowedPVS() as pvs:
for p in pvs:
p.getName()
@ -267,13 +282,13 @@ class TestLvm(unittest.TestCase):
p.getFree()
p = None
def testVersion(self):
def test_version(self):
version = lvm.getVersion()
self.assertNotEquals(version, None)
self.assertEquals(type(version), str)
self.assertTrue(len(version) > 0)
def testPvGetters(self):
def test_pv_getters(self):
with AllowedPVS() as pvs:
pv = pvs[0]
self.assertEqual(type(pv.getName()), str)
@ -301,7 +316,7 @@ class TestLvm(unittest.TestCase):
self.assertEqual(type(result[1]), bool)
self.assertTrue(result[1] == settable)
def testPvSegs(self):
def test_pv_segs(self):
with AllowedPVS() as pvs:
pv = pvs[0]
pv_segs = pv.listPVsegs()
@ -311,39 +326,39 @@ class TestLvm(unittest.TestCase):
for i in pv_segs:
self._test_prop(i, 'pvseg_start', long, False)
def testPvProperty(self):
def test_pv_property(self):
with AllowedPVS() as pvs:
pv = pvs[0]
self._test_prop(pv, 'pv_mda_count', long, False)
def testLvProperty(self):
def test_lv_property(self):
lv_name = 'lv_test'
self._createThinLV(self._get_pv_device_names(), lv_name)
lv, vg = self._get_lv(None, lv_name)
TestLvm._create_thin_lv(TestLvm._get_pv_device_names(), lv_name)
lv, vg = TestLvm._get_lv(None, lv_name)
self._test_prop(lv, 'seg_count', long, False)
vg.close()
def testLvTags(self):
def test_lv_tags(self):
lv_name = 'lv_test'
self._createThinLV(self._get_pv_device_names(), lv_name)
lv, vg = self._get_lv(None, lv_name)
self._testTags(lv)
TestLvm._create_thin_lv(TestLvm._get_pv_device_names(), lv_name)
lv, vg = TestLvm._get_lv(None, lv_name)
self._test_tags(lv)
vg.close()
def testLvActiveInactive(self):
def test_lv_active_inactive(self):
lv_name = 'lv_test'
self._createThinLV(self._get_pv_device_names(), lv_name)
lv, vg = self._get_lv(None, lv_name)
TestLvm._create_thin_lv(TestLvm._get_pv_device_names(), lv_name)
lv, vg = TestLvm._get_lv(None, lv_name)
lv.deactivate()
self.assertTrue(lv.isActive() is False)
lv.activate()
self.assertTrue(lv.isActive() is True)
vg.close()
def testLvRename(self):
def test_lv_rename(self):
lv_name = 'lv_test'
self._createThinLV(self._get_pv_device_names(), lv_name)
lv, vg = self._get_lv(None, lv_name)
TestLvm._create_thin_lv(TestLvm._get_pv_device_names(), lv_name)
lv, vg = TestLvm._get_lv(None, lv_name)
current_name = lv.getName()
new_name = rs()
@ -352,29 +367,29 @@ class TestLvm(unittest.TestCase):
lv.rename(current_name)
vg.close()
def testLvSnapshot(self):
def test_lv_snapshot(self):
thin_lv = 'thin_lv'
thick_lv = 'thick_lv'
device_names = self._get_pv_device_names()
device_names = TestLvm._get_pv_device_names()
self._createThinLV(device_names[0:2], thin_lv)
self._createThickLV(device_names[2:4], thick_lv)
TestLvm._create_thin_lv(device_names[0:2], thin_lv)
TestLvm._create_thick_lv(device_names[2:4], thick_lv)
lv, vg = self._get_lv(None, thick_lv)
lv, vg = TestLvm._get_lv(None, thick_lv)
lv.snapshot('thick_snap_shot', 1024*1024)
vg.close()
thick_ss, vg = self._get_lv(None, 'thick_snap_shot')
thick_ss, vg = TestLvm._get_lv(None, 'thick_snap_shot')
self.assertTrue(thick_ss is not None)
vg.close()
thin_lv, vg = self._get_lv(None, thin_lv)
thin_lv, vg = TestLvm._get_lv(None, thin_lv)
thin_lv.snapshot('thin_snap_shot')
vg.close()
thin_ss, vg = self._get_lv(None, 'thin_snap_shot')
thin_ss, vg = TestLvm._get_lv(None, 'thin_snap_shot')
self.assertTrue(thin_ss is not None)
origin = thin_ss.getOrigin()
@ -382,38 +397,38 @@ class TestLvm(unittest.TestCase):
vg.close()
def testLvSuspend(self):
def test_lv_suspend(self):
lv_name = 'lv_test'
self._createThinLV(self._get_pv_device_names(), lv_name)
lv, vg = self._get_lv(None, lv_name)
TestLvm._create_thin_lv(TestLvm._get_pv_device_names(), lv_name)
lv, vg = TestLvm._get_lv(None, lv_name)
result = lv.isSuspended()
self.assertTrue(type(result) == bool)
vg.close()
def testLvSize(self):
def test_lv_size(self):
lv_name = 'lv_test'
self._createThinLV(self._get_pv_device_names(), lv_name)
lv, vg = self._get_lv(None, lv_name)
TestLvm._create_thin_lv(TestLvm._get_pv_device_names(), lv_name)
lv, vg = TestLvm._get_lv(None, lv_name)
result = lv.getSize()
self.assertTrue(type(result) == int or type(result) == long)
vg.close()
def testLvResize(self):
def test_lv_resize(self):
lv_name = 'lv_test'
self._createThinLV(self._get_pv_device_names(), lv_name)
lv, vg = self._get_lv(None, lv_name)
TestLvm._create_thin_lv(TestLvm._get_pv_device_names(), lv_name)
lv, vg = TestLvm._get_lv(None, lv_name)
curr_size = lv.getSize()
lv.resize(curr_size+(1024*1024))
latest = lv.getSize()
self.assertTrue(curr_size != latest)
def testLvSeg(self):
def test_lv_seg(self):
lv_name = 'lv_test'
self._createThinLV(self._get_pv_device_names(), lv_name)
lv, vg = self._get_lv(None, lv_name)
TestLvm._create_thin_lv(TestLvm._get_pv_device_names(), lv_name)
lv, vg = TestLvm._get_lv(None, lv_name)
lv_segs = lv.listLVsegs()
@ -424,11 +439,11 @@ class TestLvm(unittest.TestCase):
vg.close()
def testGetSetExtentSize(self):
def test_get_set_extend_size(self):
thick_lv = 'get_set_prop'
device_names = self._get_pv_device_names()
self._createThickLV(device_names[0:2], thick_lv)
lv, vg = self._get_lv(None, thick_lv)
device_names = TestLvm._get_pv_device_names()
TestLvm._create_thick_lv(device_names[0:2], thick_lv)
lv, vg = TestLvm._get_lv(None, thick_lv)
new_extent = 1024 * 1024 * 4
@ -439,11 +454,11 @@ class TestLvm(unittest.TestCase):
self.assertEqual(vg.getExtentSize(), new_extent)
vg.close()
def testVGsetGetProp(self):
def test_vg_get_set_prop(self):
thick_lv = 'get_set_prop'
device_names = self._get_pv_device_names()
self._createThickLV(device_names[0:2], thick_lv)
lv, vg = self._get_lv(None, thick_lv)
device_names = TestLvm._get_pv_device_names()
TestLvm._create_thick_lv(device_names[0:2], thick_lv)
lv, vg = TestLvm._get_lv(None, thick_lv)
self.assertTrue(vg is not None)
if vg:
@ -451,14 +466,14 @@ class TestLvm(unittest.TestCase):
vg.setProperty('vg_mda_copies', vg_mda_copies[0])
vg.close()
def testVGremoveRestore(self):
def test_vg_remove_restore(self):
#Store off the list of physical devices
pv_devices = []
thick_lv = 'get_set_prop'
device_names = self._get_pv_device_names()
self._createThickLV(device_names[0:2], thick_lv)
lv, vg = self._get_lv(None, thick_lv)
device_names = TestLvm._get_pv_device_names()
TestLvm._create_thick_lv(device_names[0:2], thick_lv)
lv, vg = TestLvm._get_lv(None, thick_lv)
vg_name = vg.getName()
@ -467,22 +482,22 @@ class TestLvm(unittest.TestCase):
pv_devices.append(p.getName())
vg.close()
self._remove_VG(vg_name)
self._createThickLV(pv_devices, thick_lv)
TestLvm._remove_vg(vg_name)
self._create_thick_lv(pv_devices, thick_lv)
def testVgNames(self):
def test_vg_names(self):
vg = lvm.listVgNames()
self.assertTrue(isinstance(vg, tuple))
def testDupeLvCreate(self):
def test_dupe_lv_create(self):
"""
Try to create a lv with the same name expecting a failure
Note: This was causing a seg. fault previously
"""
thick_lv = 'dupe_name'
device_names = self._get_pv_device_names()
self._createThickLV(device_names[0:2], thick_lv)
lv, vg = self._get_lv(None, thick_lv)
device_names = TestLvm._get_pv_device_names()
TestLvm._create_thick_lv(device_names[0:2], thick_lv)
lv, vg = TestLvm._get_lv(None, thick_lv)
self.assertTrue(vg is not None)
@ -496,11 +511,11 @@ class TestLvm(unittest.TestCase):
lv.getSize())
vg.close()
def testVgUuids(self):
def test_vg_uuids(self):
device_names = self._get_pv_device_names()
self._createThinLV(device_names[0:2], 'thin')
self._createThickLV(device_names[2:4], 'thick')
device_names = TestLvm._get_pv_device_names()
TestLvm._create_thin_lv(device_names[0:2], 'thin')
TestLvm._create_thick_lv(device_names[2:4], 'thick')
vgs_uuids = lvm.listVgUuids()
@ -523,12 +538,12 @@ class TestLvm(unittest.TestCase):
self.assertTrue(len(vgs_uuids) == 0)
def testPvLookupFromVG(self):
device_names = self._get_pv_device_names()
self._createThinLV(device_names[0:2], 'thin')
self._createThickLV(device_names[2:4], 'thick')
def test_pv_lookup_from_vg(self):
device_names = TestLvm._get_pv_device_names()
TestLvm._create_thin_lv(device_names[0:2], 'thin')
TestLvm._create_thick_lv(device_names[2:4], 'thick')
vg_names = self._vg_names()
vg_names = TestLvm._vg_names()
self.assertTrue(len(vg_names) > 0)
@ -558,31 +573,31 @@ class TestLvm(unittest.TestCase):
pvs = None
vg.close()
def testPercentToFloat(self):
def test_percent_to_float(self):
self.assertEqual(lvm.percentToFloat(0), 0.0)
self.assertEqual(lvm.percentToFloat(1000000), 1.0)
self.assertEqual(lvm.percentToFloat(1000000 / 2), 0.5)
def testScan(self):
def test_scan(self):
self.assertEqual(lvm.scan(), None)
def testConfigReload(self):
def test_config_reload(self):
self.assertEqual(lvm.configReload(), None)
def testConfig_override(self):
def test_config_override(self):
self.assertEquals(lvm.configOverride("global.test = 1"), None)
def testConfigFindBool(self):
def test_config_find_bool(self):
either_or = lvm.configFindBool("global/fallback_to_local_locking")
self.assertTrue(type(either_or) == bool)
self.assertTrue(lvm.configFindBool("global/locking_type"))
def testVgFromPVLookups(self):
device_names = self._get_pv_device_names()
self._createThinLV(device_names[0:2], 'thin')
self._createThickLV(device_names[2:4], 'thick')
def test_vg_from_pv_lookups(self):
device_names = TestLvm._get_pv_device_names()
TestLvm._create_thin_lv(device_names[0:2], 'thin')
TestLvm._create_thick_lv(device_names[2:4], 'thick')
vgname_list = self._vg_names()
vgname_list = TestLvm._vg_names()
self.assertTrue(len(vgname_list) > 0)
@ -598,12 +613,12 @@ class TestLvm(unittest.TestCase):
self.assertEqual(vg_name, lvm.vgNameFromDevice(pv.getName()))
vg.close()
def testVgGetName(self):
device_names = self._get_pv_device_names()
self._createThinLV(device_names[0:2], 'thin')
self._createThickLV(device_names[2:4], 'thick')
def test_vg_get_name(self):
device_names = TestLvm._get_pv_device_names()
TestLvm._create_thin_lv(device_names[0:2], 'thin')
TestLvm._create_thick_lv(device_names[2:4], 'thick')
vgname_list = self._vg_names()
vgname_list = TestLvm._vg_names()
self.assertTrue(len(vgname_list) > 0)
@ -612,12 +627,12 @@ class TestLvm(unittest.TestCase):
self.assertEqual(vg.getName(), vg_name)
vg.close()
def testVgGetUuid(self):
device_names = self._get_pv_device_names()
self._createThinLV(device_names[0:2], 'thin')
self._createThickLV(device_names[2:4], 'thick')
def test_vg_get_uuid(self):
device_names = TestLvm._get_pv_device_names()
TestLvm._create_thin_lv(device_names[0:2], 'thin')
TestLvm._create_thick_lv(device_names[2:4], 'thick')
vgname_list = self._vg_names()
vgname_list = TestLvm._vg_names()
self.assertTrue(len(vgname_list) > 0)
@ -632,12 +647,12 @@ class TestLvm(unittest.TestCase):
"getExtentSize", "getExtentCount", "getFreeExtentCount",
"getPvCount", "getMaxPv", "getMaxLv"]
def testVgGetters(self):
device_names = self._get_pv_device_names()
self._createThinLV(device_names[0:2], 'thin')
self._createThickLV(device_names[2:4], 'thick')
def test_vg_getters(self):
device_names = TestLvm._get_pv_device_names()
TestLvm._create_thin_lv(device_names[0:2], 'thin')
TestLvm._create_thick_lv(device_names[2:4], 'thick')
vg_name_list = self._vg_names()
vg_name_list = TestLvm._vg_names()
self.assertTrue(len(vg_name_list) > 0)
@ -655,7 +670,7 @@ class TestLvm(unittest.TestCase):
vg.close()
def _testTags(self, tag_obj):
def _test_tags(self, tag_obj):
existing_tags = tag_obj.getTags()
self.assertTrue(type(existing_tags) == tuple)
@ -688,23 +703,24 @@ class TestLvm(unittest.TestCase):
for e in existing_tags:
self.assertTrue(e in current_tags)
def testVgTags(self):
device_names = self._get_pv_device_names()
def test_vg_tags(self):
device_names = TestLvm._get_pv_device_names()
i = 0
for d in device_names:
if i % 2 == 0:
self._createThinLV([d], "thin_lv%d" % i)
TestLvm._create_thin_lv([d], "thin_lv%d" % i)
else:
self._createThickLV([d], "thick_lv%d" % i)
TestLvm._create_thick_lv([d], "thick_lv%d" % i)
i += 1
for vg_name in self._vg_names():
for vg_name in TestLvm._vg_names():
vg = lvm.vgOpen(vg_name, 'w')
self._testTags(vg)
self._test_tags(vg)
vg.close()
def testListing(self):
@staticmethod
def test_listing():
env = os.environ
@ -719,12 +735,12 @@ class TestLvm(unittest.TestCase):
for v in lvm.listVgNames():
l('vg= %s' % v)
def testPVemptylisting(self):
def test_pv_empty_listing(self):
#We had a bug where we would seg. fault if we had no PVs.
l('testPVemptylisting entry')
device_names = self._get_pv_device_names()
device_names = TestLvm._get_pv_device_names()
for d in device_names:
l("Removing %s" % d)
@ -742,7 +758,7 @@ class TestLvm(unittest.TestCase):
for d in device_names:
lvm.pvCreate(d)
def testPVCreate(self):
def test_pv_create(self):
size = [0, 1024*1024*4]
pvmeta_copies = [0, 1, 2]
pvmeta_size = [0, 255, 512, 1024]
@ -750,7 +766,7 @@ class TestLvm(unittest.TestCase):
data_alignment_offset = [1, 1, 1]
zero = [0, 1]
device_names = self._get_pv_device_names()
device_names = TestLvm._get_pv_device_names()
for d in device_names:
lvm.pvRemove(d)