Re: [PATCH 03/15] scalar: create test infrastructure
- From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
- Date
- Aug 31, 2021, 08:15 UTC
- Message-ID
- <87czpuxbwp.fsf@evledraar.gmail.com>
- In-Reply-To
- <bcfde9bc7651266deaa0e5036cebd058dffc12a7.1630359290.git.gitgitgadget@gmail.com>
On Mon, Aug 30 2021, Johannes Schindelin via GitGitGadget wrote:
Show 5 quoted lines
> To test the Scalar command, create a test script in contrib/scalar/t > that is executed as `make -C contrib/scalar test`. Since Scalar has no > meaningful capabilities yet, the only test is rather simple. We will add > more tests in subsequent commits that introduce corresponding, new > functionality.
As a comment on 01..03/15: I'd really prefer if we stop using this pattern of sub-Makefile, the dependencies are a pain to manage, and we end up copy/pasting large sets of functionality.
That would mean just adding the build of this command to the top-level Makefile behind some "CONTRIB_SCALAR" flag or whatever, but I find that much cleaner than....
Show 11 quoted lines
> @@ -21,7 +22,7 @@ include ../../config.mak.uname > TARGETS = scalar$(X) scalar.o > GITLIBS = ../../common-main.o ../../libgit.a ../../xdiff/lib.a > > -all: scalar$X > +all: scalar$X ../../bin-wrappers/scalar > > [...] > +../../bin-wrappers/scalar: ../../wrap-for-bin.sh Makefile > [...] > scalar.html: | scalar.1 # prevent them from trying to build `doc.dep` in parallel
...things like this, which refer to assets built by other Makefiles, and need to plaster over the dependency issues...
Show 80 quoted lines
> +++ b/contrib/scalar/t/Makefile
> @@ -0,0 +1,78 @@
> +# Run scalar tests
> +#
> +# Copyright (c) 2005,2021 Junio C Hamano, Johannes Schindelin
> +#
> +
> +-include ../../../config.mak.autogen
> +-include ../../../config.mak
> +
> +SHELL_PATH ?= $(SHELL)
> +PERL_PATH ?= /usr/bin/perl
> +RM ?= rm -f
> +PROVE ?= prove
> +DEFAULT_TEST_TARGET ?= test
> +TEST_LINT ?= test-lint
> +
> +ifdef TEST_OUTPUT_DIRECTORY
> +TEST_RESULTS_DIRECTORY = $(TEST_OUTPUT_DIRECTORY)/test-results
> +else
> +TEST_RESULTS_DIRECTORY = ../../../t/test-results
> +endif
> +
> +# Shell quote;
> +SHELL_PATH_SQ = $(subst ','\'',$(SHELL_PATH))
> +PERL_PATH_SQ = $(subst ','\'',$(PERL_PATH))
> +TEST_RESULTS_DIRECTORY_SQ = $(subst ','\'',$(TEST_RESULTS_DIRECTORY))
> +
> +T = $(sort $(wildcard t[0-9][0-9][0-9][0-9]-*.sh))
> +
> +all: $(DEFAULT_TEST_TARGET)
> +
> +test: $(TEST_LINT)
> + $(MAKE) aggregate-results-and-cleanup
> +
> +prove: $(TEST_LINT)
> + @echo "*** prove ***"; GIT_CONFIG=.git/config $(PROVE) --exec '$(SHELL_PATH_SQ)' $(GIT_PROVE_OPTS) $(T) :: $(GIT_TEST_OPTS)
> + $(MAKE) clean-except-prove-cache
> +
> +$(T):
> + @echo "*** $@ ***"; GIT_CONFIG=.git/config '$(SHELL_PATH_SQ)' $@ $(GIT_TEST_OPTS)
> +
> +clean-except-prove-cache:
> + $(RM) -r 'trash directory'.* '$(TEST_RESULTS_DIRECTORY_SQ)'
> + $(RM) -r valgrind/bin
> +
> +clean: clean-except-prove-cache
> + $(RM) .prove
> +
> +test-lint: test-lint-duplicates test-lint-executable test-lint-shell-syntax
> +
> +test-lint-duplicates:
> + @dups=`echo $(T) | tr ' ' '\n' | sed 's/-.*//' | sort | uniq -d` && \
> + test -z "$$dups" || { \
> + echo >&2 "duplicate test numbers:" $$dups; exit 1; }
> +
> +test-lint-executable:
> + @bad=`for i in $(T); do test -x "$$i" || echo $$i; done` && \
> + test -z "$$bad" || { \
> + echo >&2 "non-executable tests:" $$bad; exit 1; }
> +
> +test-lint-shell-syntax:
> + @'$(PERL_PATH_SQ)' ../../../t/check-non-portable-shell.pl $(T)
> +
> +aggregate-results-and-cleanup: $(T)
> + $(MAKE) aggregate-results
> + $(MAKE) clean
> +
> +aggregate-results:
> + for f in '$(TEST_RESULTS_DIRECTORY_SQ)'/t*-*.counts; do \
> + echo "$$f"; \
> + done | '$(SHELL_PATH_SQ)' ../../../t/aggregate-results.sh
> +
> +valgrind:
> + $(MAKE) GIT_TEST_OPTS="$(GIT_TEST_OPTS) --valgrind"
> +
> +test-results:
> + mkdir -p test-results
> +
> +.PHONY: $(T) aggregate-results clean valgrind...and this entire copy/pasting & adjusting of t/Makefile.