[PATCH v3 2/5] test: Add tests for 'config' command

2012-04-10 Thread Jameson Graef Rollins
On Thu, Apr 05 2012, Peter Wang  wrote:
> +test_begin_subtest "Set string value"
> +notmuch config set foo.bar baz
> +test_expect_equal "$(notmuch config get foo.bar)" "baz"
> +
> +test_begin_subtest "Set list value"
> +notmuch config set foo.list xxx "yyy yyy" "zzz zzz"
> +test_expect_equal "$(notmuch config get foo.list)" "\
> +xxx
> +yyy yyy
> +zzz zzz"

I find it slightly strange to use non-existent fields here, but I also
don't see that it hurts anything either.  At least at the moment.
Another option would be to use an existing field, and change it back
when you're done.

jamie.
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: not available
URL: 



Re: [PATCH v3 2/5] test: Add tests for 'config' command

2012-04-10 Thread Jameson Graef Rollins
On Thu, Apr 05 2012, Peter Wang noval...@gmail.com wrote:
 +test_begin_subtest Set string value
 +notmuch config set foo.bar baz
 +test_expect_equal $(notmuch config get foo.bar) baz
 +
 +test_begin_subtest Set list value
 +notmuch config set foo.list xxx yyy yyy zzz zzz
 +test_expect_equal $(notmuch config get foo.list) \
 +xxx
 +yyy yyy
 +zzz zzz

I find it slightly strange to use non-existent fields here, but I also
don't see that it hurts anything either.  At least at the moment.
Another option would be to use an existing field, and change it back
when you're done.

jamie.


pgpGwpJZe589Z.pgp
Description: PGP signature
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch


[PATCH v3 2/5] test: Add tests for 'config' command

2012-04-06 Thread Peter Wang
Start a new test script.
---
 test/config   |   25 +
 test/notmuch-test |1 +
 2 files changed, 26 insertions(+), 0 deletions(-)
 create mode 100755 test/config

diff --git a/test/config b/test/config
new file mode 100755
index 000..75f662d
--- /dev/null
+++ b/test/config
@@ -0,0 +1,25 @@
+#!/usr/bin/env bash
+
+test_description='"notmuch config"'
+. test-lib.sh
+
+test_begin_subtest "Get string value"
+test_expect_equal "$(notmuch config get user.name)" "Notmuch Test Suite"
+
+test_begin_subtest "Get list value"
+test_expect_equal "$(notmuch config get new.tags)" "\
+unread
+inbox"
+
+test_begin_subtest "Set string value"
+notmuch config set foo.bar baz
+test_expect_equal "$(notmuch config get foo.bar)" "baz"
+
+test_begin_subtest "Set list value"
+notmuch config set foo.list xxx "yyy yyy" "zzz zzz"
+test_expect_equal "$(notmuch config get foo.list)" "\
+xxx
+yyy yyy
+zzz zzz"
+
+test_done
diff --git a/test/notmuch-test b/test/notmuch-test
index f03b594..e08ec72 100755
--- a/test/notmuch-test
+++ b/test/notmuch-test
@@ -19,6 +19,7 @@ cd $(dirname "$0")
 TESTS="
   basic
   help-test
+  config
   new
   count
   search
-- 
1.7.4.4



[PATCH v3 2/5] test: Add tests for 'config' command

2012-04-05 Thread Peter Wang
Start a new test script.
---
 test/config   |   25 +
 test/notmuch-test |1 +
 2 files changed, 26 insertions(+), 0 deletions(-)
 create mode 100755 test/config

diff --git a/test/config b/test/config
new file mode 100755
index 000..75f662d
--- /dev/null
+++ b/test/config
@@ -0,0 +1,25 @@
+#!/usr/bin/env bash
+
+test_description='notmuch config'
+. test-lib.sh
+
+test_begin_subtest Get string value
+test_expect_equal $(notmuch config get user.name) Notmuch Test Suite
+
+test_begin_subtest Get list value
+test_expect_equal $(notmuch config get new.tags) \
+unread
+inbox
+
+test_begin_subtest Set string value
+notmuch config set foo.bar baz
+test_expect_equal $(notmuch config get foo.bar) baz
+
+test_begin_subtest Set list value
+notmuch config set foo.list xxx yyy yyy zzz zzz
+test_expect_equal $(notmuch config get foo.list) \
+xxx
+yyy yyy
+zzz zzz
+
+test_done
diff --git a/test/notmuch-test b/test/notmuch-test
index f03b594..e08ec72 100755
--- a/test/notmuch-test
+++ b/test/notmuch-test
@@ -19,6 +19,7 @@ cd $(dirname $0)
 TESTS=
   basic
   help-test
+  config
   new
   count
   search
-- 
1.7.4.4

___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch