Re: [GUILT 07/28] Added test cases for "guilt fold".

On Wed, May 07, 2014 at 10:59:56PM +0200, Per Cederqvist wrote:

On Tue, May 6, 2014 at 9:40 PM, Jeff Sipek [off-list ref] wrote:

quoted

On Fri, Mar 21, 2014 at 08:31:45AM +0100, Per Cederqvist wrote:

quoted

Test that we can combine any combination of patches with empty and
non-empty messages, both with and without guilt.diffstat.  (All
patches are empty.)

I feel like we should have *some* content there - most of the time, I care
more about the diffs getting folded than the commit message :)

I added these tests for a reason: the reproduce a bug in guilt that I found.

I'm afraid that having some content might hide the bug I found. (I'm also
equally afraid that it might uncover other bugs in guilt, which would delay
integration of this patch series. So adding more test cases with content
is a good thing to do, but maybe not in this patch series.)

Fair enough.  I use guilt-fold all the time and it hasn't lost any of my
diffs, so I'm happy to defer this until some point in the future.

...

quoted

for using_diffstat in true false ; do
        for patcha in empty nonempty ; do
                for patchb in empty nonempty ; do
                        echo "%% $patcha + $patchb (diffstat=$using_diffstat)"
                        ${patcha}_patch $patcha
                        ${patchb}_patch $patchb
                        cmd guilt pop $patchb
                        cmd guilt fold $patchb
                        fixup_time_info $patcha
                        cmd list_files
                        cleanup $patcha
                        cmd list_files
                done
        done
done

Aha!  That's better, IMO.

I'll try that and post a version 2 of the series. It might take a few
days, though.

No problem.  I'm still the slower one of the two of us. :/

Jeff.

-- 
Keyboard not found!
Press F1 to enter Setup