[asterisk-dev] [Code Review] 4145: testsuite: update pretty_print script for recent runtests.py changes.

[George Joseph]

> On Nov. 4, 2014, 3:53 p.m., Mark Michelson wrote:
> > Why the change to being a standalone script? I personally like the idea of it being a filter more.

I changed it locally months back for 2 reasons...
The first was that runtests wasn't pre-calculating and printing the total number of tests to run back then.  So I had to run runtests twice, once to get the total number (in dry-run mode) then again for real.

The other one was buffering problems.  ./runtests.py | pretty_print wasn't printing output as often as it should so I was always doing
python -u ./runtests.py | pretty_print.

Honestly, I didn't think anyone else was using it. :)

I can change it back to a filter if that's more useful.


- George


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4145/#review13673
-----------------------------------------------------------


On Nov. 3, 2014, 8:43 a.m., George Joseph wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/4145/
> -----------------------------------------------------------
> 
> (Updated Nov. 3, 2014, 8:43 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Repository: testsuite
> 
> 
> Description
> -------
> 
> Updated contrib/scripts/pretty_print so it works with recent runtests.py changes.  Also changed it from a filter to a standalone script.
> 
> 
> Diffs
> -----
> 
>   asterisk/trunk/contrib/scripts/pretty_print 5878 
> 
> Diff: https://reviewboard.asterisk.org/r/4145/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> George Joseph
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20141104/74137b0f/attachment.html>