On Wed, Dec 08, 2021 at 04:57:52PM +0000, Bruce Richardson wrote:
> On Tue, Dec 07, 2021 at 05:51:02PM -0800, Jie Zhou wrote:
> > Enable a subset of unit tests for Windows CI
> >
> > - For driver tests, driver owners should enable corresponding tests when
> > enabling driver for Windows. For example, the cryptodev tests will be
> > enabled by "patch-18949: app/test: enable crypto unit tests on Windows"
> > (which depends on this patchset to be merged).
> > - For dump tests, currently the tests hang on Windows which require
> > further investigation.
> >
> > Signed-off-by: Jie Zhou <j...@linux.microsoft.com>
> >
>
> Mostly these changes look ok to me. One small query below.
>
> /Bruce
> > ---
> > app/test/meson.build | 111 ++++++++++++++++++++++---------------------
> > 1 file changed, 58 insertions(+), 53 deletions(-)
> >
> > diff --git a/app/test/meson.build b/app/test/meson.build
> > index 97ee83029e..fcf38729e7 100644
> > --- a/app/test/meson.build
> > +++ b/app/test/meson.build
> > @@ -1,12 +1,6 @@
> > # SPDX-License-Identifier: BSD-3-Clause
> > # Copyright(c) 2017 Intel Corporation
> >
> > -if is_windows
> > - build = false
>
> <snip>
>
> > # The following linkages are an exception to allow running the
> > # unit tests without requiring that the developer install the
> > # DPDK libraries. Explicit linkage of drivers (plugin libraries)
> > @@ -385,7 +390,7 @@ if dpdk_conf.has('RTE_LIB_METRICS')
> > test_sources += ['test_metrics.c']
> > fast_tests += [['metrics_autotest', true]]
> > endif
> > -if dpdk_conf.has('RTE_LIB_TELEMETRY')
> > +if not is_windows and dpdk_conf.has('RTE_LIB_TELEMETRY')
> > test_sources += ['test_telemetry_json.c', 'test_telemetry_data.c']
> > fast_tests += [['telemetry_json_autotest', true],
> > ['telemetry_data_autotest', true]]
>
> If telemetry is enabled, is there a reason these won't run on Windows?
Hi Bruce, the telemetry test has many POSIX socket specific codes which
requires replacement for Windows. We will work on investigation of enabling
more tests (including the telemetry tests) after this patchset. Sorry I forgot
to mention that in the commit message, but it is on my list. The current goal
is to get this patchset merged asap before another upstream churn to rebase,
rework, etc. If you prefer me to send out a V14 to add this info into commit
message or remove this change from meson.build but add test stub into the
specific test files, please just let me know. Thanks for your understanding. --
Jie
