[RHGSQE-108] Removed polarion test case ID's and updated description

Change-Id: I9ab3c6bea56a02f76ce818ee095ffe4afc0e64c0
Signed-off-by: vamahaja <vamahaja@redhat.com>

4 files changed, 42 insertions, 31 deletions

diff --git a/tests/functional/common/provisioning/test_dynamic_provisioning_block_p0_cases.py b/tests/functional/common/provisioning/test_dynamic_provisioning_block_p0_cases.py
index 43bbec84..bca0bf66 100644
--- a/tests/functional/common/provisioning/test_dynamic_provisioning_block_p0_cases.py
+++ b/tests/functional/common/provisioning/test_dynamic_provisioning_block_p0_cases.py
@@ -57,14 +57,17 @@ class TestDynamicProvisioningBlockP0(GlusterBlockBaseClass):
                 "Failed to execute '%s' command on '%s'." % (cmd, self.node))
 
     def test_dynamic_provisioning_glusterblock_hacount_true(self):
-        """ CNS-435 dynamic provisioning glusterblock """
+        """Validate dynamic provisioning for glusterblock
+        """
         self.dynamic_provisioning_glusterblock(set_hacount=True)
 
     def test_dynamic_provisioning_glusterblock_hacount_false(self):
-        """ CNS-716 storage-class mandatory parameters for block """
+        """Validate storage-class mandatory parameters for block
+        """
         self.dynamic_provisioning_glusterblock(set_hacount=False)
 
     def test_dynamic_provisioning_glusterblock_heketipod_failure(self):
+        """Validate PVC with glusterblock creation when heketi pod is down"""
         datafile_path = '/mnt/fake_file_for_%s' % self.id()
 
         # Create DC with attached PVC
@@ -131,6 +134,7 @@ class TestDynamicProvisioningBlockP0(GlusterBlockBaseClass):
 
     @skip("Blocked by BZ-1632873")
     def test_dynamic_provisioning_glusterblock_glusterpod_failure(self):
+        """Create glusterblock PVC when gluster pod is down"""
         datafile_path = '/mnt/fake_file_for_%s' % self.id()
 
         # Create DC with attached PVC
@@ -173,7 +177,7 @@ class TestDynamicProvisioningBlockP0(GlusterBlockBaseClass):
         self.assertEqual(ret, 0, "IO %s failed on %s" % (io_cmd, self.node))
 
     def test_glusterblock_logs_presence_verification(self):
-        # Verify presence of glusterblock provisioner POD and its status
+        """Validate presence of glusterblock provisioner POD and it's status"""
         gb_prov_cmd = ("oc get pods --all-namespaces "
                        "-l glusterfs=block-%s-provisioner-pod "
                        "-o=custom-columns=:.metadata.name,:.status.phase" % (
@@ -215,7 +219,7 @@ class TestDynamicProvisioningBlockP0(GlusterBlockBaseClass):
                 self.assertTrue(out, "Command '%s' output is empty." % cmd)
 
     def test_dynamic_provisioning_glusterblock_heketidown_pvc_delete(self):
-        """ Delete PVC's when heketi is down CNS-439 """
+        """Validate PVC deletion when heketi is down"""
 
         # Create Secret, SC and PVCs
         self.create_storage_class()
@@ -253,7 +257,7 @@ class TestDynamicProvisioningBlockP0(GlusterBlockBaseClass):
         self.create_and_wait_for_pvc()
 
     def test_recreate_app_pod_with_attached_block_pv(self):
-        """Test Case CNS-1392"""
+        """Validate app pod attached block device I/O after restart"""
         datafile_path = '/mnt/temporary_test_file'
 
         # Create DC with POD and attached PVC to it
@@ -281,7 +285,7 @@ class TestDynamicProvisioningBlockP0(GlusterBlockBaseClass):
         self.cmd_run(write_cmd % (new_pod_name, datafile_path))
 
     def test_volname_prefix_glusterblock(self):
-        # CNS-926 - custom_volname_prefix_blockvol
+        """Validate custom volname prefix blockvol"""
 
         self.dynamic_provisioning_glusterblock(
             set_hacount=False, create_vol_name_prefix=True)
@@ -300,7 +304,7 @@ class TestDynamicProvisioningBlockP0(GlusterBlockBaseClass):
             self.sc.get('volumenameprefix', 'autotest')))
 
     def test_dynamic_provisioning_glusterblock_reclaim_policy_retain(self):
-        # CNS-1391 - Retain policy - gluster-block - delete pvc
+        """Validate retain policy for gluster-block after PVC deletion"""
 
         self.create_storage_class(reclaim_policy='Retain')
         self.create_and_wait_for_pvc()
diff --git a/tests/functional/common/provisioning/test_dynamic_provisioning_p0_cases.py b/tests/functional/common/provisioning/test_dynamic_provisioning_p0_cases.py
index d550e51c..81329e08 100644
--- a/tests/functional/common/provisioning/test_dynamic_provisioning_p0_cases.py
+++ b/tests/functional/common/provisioning/test_dynamic_provisioning_p0_cases.py
@@ -76,14 +76,19 @@ class TestDynamicProvisioningP0(BaseClass):
                 "Failed to execute '%s' command on %s" % (cmd, self.node))
 
     def test_dynamic_provisioning_glusterfile(self):
+        """Validate dynamic provisioning for gluster file"""
         g.log.info("test_dynamic_provisioning_glusterfile")
         self.dynamic_provisioning_glusterfile(False)
 
     def test_dynamic_provisioning_glusterfile_volname_prefix(self):
+        """Validate dynamic provisioning for gluster file with vol name prefix
+        """
         g.log.info("test_dynamic_provisioning_glusterfile volname prefix")
         self.dynamic_provisioning_glusterfile(True)
 
     def test_dynamic_provisioning_glusterfile_heketipod_failure(self):
+        """Validate dynamic provisioning for gluster file when heketi pod down
+        """
         mount_path = "/mnt"
         datafile_path = '%s/fake_file_for_%s' % (mount_path, self.id())
 
@@ -165,6 +170,8 @@ class TestDynamicProvisioningP0(BaseClass):
 
     @skip("Blocked by BZ-1632873")
     def test_dynamic_provisioning_glusterfile_glusterpod_failure(self):
+        """Validate dynamic provisioning for gluster file when gluster pod down
+        """
         mount_path = "/mnt"
         datafile_path = '%s/fake_file_for_%s' % (mount_path, self.id())
 
@@ -220,7 +227,7 @@ class TestDynamicProvisioningP0(BaseClass):
         self.assertEqual(ret, 0, "IO %s failed on %s" % (io_cmd, self.node))
 
     def test_storage_class_mandatory_params_glusterfile(self):
-        """Test case CNS-442 - storage-class mandatory parameters"""
+        """Validate storage-class creation with mandatory parameters"""
 
         # create secret
         self.secret_name = oc_create_secret(
@@ -269,7 +276,7 @@ class TestDynamicProvisioningP0(BaseClass):
             ret, 0, "Failed to execute command %s on %s" % (cmd, self.node))
 
     def test_dynamic_provisioning_glusterfile_heketidown_pvc_delete(self):
-        """ Delete PVC's when heketi is down CNS-438 """
+        """Validate deletion of PVC's when heketi is down"""
 
         # Create storage class, secret and PVCs
         self.create_storage_class()
@@ -307,7 +314,7 @@ class TestDynamicProvisioningP0(BaseClass):
         self.create_and_wait_for_pvc()
 
     def test_validate_pvc_in_multiple_app_pods(self):
-        """Test case CNS-574"""
+        """Validate the use of a same claim in multiple app pods"""
         replicas = 5
 
         # Create PVC
@@ -336,7 +343,7 @@ class TestDynamicProvisioningP0(BaseClass):
             self.assertIn("temp_%s" % pod_name, ls_out)
 
     def test_pvc_deletion_while_pod_is_running(self):
-        """Test case CNS-584 - Verify PVC deletion while pod is running"""
+        """Validate PVC deletion while pod is running"""
 
         # Create DC with POD and attached PVC to it
         sc_name = self.create_storage_class()
@@ -359,7 +366,7 @@ class TestDynamicProvisioningP0(BaseClass):
             ret, 0, "Failed to execute command %s on %s" % (cmd, self.node))
 
     def test_dynamic_provisioning_glusterfile_reclaim_policy_retain(self):
-        # CNS-1390 - Retain policy - glusterfs - delete pvc
+        """Validate retain policy for glusterfs after deletion of pvc"""
 
         self.create_storage_class(reclaim_policy='Retain')
         self.create_and_wait_for_pvc()
@@ -403,7 +410,7 @@ class TestDynamicProvisioningP0(BaseClass):
         wait_for_resource_absence(self.node, 'pv', pv_name)
 
     def test_usage_of_default_storage_class(self):
-        """Test case CNS-928"""
+        """Validate PVs creation for SC with default custom volname prefix"""
 
         # Unset 'default' option from all the existing Storage Classes
         unset_sc_annotation_cmd = (
diff --git a/tests/functional/common/provisioning/test_pv_resize.py b/tests/functional/common/provisioning/test_pv_resize.py
index 52a5915f..99c1e451 100644
--- a/tests/functional/common/provisioning/test_pv_resize.py
+++ b/tests/functional/common/provisioning/test_pv_resize.py
@@ -23,7 +23,7 @@ from glusto.core import Glusto as g
 
 @ddt.ddt
 class TestPvResizeClass(BaseClass):
-    """Test cases for PV resize."""
+    """Test cases for PV resize"""
 
     @classmethod
     def setUpClass(cls):
@@ -45,7 +45,7 @@ class TestPvResizeClass(BaseClass):
     @ddt.data(False, True)
     def test_pv_resize_with_prefix_for_name(self,
                                             create_vol_name_prefix=False):
-        """testcases CNS-1037 and CNS-1038 """
+        """Validate PV resize with and without name prefix"""
         dir_path = "/mnt/"
         node = self.ocp_client[0]
 
@@ -190,15 +190,15 @@ class TestPvResizeClass(BaseClass):
                 ret, 0, "Failed to write data on the expanded PVC")
 
     def test_pv_resize_no_free_space(self):
