Commit 02e7c3ff authored by Paul Luse's avatar Paul Luse Committed by Daniel Verkamp
Browse files

ut/nvme: add coverage for nvme_free_request() 



Change-Id: I7be51b1eae787e7f87c7bc95633a5f58005c1cce
Signed-off-by: default avatarPaul Luse <paul.e.luse@intel.com>
Reviewed-on: https://review.gerrithub.io/372538


Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
parent f671be94

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

+21 −0
Original line number Diff line number Diff line
@@ -108,6 +108,25 @@ memset_trid(struct spdk_nvme_transport_id *trid1, struct spdk_nvme_transport_id 
	memset(trid2, 0, sizeof(struct spdk_nvme_transport_id));

}



static void

test_nvme_free_request(void)

{

	struct nvme_request match_req;

	struct spdk_nvme_qpair qpair;

	struct nvme_request *req;



	/* put a req on the Q, take it off and compare */

	memset(&match_req.cmd, 0x5a, sizeof(struct spdk_nvme_cmd));

	match_req.qpair = &qpair;

	/* the code under tests asserts this condition */

	match_req.num_children = 0;

	STAILQ_INIT(&qpair.free_req);



	nvme_free_request(&match_req);

	req = STAILQ_FIRST(&match_req.qpair->free_req);

	CU_ASSERT(req == &match_req);

}



static void

test_nvme_allocate_request_user_copy(void)

{
@@ -522,6 +541,8 @@ int main(int argc, char **argv) 
			    test_trid_adrfam_str) == NULL ||

		CU_add_test(suite, "test_nvme_ctrlr_probe",

			    test_nvme_ctrlr_probe) == NULL ||

		CU_add_test(suite, "test_nvme_free_request",

			    test_nvme_free_request) == NULL ||

		CU_add_test(suite, "test_nvme_allocate_request_user_copy",

			    test_nvme_allocate_request_user_copy) == NULL ||

		CU_add_test(suite, "test_nvme_robust_mutex_init_shared",