test: Rework testing of maildir-synchronization feature.
authorCarl Worth <cworth@cworth.org>
Tue, 9 Nov 2010 23:36:49 +0000 (15:36 -0800)
committerCarl Worth <cworth@cworth.org>
Thu, 11 Nov 2010 10:35:03 +0000 (02:35 -0800)
commit882b994c173475f8973d5f5a1c0d1c25e4f93818
tree67d024587fa53899aa34348a9cc4f33b27eebb71
parent736ac42c45f62103e99ef18ef06b201d0bcd546f
test: Rework testing of maildir-synchronization feature.

This change reworks these tests in several ways:

1. Bring tests into "new" test style preferring test_expect_equal over
   test_expect_success in almost all cases.

2. Don't emit test results for intermediate items not actually being
   tested, (things like "no new messages", "search for message",
   etc.). Those things are already covered by existing tests such as
   "basic" or "search" and only serve to obscure what's actually being
   tested.

3. Change sense of the test showing failure to rename a file from
   "new" to "cur" when "cur" doesn't exist.

   In this case, notmuch should detect that this is not a maildir and
   should not attempt to do any renaming of the file.

4. Extend dump/restore test to also exercise addition of tag, not just
   removal.

Both items #3 and #4 above show shortcomings in the current
implementation. These are currently resulting in test results of FAIL
and indicate bugs that need to be fixed.
test/maildir-sync