Re: [dpdk-dev] [PATCH 1/2] test: store only the test case name

[Jerin Jacob <jerin.jacob@caviumnetworks.com>]

On Mon, Mar 20, 2017 at 09:58:04AM +0000, Van Haaren, Harry wrote:
> > From: Jerin Jacob [mailto:jerin.jacob@caviumnetworks.com]
> > Sent: Sunday, March 19, 2017 1:38 PM
> > To: dev@dpdk.org
> > Cc: thomas.monjalon@6wind.com; Van Haaren, Harry <harry.van.haaren@intel.com>; Jerin
> > Jacob <jerin.jacob@caviumnetworks.com>
> > Subject: [dpdk-dev] [PATCH 1/2] test: store only the test case name
> > 
> > Store only the test case name in unit test case structure.The actor who
> > renders the test status can add appropriate test status. This enables
> > adding the new test case status without storing the additional
> > information in the unit test case structure.
> > 
> > Signed-off-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
> > ---
> >  test/test/test.c | 14 ++++++--------
> >  test/test/test.h | 23 +++++++++--------------
> >  2 files changed, 15 insertions(+), 22 deletions(-)
> > 
> > diff --git a/test/test/test.c b/test/test/test.c
> 
> 
> It looks like the patch got corrupted somehow - the above line should be  app/test/test.c, not test/test/test.c
> Applying fails here, but with a s/test/app in the right places, the patches (this and next) both apply cleanly, and work well.
> This handles the event/sw timeout_ticks() unsupported function as should so,
> 
> Tested-by: Harry van Haaren <harry.van.haaren@intel.com>
> 
> 
> With the above apply issue resolved, also

Recently there was a rework in test directory. Now test/test points to old app/test
in dpdk-master.So no issues.

http://dpdk.org/browse/dpdk/tree/test/test

> 
> Acked-by: Harry van Haaren <harry.van.haaren@intel.com>