From: Tomi Ollila <tomi.ollila@iki.fi>
To: David Bremner <david@tethera.net>, David Bremner <david@tethera.net>
Subject: Re: test suite: add more debugging output for notmuch-new
In-Reply-To: <1438670914-17203-1-git-send-email-david@tethera.net>
References: <m2a8ua2gm9.fsf@guru.guru-group.fi>
 <1438670914-17203-1-git-send-email-david@tethera.net>
User-Agent: Notmuch/0.20.2+42~gbf3bfb8 (http://notmuchmail.org) Emacs/24.3.1
 (x86_64-unknown-linux-gnu)
Date: Tue, 04 Aug 2015 11:25:51 +0300
Message-ID: <m2pp33zchc.fsf@guru.guru-group.fi>
MIME-Version: 1.0
Content-Type: text/plain
Cc: notmuch@notmuchmail.org
Precedence: list

On Tue, Aug 04 2015, David Bremner <david@tethera.net> wrote:

> The first patch in the series is a trivial cleanup.
>
> I _thought_ (and tricked Tomi ;)) that the next one was trivial too,
> but now I'm not so sure.
>
> First, I forgot to update one test, but I mostly understand (or can at
> least rationalize) that output. Then I had the bright idea that I
> should add debugging output to other tests in T050-new. In 5 cases I
> can't really understand the results. It seems like it takes several
> runs of notmuch-new to remove directory entries. Maybe this is
> intended, but I couldn't figure that out in a quick look.

I got the same test results -- and could not figure out the reason through
a brief look at the notmuch-new.c code.

+1 for the series for now, hopefully we soon find out the resolution to the
known broken tests.

Tomi