[Piglit] [PATCH] summary: Add 'Not Run' back to changes page

Tom Stellard tom at stellard.net
Tue Jul 30 14:17:22 PDT 2013


On Tue, Jul 30, 2013 at 01:58:49PM -0700, Dylan Baker wrote:
> This change seems to have slipped under the radar, and the previous
> behavior seems more desireable. Therefore, this patch reverts Piglit to
> include tests with status "Not Run" back to the changes page.
> 
> CC: tom at stellard.net
Tested-by: Tom Stellard <thomas.stellard at amd.com>
> Signed-off-by: Dylan Baker <baker.dylan.c at gmail.com>
> ---
>  framework/summary.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/framework/summary.py b/framework/summary.py
> index 1cf32f2..003728a 100644
> --- a/framework/summary.py
> +++ b/framework/summary.py
> @@ -443,7 +443,7 @@ class Summary:
>                      # A set cannot contain duplicate entries, so creating a set
>                      # out the list will reduce it's length to 1 if all entries
>                      # are the same, meaning it is not a change
> -                    if len(set(status)) > 1 and not 0 in status:
> +                    if len(set(status)) > 1:
>                          self.tests['changes'].append(test)
>  
>                  if 'problems' in lists:
> -- 
> 1.8.3.1
> 


More information about the Piglit mailing list