[PATCH] test: conditionally test help system depending on configured support

Mark Walters markwalters1009 at gmail.com
Wed Mar 26 14:56:37 PDT 2014


LGTM +1 (and tests pass on one of my systems that does not have sphinx
or rst2man)

Best wishes

Mark

On Wed, 26 Mar 2014, Jani Nikula <jani at nikula.org> wrote:
> If neither sphinx nor rst2man is available, the notmuch man pages will
> not be available. Take this into account in the help system test.
> ---
>  test/Makefile.local    |    8 ++++++++
>  test/T010-help-test.sh |   12 ++++++++++--
>  2 files changed, 18 insertions(+), 2 deletions(-)
>
> diff --git a/test/Makefile.local b/test/Makefile.local
> index 2b18bdb..987441f 100644
> --- a/test/Makefile.local
> +++ b/test/Makefile.local
> @@ -42,10 +42,18 @@ else
>  	ln -sf /bin/false $@
>  endif
>  
> +$(dir)/have-man: Makefile.config
> +ifeq ($(HAVE_SPHINX)$(HAVE_RST2MAN),00)
> +	ln -sf /bin/false $@
> +else
> +	ln -sf /bin/true $@
> +endif
> +
>  .PHONY: test check
>  
>  TEST_BINARIES=$(dir)/arg-test \
>  	      $(dir)/have-compact \
> +	      $(dir)/have-man \
>  	      $(dir)/hex-xcode \
>  	      $(dir)/random-corpus \
>  	      $(dir)/parse-time \
> diff --git a/test/T010-help-test.sh b/test/T010-help-test.sh
> index f7df725..77410bc 100755
> --- a/test/T010-help-test.sh
> +++ b/test/T010-help-test.sh
> @@ -4,9 +4,17 @@ test_description="online help"
>  . ./test-lib.sh
>  
>  test_expect_success 'notmuch --help' 'notmuch --help'
> -test_expect_success 'notmuch --help tag' 'notmuch --help tag'
>  test_expect_success 'notmuch help' 'notmuch help'
> -test_expect_success 'notmuch help tag' 'notmuch help tag'
>  test_expect_success 'notmuch --version' 'notmuch --version'
>  
> +if ${TEST_DIRECTORY}/have-man; then
> +    test_expect_success 'notmuch --help tag' 'notmuch --help tag'
> +    test_expect_success 'notmuch help tag' 'notmuch help tag'
> +else
> +    test_expect_success 'notmuch --help tag (man pages not available)' \
> +	'test_must_fail notmuch --help tag'
> +    test_expect_success 'notmuch help tag (man pages not available)' \
> +	'test_must_fail notmuch help tag'
> +fi
> +
>  test_done
> -- 
> 1.7.2.5
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch


More information about the notmuch mailing list