Commit a551ad87 authored by Darek Stojaczyk's avatar Darek Stojaczyk Committed by Jim Harris
Browse files

test/lvol: remove destroy_lvol_store_positive 



We already perform the same steps as a part of different
test case in basic.sh.

Change-Id: I0e1931307b3de69df95b2a2994b87147028677e1
Signed-off-by: default avatarDarek Stojaczyk <dariusz.stojaczyk@intel.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/462464


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarKarol Latecki <karol.latecki@intel.com>
Reviewed-by: default avatarMaciej Szwed <maciej.szwed@intel.com>
parent cf628b09

test/lvol/lvol.sh

+0 −1
Original line number Diff line number Diff line
@@ -21,7 +21,6 @@ function usage() { 
    echo "    --block-size          Block size for this bdev"

    echo "-x                        set -x for script debug"

    echo "    --test-cases=         List test cases which will be run:

                                    250: 'bdev_lvol_delete_lvstore_positive',

                                    251: 'bdev_lvol_delete_lvstore_use_name_positive',

                                    252: 'bdev_lvol_delete_lvstore_with_lvol_bdev_positive',

                                    253: 'destroy_multi_logical_volumes_positive',

test/lvol/test_cases.py

+0 −31
Original line number Diff line number Diff line
@@ -112,7 +112,6 @@ def case_message(func): 
    def inner(*args, **kwargs):

        test_name = {

            # bdev_lvol_delete_lvstore - positive tests

            250: 'bdev_lvol_delete_lvstore_positive',

            251: 'bdev_lvol_delete_lvstore_use_name_positive',

            252: 'bdev_lvol_delete_lvstore_with_lvol_bdev_positive',

            253: 'destroy_multi_logical_volumes_positive',
@@ -294,36 +293,6 @@ class TestCases(object): 
        lvs = self.c.bdev_lvol_get_lvstores(lvs_name)[0]

        return int(int(lvs['cluster_size']) / MEGABYTE)



    @case_message

    def test_case250(self):

        """

        bdev_lvol_delete_lvstore_positive



        Positive test for destroying a logical volume store.

        Call bdev_lvol_delete_lvstore with correct logical_volumes name

        """

        # Construct malloc bdev

        base_name = self.c.bdev_malloc_create(self.total_size,

                                              self.block_size)

        # Create lvol store on created malloc bdev

        uuid_store = self.c.bdev_lvol_create_lvstore(base_name,

                                                     self.lvs_name)

        # check correct uuid values in response bdev_lvol_get_lvstores command

        fail_count = self.c.check_bdev_lvol_get_lvstores(base_name, uuid_store,

                                                         self.cluster_size)

        # Destroy lvol store

        self.c.bdev_lvol_delete_lvstore(uuid_store)

        # Check correct response bdev_lvol_get_lvstores command

        if self.c.check_bdev_lvol_get_lvstores("", "", "") == 1:

            fail_count += 1

        self.c.bdev_malloc_delete(base_name)



        # Expected result:

        # - calls successful, return code = 0

        # - bdev_lvol_get_lvstores: response should be of no value after destroyed lvol store

        # - no other operation fails

        return fail_count



    @case_message

    def test_case251(self):

        """