Re: [PATCH v2 9/9] Test read_early_config()

2017-03-03 Thread Johannes Schindelin
Hi Peff, On Fri, 3 Mar 2017, Jeff King wrote: > On Fri, Mar 03, 2017 at 03:04:36AM +0100, Johannes Schindelin wrote: > > > So far, we had no explicit tests of that function. > > Makes sense. The pager tests fixed in an earlier commit were effectively > checking those, but I don't mind making it

Re: [PATCH v2 9/9] Test read_early_config()

2017-03-02 Thread Jeff King
On Fri, Mar 03, 2017 at 03:04:36AM +0100, Johannes Schindelin wrote: > So far, we had no explicit tests of that function. Makes sense. The pager tests fixed in an earlier commit were effectively checking those, but I don't mind making it explicit. -Peff

[PATCH v2 9/9] Test read_early_config()

2017-03-02 Thread Johannes Schindelin
So far, we had no explicit tests of that function. Signed-off-by: Johannes Schindelin --- t/helper/test-config.c | 15 +++ t/t1309-early-config.sh | 50 + 2 files changed, 65 insertions(+) create mode 100755 t/t1309-early-config.sh d