Date: Sun, 27 Jul 2014 12:12:22 -0400
From: Austin Clements <amdragon@MIT.EDU>
To: Mark Walters <markwalters1009@gmail.com>
Subject: Re: [PATCH 07/14] test: Tests for future version and unknown feature
	handling
Message-ID: <20140727161222.GE13893@mit.edu>
References: <1406433173-19169-1-git-send-email-amdragon@mit.edu>
	<1406433173-19169-8-git-send-email-amdragon@mit.edu>
	<871tt718xp.fsf@qmul.ac.uk>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <871tt718xp.fsf@qmul.ac.uk>
User-Agent: Mutt/1.5.21 (2010-09-15)
Cc: notmuch@notmuchmail.org
Precedence: list

Quoth Mark Walters on Jul 27 at  9:51 am:
> 
> On Sun, 27 Jul 2014, Austin Clements <amdragon@MIT.EDU> wrote:
> > ---
> >  test/T530-upgrade.sh | 46 ++++++++++++++++++++++++++++++++++++++++++++++
> >  1 file changed, 46 insertions(+)
> 
> Hi
> 
> Just a quick comment on this patch so far. After this patch I got lots
> of test failures in the second (old) part of T530-upgrade because I
> hadn't downloaded the test databases: prior to this patch no tests
> failed because those tests were skipped.
> 
> Once I downloaded the databases everything was fine again so the problem
> is just that this patch is breaking the automatic skipping of the
> old tests in T530-upgrade.

Oh.  Apparently the prereq stuff only works before the first call to
test_begin_subtest.  Maybe I should just move the subtests I added to
a separate top-level test?