Renaming of things which may be, one day, exported in a lib.
This renaming is important, as it creates consistency between test
symbols, which is needed if things get eventually exported in the lib.
Also, tests are often created from other tests: Fixing the first exemples
will help geting future tests better.
Things that are candidate to be exported in the lib in the future
have been named as follows:
-Tests, i.e. functions which are used in CUNIT testsuites are named:
<Module>_test_*
-Test arrays, i.e. arrays of CU_TestInfo, listing the test functions
belonging to a suite, are called:
<Module>_suite[_*]
where the possible suffix can be used if many suites are declared.
-CUNIT suite init and termination functions are called:
<Module>_suite[_*]_init() and <Module>_suite[_*]_term()
respectively.
-Suite arrays, i.e. arrays of CU_SuiteInfo used in executables are called:
<Module>_suites[_*]
where the possible suffix identifies the executable using it, if many.
-Main function(s), are called:
<Module>_main[_*]
where the possible suffix identifies the executable using it
Signed-off-by: Christophe Milard <christophe.mil...@linaro.org>
---
test/validation/odp_pktio.c | 68 ++++++++++++++++++++++-----------------------
1 file changed, 34 insertions(+), 34 deletions(-)
diff --git a/test/validation/odp_pktio.c b/test/validation/odp_pktio.c
index 592136f..3ac5df9 100644
--- a/test/validation/odp_pktio.c
+++ b/test/validation/odp_pktio.c
@@ -404,7 +404,7 @@ static void pktio_txrx_multi(pktio_info_t *pktio_a,
pktio_info_t *pktio_b,
CU_ASSERT(i == num_pkts);
}
-static void pktio_test_txrx(odp_queue_type_t q_type, int num_pkts)
+static void test_txrx(odp_queue_type_t q_type, int num_pkts)
{
int ret, i, if_b;
pktio_info_t pktios[MAX_NUM_IFACES];
@@ -440,34 +440,34 @@ static void pktio_test_txrx(odp_queue_type_t q_type, int
num_pkts)
}
}
-static void test_odp_pktio_poll_queue(void)
+static void pktio_test_poll_queue(void)
{
- pktio_test_txrx(ODP_QUEUE_TYPE_POLL, 1);
+ test_txrx(ODP_QUEUE_TYPE_POLL, 1);
}
-static void test_odp_pktio_poll_multi(void)
+static void pktio_test_poll_multi(void)
{
- pktio_test_txrx(ODP_QUEUE_TYPE_POLL, 4);
+ test_txrx(ODP_QUEUE_TYPE_POLL, 4);
}
-static void test_odp_pktio_sched_queue(void)
+static void pktio_test_sched_queue(void)
{
- pktio_test_txrx(ODP_QUEUE_TYPE_SCHED, 1);
+ test_txrx(ODP_QUEUE_TYPE_SCHED, 1);
}
-static void test_odp_pktio_sched_multi(void)
+static void pktio_test_sched_multi(void)
{
- pktio_test_txrx(ODP_QUEUE_TYPE_SCHED, 4);
+ test_txrx(ODP_QUEUE_TYPE_SCHED, 4);
}
-static void test_odp_pktio_jumbo(void)
+static void pktio_test_jumbo(void)
{
packet_len = PKT_LEN_JUMBO;
- test_odp_pktio_sched_multi();
+ pktio_test_sched_multi();
packet_len = PKT_LEN_NORMAL;
}
-static void test_odp_pktio_mtu(void)
+static void pktio_test_mtu(void)
{
int ret;
int mtu;
@@ -484,7 +484,7 @@ static void test_odp_pktio_mtu(void)
return;
}
-static void test_odp_pktio_promisc(void)
+static void pktio_test_promisc(void)
{
int ret;
odp_pktio_t pktio = create_pktio(iface_name[0], 0);
@@ -509,7 +509,7 @@ static void test_odp_pktio_promisc(void)
return;
}
-static void test_odp_pktio_mac(void)
+static void pktio_test_mac(void)
{
unsigned char mac_addr[ODPH_ETHADDR_LEN];
int mac_len;
@@ -535,7 +535,7 @@ static void test_odp_pktio_mac(void)
return;
}
-static void test_odp_pktio_inq_remdef(void)
+static void pktio_test_inq_remdef(void)
{
odp_pktio_t pktio = create_pktio(iface_name[0], 0);
odp_queue_t inq;
@@ -559,7 +559,7 @@ static void test_odp_pktio_inq_remdef(void)
CU_ASSERT(odp_pktio_close(pktio) == 0);
}
-static void test_odp_pktio_open(void)
+static void pktio_test_open(void)
{
odp_pktio_t pktio;
int i;
@@ -575,7 +575,7 @@ static void test_odp_pktio_open(void)
CU_ASSERT(pktio == ODP_PKTIO_INVALID);
}
-static void test_odp_pktio_lookup(void)
+static void pktio_test_lookup(void)
{
odp_pktio_t pktio, pktio_inval;
@@ -593,7 +593,7 @@ static void test_odp_pktio_lookup(void)
CU_ASSERT(odp_pktio_lookup(iface_name[0]) == ODP_PKTIO_INVALID);
}
-static void test_odp_pktio_inq(void)
+static void pktio_test_inq(void)
{
odp_pktio_t pktio;
@@ -627,7 +627,7 @@ static int create_pool(const char *iface, int num)
return 0;
}
-static int init_pktio_suite(void)
+static int pktio_suite_init(void)
{
odp_atomic_init_u32(&ip_seq, 0);
iface_name[0] = getenv("ODP_PKTIO_IF0");
@@ -659,7 +659,7 @@ static int init_pktio_suite(void)
return 0;
}
-static int term_pktio_suite(void)
+static int pktio_suite_term(void)
{
char pool_name[ODP_POOL_NAME_LEN];
odp_pool_t pool;
@@ -688,25 +688,25 @@ static int term_pktio_suite(void)
return ret;
}
-static CU_TestInfo pktio_tests[] = {
- {"pktio open", test_odp_pktio_open},
- {"pktio lookup", test_odp_pktio_lookup},
- {"pktio inq", test_odp_pktio_inq},
- {"pktio poll queues", test_odp_pktio_poll_queue},
- {"pktio poll multi", test_odp_pktio_poll_multi},
- {"pktio sched queues", test_odp_pktio_sched_queue},
- {"pktio sched multi", test_odp_pktio_sched_multi},
- {"pktio jumbo frames", test_odp_pktio_jumbo},
- {"pktio mtu", test_odp_pktio_mtu},
- {"pktio promisc mode", test_odp_pktio_promisc},
- {"pktio mac", test_odp_pktio_mac},
- {"pktio inq_remdef", test_odp_pktio_inq_remdef},
+static CU_TestInfo pktio_suite[] = {
+ {"pktio open", pktio_test_open},
+ {"pktio lookup", pktio_test_lookup},
+ {"pktio inq", pktio_test_inq},
+ {"pktio poll queues", pktio_test_poll_queue},
+ {"pktio poll multi", pktio_test_poll_multi},
+ {"pktio sched queues", pktio_test_sched_queue},
+ {"pktio sched multi", pktio_test_sched_multi},
+ {"pktio jumbo frames", pktio_test_jumbo},
+ {"pktio mtu", pktio_test_mtu},
+ {"pktio promisc mode", pktio_test_promisc},
+ {"pktio mac", pktio_test_mac},
+ {"pktio inq_remdef", pktio_test_inq_remdef},
CU_TEST_INFO_NULL
};
static CU_SuiteInfo pktio_suites[] = {
{"Packet I/O",
- init_pktio_suite, term_pktio_suite, NULL, NULL, pktio_tests},
+ pktio_suite_init, pktio_suite_term, NULL, NULL, pktio_suite},
CU_SUITE_INFO_NULL
};
--
1.9.1
_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
