From 8dc056bc71e788d8423b260b2f835e89cc5cd49f Mon Sep 17 00:00:00 2001 From: paul luse Date: Tue, 9 Feb 2021 19:00:30 -0500 Subject: [PATCH] test/accel: add UT for _is_batch_valid() Signed-off-by: paul luse Change-Id: Ifd9197c6a9941b3643c902cbf2f1ca670b455bd5 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/6357 Reviewed-by: Jim Harris Reviewed-by: Shuhei Matsumoto Tested-by: SPDK CI Jenkins Community-CI: Mellanox Build Bot --- test/unit/lib/accel/accel.c/accel_engine_ut.c | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/test/unit/lib/accel/accel.c/accel_engine_ut.c b/test/unit/lib/accel/accel.c/accel_engine_ut.c index 3c0f16884..6b7ce6cb3 100644 --- a/test/unit/lib/accel/accel.c/accel_engine_ut.c +++ b/test/unit/lib/accel/accel.c/accel_engine_ut.c @@ -196,6 +196,24 @@ test_spdk_accel_get_capabilities(void) free(ch); } +static void +test_is_batch_valid(void) +{ + struct spdk_accel_batch batch = {}; + struct accel_io_channel accel_ch = {}; + bool rc; + + /* This batch doesn't go with this channel. */ + batch.accel_ch = (struct accel_io_channel *)0xDEADBEEF; + rc = _is_batch_valid(&batch, &accel_ch); + CU_ASSERT(rc == false); + + /* This one does. */ + batch.accel_ch = &accel_ch; + rc = _is_batch_valid(&batch, &accel_ch); + CU_ASSERT(rc == true); +} + int main(int argc, char **argv) { CU_pSuite suite = NULL; @@ -212,6 +230,7 @@ int main(int argc, char **argv) CU_ADD_TEST(suite, test_is_supported); CU_ADD_TEST(suite, test_spdk_accel_task_complete); CU_ADD_TEST(suite, test_spdk_accel_get_capabilities); + CU_ADD_TEST(suite, test_is_batch_valid); CU_basic_set_mode(CU_BRM_VERBOSE); CU_basic_run_tests();