Thread (34 messages) flat view 34 messages, 5 authors, 2016-07-21

Re: [PATCH v1 2/6] Status and checkout unit tests for --porcelain[=<n>]

From: Johannes Schindelin <hidden>
Date: 2016-07-20 15:19:52

Hi Jeff,

On Tue, 19 Jul 2016, Jeff Hostetler wrote:
Simple unit tests to validate the argument parsing.

Signed-off-by: Jeff Hostetler <redacted>
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help