[Piglit] [PATCH] xts: Fix skip handling.

[Dylan Baker]

Reviewed-by: Dylan Baker <baker.dylan.c at gmail.com>

Quoting Eric Anholt (2016-01-27 17:28:31)
> Commit 23e06ffaa51b9075c1e5e0c220c991efc5609838 broke skip handling by
> overwriting the result after we had set it.
> ---
>  tests/xts.py | 12 ++++--------
>  1 file changed, 4 insertions(+), 8 deletions(-)
> 
> diff --git a/tests/xts.py b/tests/xts.py
> index 6798e70..e528626 100644
> --- a/tests/xts.py
> +++ b/tests/xts.py
> @@ -133,6 +133,8 @@ class XTSTest(Test):  # pylint: disable=too-few-public-methods
>          return images
>  
>      def interpret_result(self):
> +        super(XTSTest, self).interpret_result()
> +
>          try:
>              with open(self.test_results_file, 'r') as rfile:
>                  log = rfile.read()
> @@ -155,13 +157,9 @@ class XTSTest(Test):  # pylint: disable=too-few-public-methods
>                  self.result.result = 'warn'
>              else:
>                  self.result.result = 'fail'
> -        else:
> -            self.result.result = 'fail'
>  
>          self.result.images = self._process_log_for_images(log)
>  
> -        super(XTSTest, self).interpret_result()
> -
>  
>  class RendercheckTest(Test):
>      def __init__(self, args):
> @@ -169,14 +167,12 @@ class RendercheckTest(Test):
>          self.testname = "rendercheck " + " ".join(args)
>  
>      def interpret_result(self):
> +        super(RendercheckTest, self).interpret_result()
> +
>          if self.result.returncode == 0:
>              self.result.result = 'pass'
>          elif self.result.returncode == 77:
>              self.result.result = 'skip'
> -        else:
> -            self.result.result = 'fail'
> -
> -        super(RendercheckTest, self).interpret_result()
>  
>  
>  def _populate_profile_xts(profile):
> -- 
> 2.6.4
> 
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: signature
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20160128/83240292/attachment.sig>