-        """Test case CNS-1040"""
+        """Validate PVC resize fails if there is no free space available"""
         self._pv_resize(exceed_free_space=True)
 
     def test_pv_resize_by_exact_free_space(self):
-        """Test case CNS-1041"""
+        """Validate PVC resize when resized by exact available free space"""
         self._pv_resize(exceed_free_space=False)
 
     def test_pv_resize_try_shrink_pv_size(self):
-        """testcase CNS-1039 """
+        """Validate whether reducing the PV size is allowed"""
         dir_path = "/mnt/"
         node = self.ocp_master_node[0]
 
diff --git a/tests/functional/common/provisioning/test_storage_class_cases.py b/tests/functional/common/provisioning/test_storage_class_cases.py
index 8fd001dd..e9dc8dbe 100644
--- a/tests/functional/common/provisioning/test_storage_class_cases.py
+++ b/tests/functional/common/provisioning/test_storage_class_cases.py
@@ -151,7 +151,7 @@ class TestStorageClassCases(cns_baseclass.BaseClass):
         {"volumenameprefix": "dept_qe"},
         )
     def test_sc_glusterfile_incorrect_parameter(self, parameter={}):
-        """Polarion testcase id- CNS-708,709,713,714,715,921"""
+        """Validate glusterfile storage with different incorrect parameters"""
         self.create_sc_with_parameter("glusterfile", parameter=parameter)
 
     @ddt.data(
@@ -161,15 +161,15 @@ class TestStorageClassCases(cns_baseclass.BaseClass):
         {"restuser": "fakeuser"},
         )
     def test_sc_glusterblock_incorrect_parameter(self, parameter={}):
-        """ Polarion testcase id- CNS-727,725,728"""
+        """Validate glusterblock storage with different incorrect parameters"""
         self.create_sc_with_parameter("glusterblock", parameter=parameter)
 
     @skip("Blocked by BZ-1609703")
     @ddt.data(1, 2)
     def test_gluster_block_provisioning_with_valid_ha_count(self, hacount):
-        '''[CNS-544][CNS-1453] gluster-block provisioning with different valid
-           'hacount' values
-        '''
+        """Validate gluster-block provisioning with different valid 'hacount'
+           values
+        """
         # create storage class and pvc with given parameters
         self.create_sc_with_parameter(
             'glusterblock', success=True, parameter={'hacount': str(hacount)}
@@ -186,9 +186,9 @@ class TestStorageClassCases(cns_baseclass.BaseClass):
             self.validate_multipath_info(hacount)
 
     def test_gluster_block_provisioning_with_ha_count_as_glusterpod(self):
-        '''[CNS-1443] gluster-block provisioning with "hacount" value equal to
-           gluster pods count
-        '''
+        """Validate gluster-block provisioning with "hacount" value equal
+           to gluster pods count
+        """
         # get hacount as no of gluster pods the pvc creation
         hacount = get_amount_of_gluster_nodes(self.ocp_master_node[0])
 
@@ -205,9 +205,9 @@ class TestStorageClassCases(cns_baseclass.BaseClass):
 
     @skip("Blocked by BZ-1644685")
     def test_gluster_block_provisioning_with_invalid_ha_count(self):
-        '''[CNS-1444] gluster-block provisioning with any invalid 'hacount'
+        """Validate gluster-block provisioning with any invalid 'hacount'
            value
-        '''
+        """
         # get hacount as no of gluster pods + 1 to fail the pvc creation
         hacount = get_amount_of_gluster_nodes(self.ocp_master_node[0]) + 1
 
@@ -218,9 +218,9 @@ class TestStorageClassCases(cns_baseclass.BaseClass):
 
     @ddt.data('true', 'false', '')
     def test_gluster_block_chapauthenabled_parameter(self, chapauthenabled):
-        '''[CNS-545][CNS-1445][CNS-1446] gluster-block provisioning with
-           different 'chapauthenabled' values
-        '''
+        """Validate gluster-block provisioning with different
+           'chapauthenabled' values
+        """
         parameter = {}
         if chapauthenabled:
             parameter = {"chapauthenabled": chapauthenabled}