[Piglit] [PATCH 3/6] README: Update result statuses

Kenneth Graunke kenneth at whitecape.org
Fri Oct 24 11:25:39 PDT 2014


On Tuesday, October 21, 2014 11:23:46 AM Dylan Baker wrote:
> Adds the crash, timeout and the dmesg-* statuses.
> 
> Signed-off-by: Dylan Baker <dylanx.c.baker at intel.com>
> ---
>  README | 23 ++++++++++++-----------
>  1 file changed, 12 insertions(+), 11 deletions(-)
> 
> diff --git a/README b/README
> index 055e8e4..e302ec3 100644
> --- a/README
> +++ b/README
> @@ -217,21 +217,22 @@ Have a look at the results with a browser:
>  
>  The summary shows the 'status' of a test:
>  
> - pass  This test has completed successfully.
> + pass   This test has completed successfully.
>  
> - warn  The test completed successfully, but something unexpected happened.
> -       Look at the details for more information.
> + warn   The test completed successfully, but something unexpected happened.
> +        Look at the details for more information.
>  
> - fail  The test failed.
> + fail   The test failed.
>  
> - skip  The test was skipped.
> + crash  The test binary exited with a non-zero exit code
>  
> -[Note: Once performance tests are implemented, 'fail' will mean that the 
test
> -       rendered incorrectly or didn't complete, while 'warn' will indicate 
a
> -       performance regression]
> -[Note: For performance tests, result and status will be different concepts.
> -       While status is always restricted to one of the four values above,
> -       the result can contain a performance number like frames per second]
> + skip   The test was skipped.
> +
> + timeout  The test binary ran longer than it was allocated to and was
> +          forcably killed

Bad grammar (ran longer than it was allocated to) and typos (forcably).

How about:

 timeout  The test ran longer than its allotted time and was forcibly killed.


> +
> +There are also dmesg-* statuses. These have the same meaning as above, but 
were

are triggered (nothing else seems to use past tense)

> +triggered by dmesg related messages.
>  
>  
>  4. Available test sets
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20141024/9b2e84bc/attachment.sig>


More information about the Piglit mailing list