Commit c29329fe authored by Darek Stojaczyk's avatar Darek Stojaczyk Committed by Tomasz Zawadzki
Browse files

test/common: add NOT() utility function 



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


Community-CI: Mellanox Build Bot
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
parent 52330a6b

test/common/autotest_common.sh

+7 −0
Original line number Diff line number Diff line
@@ -378,6 +378,13 @@ function rpc_cmd_simple_data_json() { 
	(( ${#jq_out[@]} > 0 )) || return 1

}



# invert error code of any command and also trigger ERR on 0 (unlike bash ! prefix)

function NOT() {

	if "$@"; then

		return 1

	fi

}



function timing() {

	direction="$1"

	testname="$2"

test/lvol/basic.sh

+3 −3
Original line number Diff line number Diff line
@@ -17,7 +17,7 @@ function test_construct_lvs() { 


	# try to destroy inexistent lvs, this should obviously fail

	dummy_uuid="00000000-0000-0000-0000-000000000000"

	rpc_cmd bdev_lvol_delete_lvstore -u "$dummy_uuid" && false

	NOT rpc_cmd bdev_lvol_delete_lvstore -u "$dummy_uuid"

	# our lvs should not be impacted

	rpc_cmd bdev_lvol_get_lvstores -u "$lvs_uuid"
@@ -35,9 +35,9 @@ function test_construct_lvs() { 


	# remove the lvs and verify it's gone

	rpc_cmd bdev_lvol_delete_lvstore -u "$lvs_uuid"

	rpc_cmd bdev_lvol_get_lvstores -u "$lvs_uuid" && false

	NOT rpc_cmd bdev_lvol_get_lvstores -u "$lvs_uuid"

	# make sure we can't delete the same lvs again

	rpc_cmd bdev_lvol_delete_lvstore -u "$lvs_uuid" && false

	NOT rpc_cmd bdev_lvol_delete_lvstore -u "$lvs_uuid"



	rpc_cmd bdev_malloc_delete "$malloc_name"

	check_leftover_devices