[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