Thread (3 messages) 3 messages, 3 authors, 2017-12-07

Re: [PATCH] diff: add tests for --relative without optional prefix value

flat view

From: Jeff King <hidden>
Date: 2017-12-07 21:13:11

On Thu, Dec 07, 2017 at 11:01:35AM -0800, Jacob Keller wrote:
From: Jacob Keller <redacted>

We already have tests for --relative, but they currently only test when
a prefix has been provided. This fails to test the case where --relative
by itself should use the current directory as the prefix.

Teach the check_$type functions to take a directory argument to indicate
which subdirectory to run the git commands in. Add a new test which uses
this to test --relative without a prefix value.
This looks good to me (and I slightly prefer it over Junio's for the
simplicity).

I agree on the ordering suggestion Junio made.

-Peff
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help