Re: [PATCH v2 08/10] app/test: add some unattached tests to fast-test suite

[Bruce Richardson]

On Fri, Dec 05, 2025 at 01:37:06PM +0100, David Marchand wrote:
> On Thu, 4 Dec 2025 at 19:22, Bruce Richardson
> <bruce.richard...@intel.com> wrote:
> > diff --git a/app/test/test_red.c b/app/test/test_red.c
> > index 7f38ed1469..4b17173999 100644
> > --- a/app/test/test_red.c
> > +++ b/app/test/test_red.c
> > @@ -1872,6 +1872,6 @@ test_red_all(void)
> >
> >  #endif /* !RTE_EXEC_ENV_WINDOWS */
> >
> > -REGISTER_TEST_COMMAND(red_autotest, test_red);
> > +REGISTER_FAST_TEST(red_autotest, NOHUGE_OK, ASAN_OK, test_red);
> >  REGISTER_PERF_TEST(red_perf, test_red_perf);
> >  REGISTER_PERF_TEST(red_all, test_red_all);
> 
> The commit 50823f30f0c8 ("test: build using per-file dependencies")
> removed some context on why some tests were not part of a testsuite.
> -# Tests known to have issues or which don't belong in other tests lists.
> -extra_test_names = [
> -        'alarm_autotest', # ee00af60170b ("test: remove strict timing
> requirements some tests")
> -        'red_autotest', # https://bugs.dpdk.org/show_bug.cgi?id=826
> -]
> 
> I would prefer we don't put an unreliable test in fast-tests.
> 
Agreed, but for many cases for developers running these on their systems,
it's probably worth having them in. What would you think about having
checks in the tests themselves to detect if they are running in github CI
and to skip themselves in that case?