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

[David Marchand <david.marchand@redhat.com>]

On Thu, 4 Dec 2025 at 19:22, Bruce Richardson
<bruce.richardson@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.


-- 
David Marchand