Commit 04080101 authored by JinYu's avatar JinYu Committed by Jim Harris
Browse files

unit/nvme: add spdk_nvme_connect unit test case 



cover the opts_size examination

Change-Id: I8058f1db98713b5f19151b1b127180ae5ec58b96
Signed-off-by: default avatarJinYu <jin.yu@intel.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/453944


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>
parent 14777890

test/unit/lib/nvme/nvme.c/nvme_ut.c

+3 −0
Original line number Diff line number Diff line
@@ -220,6 +220,9 @@ test_spdk_nvme_connect(void) 
	ret_ctrlr = spdk_nvme_connect(&trid, &opts, sizeof(opts));

	CU_ASSERT(ret_ctrlr == &ctrlr);

	CU_ASSERT_EQUAL(ret_ctrlr->opts.num_io_queues, 1);

	/* opts_size must be sizeof(*opts) if opts != NULL */

	ret_ctrlr = spdk_nvme_connect(&trid, &opts, sizeof(opts) + 1);

	CU_ASSERT(ret_ctrlr == NULL);

	/* remove the attached ctrlr on the attached_list */

	CU_ASSERT(spdk_nvme_detach(&ctrlr) == 0);

	CU_ASSERT(TAILQ_EMPTY(&g_spdk_nvme_driver->shared_attached_ctrlrs));