[LTP] [PATCH v3 2/7] docparse: Add tests
Petr Vorel
pvorel@suse.cz
Thu Nov 4 17:06:18 CET 2021
Hi Cyril,
Reviewed-by: Petr Vorel <pvorel@suse.cz>
> diff --git a/docparse/tests/Makefile b/docparse/tests/Makefile
> new file mode 100644
> index 000000000..b5c8c4668
> --- /dev/null
> +++ b/docparse/tests/Makefile
> @@ -0,0 +1,4 @@
> +all:
> +
> +test:
> + @./test.sh
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
+
## Help
.PHONY: help
help:
diff --git docparse/Makefile docparse/Makefile
index e2defad38..7764d1dc1 100644
--- docparse/Makefile
+++ docparse/Makefile
@@ -74,4 +74,9 @@ metadata.pdf: txt
$(METADATA_GENERATOR_CMD) $(METADATA_GENERATOR_PARAMS) $(METADATA_GENERATOR_PARAMS_PDF)
endif
+.PHONY: test
+
+test: metadata.json
+ $(MAKE) -C tests $@
+
include $(top_srcdir)/include/mk/generic_leaf_target.mk
diff --git docparse/tests/Makefile docparse/tests/Makefile
index b5c8c4668..69131ffad 100644
--- docparse/tests/Makefile
+++ docparse/tests/Makefile
@@ -1,4 +1,5 @@
-all:
+all: test
test:
+ $(info Testing docparse)
@./test.sh
More information about the ltp
mailing list