[PATCH 6 of 6] blackbox: tests for the blackbox extension
Durham Goode
durham at fb.com
Tue Feb 12 17:18:08 CST 2013
On 2/12/13 11:36 AM, "Pierre-Yves David" <pierre-yves.david at logilab.fr>
wrote:
>On Mon, Feb 11, 2013 at 01:39:57PM -0800, Bryan O'Sullivan wrote:
>> On Sun, Feb 10, 2013 at 4:24 AM, Brodie Rao <brodie at sf.io> wrote:
>>
>> > I think it's a little confusing that the date is completely globbed
>> > out here. Can you make the date stable and add it to the test?
>> >
>>
>> The date is in the output from blackbox. It doesn't make sense to me to
>>add
>> a hack to make those dates constant for testing.
>
>Stable date does not make sense. Using regex instead of glob would be me
>accurate.
>
I don't think the way we test the date/user string is particularly
important. The latest patch series uses a fixed date and user so that if
the test fails it's easy to visually compare the lines without a glob or
regex uglying it up.
More information about the Mercurial-devel
mailing list