From patchwork Mon Jul 16 16:58:39 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Pattan, Reshma" X-Patchwork-Id: 43120 X-Patchwork-Delegate: thomas@monjalon.net Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 1ACBD255; Mon, 16 Jul 2018 18:58:48 +0200 (CEST) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id 2D590200; Mon, 16 Jul 2018 18:58:45 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 16 Jul 2018 09:58:44 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.51,362,1526367600"; d="scan'208";a="54763897" Received: from sivswdev02.ir.intel.com (HELO localhost.localdomain) ([10.237.217.46]) by fmsmga007.fm.intel.com with ESMTP; 16 Jul 2018 09:58:42 -0700 From: Reshma Pattan To: dev@dpdk.org Cc: jasvinder.singh@intel.com, bernard.iremonger@intel.com, stable@dpdk.org, Reshma Pattan Date: Mon, 16 Jul 2018 17:58:39 +0100 Message-Id: <1531760319-2996-1-git-send-email-reshma.pattan@intel.com> X-Mailer: git-send-email 1.7.0.7 Subject: [dpdk-dev] [PATCH] test: fix incorrect return types X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" UTs should return either TEST_SUCCESS or TEST_FAILED only. They should not return 0, -1 and any other value. Fixes: 9c9befea4f ("test: add flow classify unit tests") CC: jasvinder.singh@intel.com CC: bernard.iremonger@intel.com CC: stable@dpdk.org Signed-off-by: Reshma Pattan Reviewed-by: Anatoly Burakov --- test/test/test_flow_classify.c | 110 ++++++++++++++++++++--------------------- 1 file changed, 55 insertions(+), 55 deletions(-) diff --git a/test/test/test_flow_classify.c b/test/test/test_flow_classify.c index fc83b69ae..2ff1ca831 100644 --- a/test/test/test_flow_classify.c +++ b/test/test/test_flow_classify.c @@ -231,7 +231,7 @@ test_invalid_parameters(void) printf("Line %i: rte_flow_classify_validate", __LINE__); printf(" with NULL param should have failed!\n"); - return -1; + return TEST_FAILED; } rule = rte_flow_classify_table_entry_add(NULL, NULL, NULL, NULL, @@ -239,7 +239,7 @@ test_invalid_parameters(void) if (rule) { printf("Line %i: flow_classifier_table_entry_add", __LINE__); printf(" with NULL param should have failed!\n"); - return -1; + return TEST_FAILED; } ret = rte_flow_classify_table_entry_delete(NULL, NULL); @@ -247,14 +247,14 @@ test_invalid_parameters(void) printf("Line %i: rte_flow_classify_table_entry_delete", __LINE__); printf(" with NULL param should have failed!\n"); - return -1; + return TEST_FAILED; } ret = rte_flow_classifier_query(NULL, NULL, 0, NULL, NULL); if (!ret) { printf("Line %i: flow_classifier_query", __LINE__); printf(" with NULL param should have failed!\n"); - return -1; + return TEST_FAILED; } rule = rte_flow_classify_table_entry_add(NULL, NULL, NULL, NULL, @@ -262,7 +262,7 @@ test_invalid_parameters(void) if (rule) { printf("Line %i: flow_classify_table_entry_add ", __LINE__); printf("with NULL param should have failed!\n"); - return -1; + return TEST_FAILED; } ret = rte_flow_classify_table_entry_delete(NULL, NULL); @@ -270,16 +270,16 @@ test_invalid_parameters(void) printf("Line %i: rte_flow_classify_table_entry_delete", __LINE__); printf("with NULL param should have failed!\n"); - return -1; + return TEST_FAILED; } ret = rte_flow_classifier_query(NULL, NULL, 0, NULL, NULL); if (!ret) { printf("Line %i: flow_classifier_query", __LINE__); printf(" with NULL param should have failed!\n"); - return -1; + return TEST_FAILED; } - return 0; + return TEST_SUCCESS; } static int @@ -310,7 +310,7 @@ test_valid_parameters(void) printf("Line %i: rte_flow_classify_validate", __LINE__); printf(" should not have failed!\n"); - return -1; + return TEST_FAILED; } rule = rte_flow_classify_table_entry_add(cls->cls, &attr, pattern, actions, &key_found, &error); @@ -318,7 +318,7 @@ test_valid_parameters(void) if (!rule) { printf("Line %i: flow_classify_table_entry_add", __LINE__); printf(" should not have failed!\n"); - return -1; + return TEST_FAILED; } ret = rte_flow_classify_table_entry_delete(cls->cls, rule); @@ -326,9 +326,9 @@ test_valid_parameters(void) printf("Line %i: rte_flow_classify_table_entry_delete", __LINE__); printf(" should not have failed!\n"); - return -1; + return TEST_FAILED; } - return 0; + return TEST_SUCCESS; } static int @@ -361,7 +361,7 @@ test_invalid_patterns(void) if (!ret) { printf("Line %i: rte_flow_classify_validate", __LINE__); printf(" should have failed!\n"); - return -1; + return TEST_FAILED; } rule = rte_flow_classify_table_entry_add(cls->cls, &attr, pattern, @@ -369,7 +369,7 @@ test_invalid_patterns(void) if (rule) { printf("Line %i: flow_classify_table_entry_add", __LINE__); printf(" should have failed!\n"); - return -1; + return TEST_FAILED; } ret = rte_flow_classify_table_entry_delete(cls->cls, rule); @@ -377,7 +377,7 @@ test_invalid_patterns(void) printf("Line %i: rte_flow_classify_table_entry_delete", __LINE__); printf(" should have failed!\n"); - return -1; + return TEST_FAILED; } pattern[1] = ipv4_udp_item_1; @@ -389,7 +389,7 @@ test_invalid_patterns(void) if (!ret) { printf("Line %i: rte_flow_classify_validate", __LINE__); printf(" should have failed!\n"); - return -1; + return TEST_FAILED; } rule = rte_flow_classify_table_entry_add(cls->cls, &attr, pattern, @@ -397,7 +397,7 @@ test_invalid_patterns(void) if (rule) { printf("Line %i: flow_classify_table_entry_add", __LINE__); printf(" should have failed!\n"); - return -1; + return TEST_FAILED; } ret = rte_flow_classify_table_entry_delete(cls->cls, rule); @@ -405,9 +405,9 @@ test_invalid_patterns(void) printf("Line %i: rte_flow_classify_table_entry_delete", __LINE__); printf(" should have failed!\n"); - return -1; + return TEST_FAILED; } - return 0; + return TEST_SUCCESS; } static int @@ -437,7 +437,7 @@ test_invalid_actions(void) if (!ret) { printf("Line %i: rte_flow_classify_validate", __LINE__); printf(" should have failed!\n"); - return -1; + return TEST_FAILED; } rule = rte_flow_classify_table_entry_add(cls->cls, &attr, pattern, @@ -445,7 +445,7 @@ test_invalid_actions(void) if (rule) { printf("Line %i: flow_classify_table_entry_add", __LINE__); printf(" should have failed!\n"); - return -1; + return TEST_FAILED; } ret = rte_flow_classify_table_entry_delete(cls->cls, rule); @@ -453,7 +453,7 @@ test_invalid_actions(void) printf("Line %i: rte_flow_classify_table_entry_delete", __LINE__); printf(" should have failed!\n"); - return -1; + return TEST_FAILED; } actions[0] = count_action; @@ -464,7 +464,7 @@ test_invalid_actions(void) if (!ret) { printf("Line %i: rte_flow_classify_validate", __LINE__); printf(" should have failed!\n"); - return -1; + return TEST_FAILED; } rule = rte_flow_classify_table_entry_add(cls->cls, &attr, pattern, @@ -472,7 +472,7 @@ test_invalid_actions(void) if (rule) { printf("Line %i: flow_classify_table_entry_add", __LINE__); printf(" should have failed!\n"); - return -1; + return TEST_FAILED; } ret = rte_flow_classify_table_entry_delete(cls->cls, rule); @@ -480,9 +480,9 @@ test_invalid_actions(void) printf("Line %i: rte_flow_classify_table_entry_delete", __LINE__); printf("should have failed!\n"); - return -1; + return TEST_FAILED; } - return 0; + return TEST_SUCCESS; } static int @@ -628,7 +628,7 @@ init_mbufpool(void) } else { printf("Cannot init mbuf pool on socket %d\n", socketid); - ret = -ENOMEM; + ret = TEST_FAILED; break; } } @@ -649,7 +649,7 @@ test_query_udp(void) if (ret != MAX_PKT_BURST) { printf("Line %i: init_udp_ipv4_traffic has failed!\n", __LINE__); - return -1; + return TEST_FAILED; } for (i = 0; i < MAX_PKT_BURST; i++) @@ -675,7 +675,7 @@ test_query_udp(void) if (ret) { printf("Line %i: rte_flow_classify_validate", __LINE__); printf(" should not have failed!\n"); - return -1; + return TEST_FAILED; } rule = rte_flow_classify_table_entry_add(cls->cls, &attr, pattern, @@ -683,7 +683,7 @@ test_query_udp(void) if (!rule) { printf("Line %i: flow_classify_table_entry_add", __LINE__); printf(" should not have failed!\n"); - return -1; + return TEST_FAILED; } ret = rte_flow_classifier_query(cls->cls, bufs, MAX_PKT_BURST, @@ -691,7 +691,7 @@ test_query_udp(void) if (ret) { printf("Line %i: flow_classifier_query", __LINE__); printf(" should not have failed!\n"); - return -1; + return TEST_FAILED; } ret = rte_flow_classify_table_entry_delete(cls->cls, rule); @@ -699,9 +699,9 @@ test_query_udp(void) printf("Line %i: rte_flow_classify_table_entry_delete", __LINE__); printf(" should not have failed!\n"); - return -1; + return TEST_FAILED; } - return 0; + return TEST_SUCCESS; } static int @@ -716,7 +716,7 @@ test_query_tcp(void) if (ret != MAX_PKT_BURST) { printf("Line %i: init_ipv4_tcp_traffic has failed!\n", __LINE__); - return -1; + return TEST_FAILED; } for (i = 0; i < MAX_PKT_BURST; i++) @@ -742,7 +742,7 @@ test_query_tcp(void) if (ret) { printf("Line %i: flow_classifier_query", __LINE__); printf(" should not have failed!\n"); - return -1; + return TEST_FAILED; } rule = rte_flow_classify_table_entry_add(cls->cls, &attr, pattern, @@ -750,7 +750,7 @@ test_query_tcp(void) if (!rule) { printf("Line %i: flow_classify_table_entry_add", __LINE__); printf(" should not have failed!\n"); - return -1; + return TEST_FAILED; } ret = rte_flow_classifier_query(cls->cls, bufs, MAX_PKT_BURST, @@ -758,7 +758,7 @@ test_query_tcp(void) if (ret) { printf("Line %i: flow_classifier_query", __LINE__); printf(" should not have failed!\n"); - return -1; + return TEST_FAILED; } ret = rte_flow_classify_table_entry_delete(cls->cls, rule); @@ -766,9 +766,9 @@ test_query_tcp(void) printf("Line %i: rte_flow_classify_table_entry_delete", __LINE__); printf(" should not have failed!\n"); - return -1; + return TEST_FAILED; } - return 0; + return TEST_SUCCESS; } static int @@ -783,7 +783,7 @@ test_query_sctp(void) if (ret != MAX_PKT_BURST) { printf("Line %i: init_ipv4_tcp_traffic has failed!\n", __LINE__); - return -1; + return TEST_FAILED; } for (i = 0; i < MAX_PKT_BURST; i++) @@ -809,7 +809,7 @@ test_query_sctp(void) if (ret) { printf("Line %i: flow_classifier_query", __LINE__); printf(" should not have failed!\n"); - return -1; + return TEST_FAILED; } rule = rte_flow_classify_table_entry_add(cls->cls, &attr, pattern, @@ -817,7 +817,7 @@ test_query_sctp(void) if (!rule) { printf("Line %i: flow_classify_table_entry_add", __LINE__); printf(" should not have failed!\n"); - return -1; + return TEST_FAILED; } ret = rte_flow_classifier_query(cls->cls, bufs, MAX_PKT_BURST, @@ -825,7 +825,7 @@ test_query_sctp(void) if (ret) { printf("Line %i: flow_classifier_query", __LINE__); printf(" should not have failed!\n"); - return -1; + return TEST_FAILED; } ret = rte_flow_classify_table_entry_delete(cls->cls, rule); @@ -833,9 +833,9 @@ test_query_sctp(void) printf("Line %i: rte_flow_classify_table_entry_delete", __LINE__); printf(" should not have failed!\n"); - return -1; + return TEST_FAILED; } - return 0; + return TEST_SUCCESS; } static int @@ -871,32 +871,32 @@ test_flow_classify(void) printf("Line %i: f_create has failed!\n", __LINE__); rte_flow_classifier_free(cls->cls); rte_free(cls); - return -1; + return TEST_FAILED; } printf("Created table_acl for for IPv4 five tuple packets\n"); ret = init_mbufpool(); if (ret) { printf("Line %i: init_mbufpool has failed!\n", __LINE__); - return -1; + return TEST_FAILED; } if (test_invalid_parameters() < 0) - return -1; + return TEST_FAILED; if (test_valid_parameters() < 0) - return -1; + return TEST_FAILED; if (test_invalid_patterns() < 0) - return -1; + return TEST_FAILED; if (test_invalid_actions() < 0) - return -1; + return TEST_FAILED; if (test_query_udp() < 0) - return -1; + return TEST_FAILED; if (test_query_tcp() < 0) - return -1; + return TEST_FAILED; if (test_query_sctp() < 0) - return -1; + return TEST_FAILED; - return 0; + return TEST_SUCCESS; } REGISTER_TEST_COMMAND(flow_classify_autotest, test_flow_classify);