[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