[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