From: Thomas Monjalon <thomas@monjalon.net>
To: Chengwen Feng <fengchengwen@huawei.com>
Cc: ferruh.yigit@amd.com, david.marchand@redhat.com, dev@dpdk.org,
stephen@networkplumber.org,
"lihuisong (C)" <lihuisong@huawei.com>
Subject: Re: [PATCH v4 0/7] refactor kvargs test
Date: Tue, 19 Nov 2024 20:52:06 +0100 [thread overview]
Message-ID: <12930730.VsHLxoZxqI@thomas> (raw)
In-Reply-To: <8c2be190-71d1-20b7-a514-25b0296b38e8@huawei.com>
01/11/2024 03:26, lihuisong (C):
> For this series,
> Acked-by: Huisong Li <lihuisong@huawei.com>
> 在 2024/10/30 16:54, Chengwen Feng 写道:
> > When developing patchset [1], I found the kvargs test is hard to
> > understand when tried to add some testcase.
> >
> > So refactor kvargs by:
> > 1. introduce UT suite framework.
> > 2. extract big test_valid_kvargs() to five part.
> > 3. add new introduced rte_kvargs_process_opt() UT.
> >
> > [1] https://patchwork.dpdk.org/project/dpdk/cover/20231103073811.13196-1-fengchengwen@huawei.com/
> >
> > Chengwen Feng (7):
> > app/test: introduce UT suite framework for kvargs
> > app/test: extract basic token count testcase for kvargs
> > app/test: extract without keys testcase for kvargs
> > app/test: extract with keys testcase for kvargs
> > app/test: extract parse list value testcase for kvargs
> > app/test: extract parse empty elements testcase for kvargs
> > app/test: add process opt testcase for kvargs
Applied, thanks.
prev parent reply other threads:[~2024-11-19 19:52 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-03 9:53 [PATCH 0/7] refactor kvargs test Chengwen Feng
2023-11-03 9:53 ` [PATCH 1/7] app/test: introduce UT suite framework for kvargs Chengwen Feng
2024-10-11 2:08 ` Stephen Hemminger
2024-10-11 3:35 ` fengchengwen
2023-11-03 9:53 ` [PATCH 2/7] app/test: extract basic token count testcase " Chengwen Feng
2023-11-03 9:53 ` [PATCH 3/7] app/test: extract without keys " Chengwen Feng
2023-11-03 9:53 ` [PATCH 4/7] app/test: extract with " Chengwen Feng
2023-11-03 9:53 ` [PATCH 5/7] app/test: extract parse list value " Chengwen Feng
2023-11-03 9:53 ` [PATCH 6/7] app/test: extract parse empty elements " Chengwen Feng
2023-11-03 9:53 ` [PATCH 7/7] maintainers: update for kvargs library Chengwen Feng
2023-12-05 2:55 ` [PATCH 0/7] refactor kvargs test fengchengwen
2024-10-05 15:49 ` Stephen Hemminger
2024-10-11 3:34 ` [PATCH v2 " Chengwen Feng
2024-10-11 3:34 ` [PATCH v2 1/7] app/test: introduce UT suite framework for kvargs Chengwen Feng
2024-10-11 3:34 ` [PATCH v2 2/7] app/test: extract basic token count testcase " Chengwen Feng
2024-10-11 3:34 ` [PATCH v2 3/7] app/test: extract without keys " Chengwen Feng
2024-10-11 3:34 ` [PATCH v2 4/7] app/test: extract with " Chengwen Feng
2024-10-11 3:34 ` [PATCH v2 5/7] app/test: extract parse list value " Chengwen Feng
2024-10-11 3:34 ` [PATCH v2 6/7] app/test: extract parse empty elements " Chengwen Feng
2024-10-11 3:34 ` [PATCH v2 7/7] maintainers: update for kvargs library Chengwen Feng
2024-10-22 6:14 ` [PATCH v3 0/8] refactor kvargs test Chengwen Feng
2024-10-22 6:14 ` [PATCH v3 1/8] app/test: introduce UT suite framework for kvargs Chengwen Feng
2024-10-22 6:14 ` [PATCH v3 2/8] app/test: extract basic token count testcase " Chengwen Feng
2024-10-22 6:14 ` [PATCH v3 3/8] app/test: extract without keys " Chengwen Feng
2024-10-22 6:14 ` [PATCH v3 4/8] app/test: extract with " Chengwen Feng
2024-10-22 6:14 ` [PATCH v3 5/8] app/test: extract parse list value " Chengwen Feng
2024-10-22 6:14 ` [PATCH v3 6/8] app/test: extract parse empty elements " Chengwen Feng
2024-10-22 6:14 ` [PATCH v3 7/8] app/test: add process opt " Chengwen Feng
2024-10-22 6:14 ` [PATCH v3 8/8] maintainers: update for kvargs library Chengwen Feng
2024-10-30 8:54 ` [PATCH v4 0/7] refactor kvargs test Chengwen Feng
2024-10-30 8:54 ` [PATCH v4 1/7] app/test: introduce UT suite framework for kvargs Chengwen Feng
2024-10-30 8:54 ` [PATCH v4 2/7] app/test: extract basic token count testcase " Chengwen Feng
2024-10-30 8:54 ` [PATCH v4 3/7] app/test: extract without keys " Chengwen Feng
2024-10-30 8:54 ` [PATCH v4 4/7] app/test: extract with " Chengwen Feng
2024-10-30 8:54 ` [PATCH v4 5/7] app/test: extract parse list value " Chengwen Feng
2024-10-30 8:54 ` [PATCH v4 6/7] app/test: extract parse empty elements " Chengwen Feng
2024-10-30 8:54 ` [PATCH v4 7/7] app/test: add process opt " Chengwen Feng
2024-11-01 2:26 ` [PATCH v4 0/7] refactor kvargs test lihuisong (C)
2024-11-19 19:52 ` Thomas Monjalon [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=12930730.VsHLxoZxqI@thomas \
--to=thomas@monjalon.net \
--cc=david.marchand@redhat.com \
--cc=dev@dpdk.org \
--cc=fengchengwen@huawei.com \
--cc=ferruh.yigit@amd.com \
--cc=lihuisong@huawei.com \
--cc=stephen@networkplumber.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.