From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Wed, 20 Jul 2016 15:19:05 GMT
Subject: Re: [PATCH v1 2/6] Status and checkout unit tests for --porcelain[=<n>]
Message-ID: <alpine.DEB.2.20.1607201710240.14111@virtualbox>
In-Reply-To: <1468966258-11191-3-git-send-email-jeffhost@microsoft.com>

Hi Jeff,

On Tue, 19 Jul 2016, Jeff Hostetler wrote:

> Simple unit tests to validate the argument parsing.
> 
> Signed-off-by: Jeff Hostetler <jeffhost@microsoft.com>

They are simple alright, but do we really need so many of them? I would
like to keep the ones in t7060, but I do not think that we necessarily
have to add the t7501 ones.

I know I am a bit at odds here with Junio, who frequently prefers more
tests. It's just that I have to run the complete test suite so often and
it does take 30-45 minutes to run here (due to the fact that the test
suite exercises quite a lot of the POSIX emulation layer via shell
scripting).

So do not take my suggestions as the sole basis for deciding how to go
from here.

Ciao,
Johannes