[Piglit] [PATCH 1/2] framework: Fix summary feature

Dylan Baker baker.dylan.c at gmail.com
Wed Mar 2 16:58:57 UTC 2016


Quoting Gabriel Feceoru (2016-03-02 05:49:20)
> The options module has been refactored and broke the 'Feature Readiness"
> summary feature.
> 
> Added this feature in line with the latest options changes.
> 
> Signed-off-by: Gabriel Feceoru <gabriel.feceoru at intel.com>
> ---
>  framework/summary/feature.py | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/framework/summary/feature.py b/framework/summary/feature.py
> index 7298cff..fe2ea23 100644
> --- a/framework/summary/feature.py
> +++ b/framework/summary/feature.py
> @@ -32,7 +32,7 @@ try:
>  except ImportError:
>      import json
>  
> -from framework import core, exceptions, profile, status
> +from framework import options, exceptions, profile, status, profile
>  
>  
>  class FeatResults(object):  # pylint: disable=too-few-public-methods
> @@ -65,15 +65,15 @@ class FeatResults(object):  # pylint: disable=too-few-public-methods
>              include_filter = [incl_str] if incl_str and not incl_str.isspace() else []
>              exclude_filter = [excl_str] if excl_str and not excl_str.isspace() else []
>  
> -            opts = core.Options(include_filter=include_filter,
> -                                exclude_filter=exclude_filter)
> +            options.OPTIONS.exclude_filter = exclude_filter
> +            options.OPTIONS.include_filter = include_filter
>  
>              profiles[feature] = copy.deepcopy(profile_orig)
>  
>              # An empty list will raise PiglitFatalError exception
>              # But for reporting we need to handle this situation
>              try:
> -                profiles[feature]._prepare_test_list(opts)
> +                profiles[feature]._prepare_test_list()
>              except exceptions.PiglitFatalError:
>                  pass
>  
> -- 
> 1.9.1
> 
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/piglit

Reviewed-by: Dylan Baker <baker.dylan.c at gmail.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: signature
URL: <https://lists.freedesktop.org/archives/piglit/attachments/20160302/d289ee73/attachment.sig>


More information about the Piglit mailing list