Commit 8d16797a authored by Tomasz Zawadzki's avatar Tomasz Zawadzki Committed by Jim Harris
Browse files

ut/nvme: reset aer counter for each case in UT 



Make sure to reset the counter to 0 after each
test case to make the expected state after each
easier to understand.

Will be usefull once test for 2 AERs is added.

Change-Id: Ib795ee408b1ff3e608c9a8968d2d962667ac9eef
Signed-off-by: default avatarTomasz Zawadzki <tomasz.zawadzki@nutanix.com>
Reviewed-on: https://review.spdk.io/c/spdk/spdk/+/26191


Reviewed-by: default avatarJim Harris <jim.harris@nvidia.com>
Community-CI: Mellanox Build Bot
Tested-by: default avatarSPDK Automated Test System <spdkbot@gmail.com>
Reviewed-by: default avatarKonrad Sztyber <ksztyber@nvidia.com>
parent a9780230

test/unit/lib/nvme/nvme_ctrlr.c/nvme_ctrlr_ut.c

+4 −1
Original line number Diff line number Diff line
@@ -3062,17 +3062,20 @@ test_nvme_ctrlr_ns_attr_changed(void) 
	CU_ASSERT(!spdk_nvme_ctrlr_is_active_ns(&ctrlr, 100));



	/* Add NS 101 */

	g_aer_cb_counter = 0;

	g_active_ns_list = active_ns_list3;

	g_active_ns_list_length = SPDK_COUNTOF(active_ns_list3);

	g_nvme_ns_constructed = 0;

	nvme_ctrlr_async_event_cb(&ctrlr.aer[0], &aer_cpl);

	nvme_ctrlr_complete_queued_async_events(&ctrlr);

	CU_ASSERT(g_aer_cb_counter == 2);

	CU_ASSERT(g_aer_cb_counter == 1);

	CU_ASSERT(g_nvme_ns_constructed == g_active_ns_list_length);

	check_active_ns(&ctrlr, active_ns_list3, SPDK_COUNTOF(active_ns_list3));



	g_active_ns_list = NULL;

	g_active_ns_list_length = 0;

	g_aer_cb_counter = 0;

	g_nvme_ns_constructed = 0;

	nvme_ctrlr_free_processes(&ctrlr);

	nvme_ctrlr_destruct(&ctrlr);

}