]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
GlusterFS: Renaming test case to test_mount_glusterfs
authorBharat Kumar Kobagana <bharat.kobagana@redhat.com>
Mon, 11 May 2015 07:19:21 +0000 (12:49 +0530)
committerBharat Kumar Kobagana <bharat.kobagana@redhat.com>
Mon, 11 May 2015 10:19:12 +0000 (15:49 +0530)
With the patch [1], GlusterFS is now using mount method in
RemoteFsClient. That method mounts a gluster share only if it has
not already been mounted.

So, having test case name as
test_mount_glusterfs_should_mount_correctly, doesn't make sense now.
This patch changes its name to 'test_mount_glusterfs'.

[1] https://review.openstack.org/#/c/165424/

Change-Id: I33c16a8315a11fc16ab478d667490f3f73002ee8

cinder/tests/unit/test_glusterfs.py

index 8e453e41484cadd69121788e3d79c44565d7654f..62ce241f66a86bf69449e7b3dfb7134d0b01d01a 100644 (file)
@@ -147,7 +147,7 @@ class GlusterFsDriverTestCase(test.TestCase):
             '/mnt/test/ab03ab34eaca46a5fb81878f7e9b91fc/volume-123',
             drv.local_path(volume))
 
-    def test_mount_glusterfs_should_mount_correctly(self):
+    def test_mount_glusterfs(self):
         """_mount_glusterfs common case usage."""
         drv = self._driver