diff options
author | Petri Savolainen <petri.savolainen@nokia.com> | 2021-02-24 17:41:57 +0200 |
---|---|---|
committer | Petri Savolainen <petri.savolainen@nokia.com> | 2021-03-09 17:37:09 +0200 |
commit | 802a735c97a3058af4800fb4eea2e1ebb18d605b (patch) | |
tree | 270b7997b0954d8b91cca6954b3b083fa2f832c9 /test/validation/api/queue | |
parent | a12cdac820b1a4633d1c24e4c3d3e3a4f24cdcce (diff) |
validation: queue: fatal failure when create fails
Use CU_ASSERT_FATAL when queue create fails. Test should not
call APIs with an invalid handle.
Signed-off-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
Tested-by: Shijith Thotton <sthotton@marvell.com>
Diffstat (limited to 'test/validation/api/queue')
-rw-r--r-- | test/validation/api/queue/queue.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/test/validation/api/queue/queue.c b/test/validation/api/queue/queue.c index b5d594a9a..fa2cd8549 100644 --- a/test/validation/api/queue/queue.c +++ b/test/validation/api/queue/queue.c @@ -1,4 +1,5 @@ /* Copyright (c) 2014-2018, Linaro Limited + * Copyright (c) 2021, Nokia * All rights reserved. * * SPDX-License-Identifier: BSD-3-Clause @@ -612,7 +613,7 @@ static void queue_test_param(void) qparams.sched.group = ODP_SCHED_GROUP_WORKER; queue = odp_queue_create("test_queue", &qparams); - CU_ASSERT(ODP_QUEUE_INVALID != queue); + CU_ASSERT_FATAL(ODP_QUEUE_INVALID != queue); CU_ASSERT(odp_queue_to_u64(queue) != odp_queue_to_u64(ODP_QUEUE_INVALID)); CU_ASSERT(queue == odp_queue_lookup("test_queue")); @@ -631,7 +632,7 @@ static void queue_test_param(void) /* Create queue with no name */ odp_queue_param_init(&qparams); null_queue = odp_queue_create(NULL, &qparams); - CU_ASSERT(ODP_QUEUE_INVALID != null_queue); + CU_ASSERT_FATAL(ODP_QUEUE_INVALID != null_queue); CU_ASSERT(odp_queue_context(null_queue) == NULL); /* Plain type queue */ @@ -641,7 +642,7 @@ static void queue_test_param(void) qparams.context_len = sizeof(queue_context); queue = odp_queue_create("test_queue", &qparams); - CU_ASSERT(ODP_QUEUE_INVALID != queue); + CU_ASSERT_FATAL(ODP_QUEUE_INVALID != queue); CU_ASSERT(queue == odp_queue_lookup("test_queue")); CU_ASSERT(ODP_QUEUE_TYPE_PLAIN == odp_queue_type(queue)); CU_ASSERT(&queue_context == odp_queue_context(queue)); @@ -712,7 +713,7 @@ static void queue_test_info(void) /* Create a plain queue and set context */ q_plain = odp_queue_create(nq_plain, NULL); - CU_ASSERT(ODP_QUEUE_INVALID != q_plain); + CU_ASSERT_FATAL(ODP_QUEUE_INVALID != q_plain); CU_ASSERT(odp_queue_context_set(q_plain, q_plain_ctx, sizeof(q_plain_ctx)) == 0); @@ -730,7 +731,7 @@ static void queue_test_info(void) printf("\n Ordered locks NOT supported\n"); param.context = q_order_ctx; q_order = odp_queue_create(nq_order, ¶m); - CU_ASSERT(ODP_QUEUE_INVALID != q_order); + CU_ASSERT_FATAL(ODP_QUEUE_INVALID != q_order); /* Check info and call print for a plain queue */ CU_ASSERT(odp_queue_info(q_plain, &info) == 0); |