From: Alexander Gorodnev <agorodnev@mirantis.com>
Date: Tue, 6 Aug 2013 12:31:51 +0000 (-0400)
Subject: Add test for brick.local_dev.lvm
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=945a48e3b62192ce72440f0bb382c5dccab954a5;p=openstack-build%2Fcinder-build.git

Add test for brick.local_dev.lvm

Add positive and negative tests for LVM.lv_has_snapshot() method

Change-Id: Ibb261365eae97d8e3843ff48f7389f759226d2af
---

diff --git a/cinder/tests/brick/test_brick_lvm.py b/cinder/tests/brick/test_brick_lvm.py
index 96775cad8..1751ece7d 100644
--- a/cinder/tests/brick/test_brick_lvm.py
+++ b/cinder/tests/brick/test_brick_lvm.py
@@ -83,6 +83,11 @@ class BrickLvmTestCase(test.TestCase):
                 in cmd_string:
             data = "  fake-volumes fake-1 1.00g\n"
             data += "  fake-volumes fake-2 1.00g\n"
+        elif 'lvdisplay, --noheading, -C, -o, Attr' in cmd_string:
+            if 'test-volumes' in cmd_string:
+                data = '  wi-a-'
+            else:
+                data = '  owi-a-'
         elif 'pvs, --noheadings' and 'fake-volumes' in cmd_string:
             data = "  fake-volumes:/dev/sda:10.00g:8.99g\n"
         elif 'pvs, --noheadings' in cmd_string:
@@ -136,3 +141,7 @@ class BrickLvmTestCase(test.TestCase):
 
         self.stubs.Set(processutils, 'execute', self.fake_old_lvm_version)
         self.assertFalse(self.vg.supports_thin_provisioning())
+
+    def test_lv_has_snapshot(self):
+        self.assertTrue(self.vg.lv_has_snapshot('fake-volumes'))
+        self.assertFalse(self.vg.lv_has_snapshot('test-volumes'))