[LTP] [PATCH v3 2/7] docparse: Add tests
Cyril Hrubis
chrubis@suse.cz
Fri Nov 5 11:40:52 CET 2021
Hi!
> How about
> 1) Run test as default (all: test)?
> 2) Propagate test target into docparse/Makefile
> 3) Run docparse testing as part of make test
> 4) Be more verbose about running docparse tests
>
> These are in patch below.
>
> Also part of lib/newlib_tests/runtest.sh could be put into separate file, which
> could be reused in docparse/tests/test.sh (color etc), but that should be postponed
> to later to not blocking this patchset.
>
> Kind regards,
> Petr
>
> diff --git Makefile Makefile
> index 4e37362f9..9ead7b205 100644
> --- Makefile
> +++ Makefile
> @@ -210,6 +210,7 @@ ifneq ($(build),$(host))
> $(error running tests on cross-compile build not supported)
> endif
> $(call _test)
> + $(MAKE) test-docparse
>
> test-c: lib-all
> ifneq ($(build),$(host))
> @@ -223,6 +224,9 @@ ifneq ($(build),$(host))
> endif
> $(call _test,-s)
>
> +test-docparse:
> + $(MAKE) -C docparse/tests test
That's have been moved to metadata/tests/ in the last patch of the
series.
Other than that the rest looks good. Feel free to push this on the top
of my patchset with my Reviewed-by:
--
Cyril Hrubis
chrubis@suse.cz
More information about the ltp
mailing list