[Piglit] [PATCH 4/4] framework/summary.py: print test names with '/' instead of '@' for console

Ilia Mirkin imirkin at alum.mit.edu
Mon May 11 15:02:11 PDT 2015


Series is

Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>

(assuming that you double-check that the 'self' thing works for your
summary helper function)

On Mon, May 11, 2015 at 5:57 PM, Dylan Baker <baker.dylan.c at gmail.com> wrote:
> While storing with '@' solves a lot of problems (especially cross
> platform issues with mixing grouptools and os.path functions), it isn't
> particularly easy to read.
>
> This patch converts the printed representation to '/' separated.
>
> Signed-off-by: Dylan Baker <dylanx.c.baker at intel.com>
> ---
>  framework/summary.py | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/framework/summary.py b/framework/summary.py
> index d133aca..cad8cc7 100644
> --- a/framework/summary.py
> +++ b/framework/summary.py
> @@ -570,7 +570,7 @@ class Summary:
>              """
>              for test in list_:
>                  print("{test}: {statuses}".format(
> -                    test=test,
> +                    test='/'.join(test.split(grouptools.SEPARATOR)),
>                      statuses=' '.join(str(i.tests.get(test, {'result': so.SKIP})
>                                            ['result']) for i in self.results)))
>
> @@ -605,4 +605,3 @@ class Summary:
>              printer(self.tests['incomplete'])
>          elif mode == 'summary':
>              print_summary()
> -
> --
> 2.4.0
>


More information about the Piglit mailing list