[Piglit] [PATCH] framework: Add support for a test timeout

Daniel Vetter daniel.vetter at ffwll.ch
Wed Nov 13 16:56:58 PST 2013


On Wed, Nov 13, 2013 at 10:03 PM, Dylan Baker <baker.dylan.c at gmail.com> wrote:
>> +class Timeout(Status):
>> +    name = 'timeout'
>> +    value = 50
>> +    fraction = (0, 0)
>
> Are you sure that you want this to be (0, 0)? That means that tests with this
> status will not be counted toward the total number of tests.  I don't think
> you want to change this from the default (0, 1), but if you're sure I wont
> argue :)

Indeed, that's just copypaste fail. Fixed and patches pushed, thanks
for your review.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch


More information about the Piglit mailing list