On Thursday, March 20, 2014 16:04:59 Thomas Wood wrote:
> Signed-off-by: Thomas Wood <[email protected]>
> ---
>  framework/log.py | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/framework/log.py b/framework/log.py
> index c15134b..fc37001 100644
> --- a/framework/log.py
> +++ b/framework/log.py
> @@ -93,6 +93,10 @@ class Log(object):
>          assert result in self.__summary_keys
>          self.__summary[result] += 1
>
> +        # print a final summary line
> +        if self.__complete == self.__total:
> +            self.__print(value, result)
> +
>      @synchronized_self
>      def log(self, name, result):
>          """ Print to the screen

IMHO, it would better to add a final call method (or modify Log.post_log to
have only keyword arguments) that just prints [x/y] and the summary (ie:
[8/8] pass: 7 fail: 1) that we add to TestProfile.run. That would remove the
Running Test(s) as well.

I'm seeing some weird artifiacts from this:
master: [15/16] pass: 13, skip: 2 Running Test(s): 14 14 15 15
patch:  [16/16] pass: 14, skip: 2 Running Test(s): 14 15 15 1508

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to