Re: [PATCH 1/2] test: add basic test for notmuch setup

Subject: Re: [PATCH 1/2] test: add basic test for notmuch setup

Date: Sun, 05 May 2013 22:27:44 +0300

To: notmuch@notmuchmail.org

Cc:

From: Tomi Ollila


On Sun, May 05 2013, Jani Nikula <jani@nikula.org> wrote:

> And annotate with test_subtest_known_broken. Hooray.
> ---

LGTM (both patches). Tests pass (both make test and hand-tested
setup without having file ~/.notmuch-config).

Tomi

>  test/notmuch-test |    1 +
>  test/setup        |   28 ++++++++++++++++++++++++++++
>  2 files changed, 29 insertions(+)
>  create mode 100755 test/setup
>
> diff --git a/test/notmuch-test b/test/notmuch-test
> index ca9c3dc..27a144e 100755
> --- a/test/notmuch-test
> +++ b/test/notmuch-test
> @@ -20,6 +20,7 @@ TESTS="
>    basic
>    help-test
>    config
> +  setup
>    new
>    count
>    search
> diff --git a/test/setup b/test/setup
> new file mode 100755
> index 0000000..8cc5576
> --- /dev/null
> +++ b/test/setup
> @@ -0,0 +1,28 @@
> +#!/usr/bin/env bash
> +
> +test_description='"notmuch setup"'
> +. ./test-lib.sh
> +
> +test_begin_subtest "Create a new config interactively"
> +test_subtest_known_broken
> +notmuch --config=new-notmuch-config > /dev/null <<EOF
> +Test Suite
> +test.suite@example.com
> +another.suite@example.com
> +
> +/path/to/maildir
> +foo bar
> +baz
> +EOF
> +output=$(notmuch --config=new-notmuch-config config list)
> +test_expect_equal "$output" "\
> +database.path=/path/to/maildir
> +user.name=Test Suite
> +user.primary_email=test.suite@example.com
> +user.other_email=another.suite@example.com;
> +new.tags=foo;bar;
> +new.ignore=
> +search.exclude_tags=baz;
> +maildir.synchronize_flags=true"
> +
> +test_done
> -- 
> 1.7.10.4
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch

Thread: