[PATCH] run-tests: report path for tests outside current directory

Pierre-Yves David pierre-yves.david at ens-lyon.org
Fri Feb 26 16:39:04 EST 2016



On 02/25/2016 09:27 PM, timeless wrote:
> On Thu, Feb 25, 2016 at 3:21 PM, timeless <timeless at mozdev.org> wrote:
>>
>> @@ -1640,6 +1645,7 @@
>>   # newest.
>>
>>   def loadtimes(testdir):
>> +    # This should only deal with times for tests in testdir
>>       times = []
>>       try:
>>           with open(os.path.join(testdir, '.testtimes-')) as fp:
>> @@ -1652,6 +1658,7 @@
>>       return times
>>
>>   def savetimes(testdir, result):
>> +    # This should only deal with times for tests in testdir
>>       saved = dict(loadtimes(testdir))
>>       maxruns = 5
>>       skipped = set([str(t[0]) for t in result.skipped])
>
> These hunks are a todo which shouldn't really be included in this commit.

Can you send a V2 series with them excluded (and the todo in other 
commit(s) if applicable)

-- 
Pierre-Yves David


More information about the Mercurial-devel mailing list