Re: [PATCH 2/4] t5310: test delta reuse with bitmaps

2018-09-04 Thread Jeff King
On Tue, Sep 04, 2018 at 12:05:58PM -0700, Stefan Beller wrote: > Yeah, maybe we need to ask for more tests in the 'real' test suite, and not > just in some special corner (such as t/perf/ or any of the environment > variable proposals nearby). > > I wonder if we can make use of git.git in the tes

Re: [PATCH 2/4] t5310: test delta reuse with bitmaps

2018-09-04 Thread Junio C Hamano
Stefan Beller writes: > From some offline discussion, maybe we want to adapt a philosophy of > > Each patch needs to add a test, that fails when the patch > is not applied, but succeeds when it is applied. This shows > that _some_ code in the patch is exercised at least. > > (and automatica

Re: [PATCH 2/4] t5310: test delta reuse with bitmaps

2018-09-04 Thread Stefan Beller
On Sat, Sep 1, 2018 at 1:03 AM Jeff King wrote: > > On Sat, Sep 01, 2018 at 03:48:13AM -0400, Jeff King wrote: > > > Commit 6a1e32d532 (pack-objects: reuse on-disk deltas for > > thin "have" objects, 2018-08-21) taught pack-objects a new > > optimization trick. Since this wasn't meant to change >

Re: [PATCH 2/4] t5310: test delta reuse with bitmaps

2018-09-01 Thread Jeff King
On Sat, Sep 01, 2018 at 10:29:25PM +0200, Ævar Arnfjörð Bjarmason wrote: > > Anyway. Not that exciting, and kind of obviously dumb in retrospect. But > > I think it was worth analyzing to see what went wrong. If there's an > > immediate lesson, it is probably: add tests even for changes that aren'

Re: [PATCH 2/4] t5310: test delta reuse with bitmaps

2018-09-01 Thread Ben Peart
On 9/1/2018 4:29 PM, Ævar Arnfjörð Bjarmason wrote: B.t.w. for Ben or anyone else who knows about the fsmonitor part of this: I've long been running the whole test suite with `GIT_FSMONITOR_TEST=$PWD/t7519/fsmonitor-all prove ...` (also along with GIT_TEST_SPLIT_INDEX=) after all the main te

Re: [PATCH 2/4] t5310: test delta reuse with bitmaps

2018-09-01 Thread Ævar Arnfjörð Bjarmason
On Sat, Sep 01 2018, Jeff King wrote: > On Sat, Sep 01, 2018 at 03:48:13AM -0400, Jeff King wrote: > >> Commit 6a1e32d532 (pack-objects: reuse on-disk deltas for >> thin "have" objects, 2018-08-21) taught pack-objects a new >> optimization trick. Since this wasn't meant to change >> user-visible

Re: [PATCH 2/4] t5310: test delta reuse with bitmaps

2018-09-01 Thread Jeff King
On Sat, Sep 01, 2018 at 03:48:13AM -0400, Jeff King wrote: > Commit 6a1e32d532 (pack-objects: reuse on-disk deltas for > thin "have" objects, 2018-08-21) taught pack-objects a new > optimization trick. Since this wasn't meant to change > user-visible behavior, but only produce smaller packs more >

[PATCH 2/4] t5310: test delta reuse with bitmaps

2018-09-01 Thread Jeff King
Commit 6a1e32d532 (pack-objects: reuse on-disk deltas for thin "have" objects, 2018-08-21) taught pack-objects a new optimization trick. Since this wasn't meant to change user-visible behavior, but only produce smaller packs more quickly, testing focused on t/perf/p5311. However, since people don'