[asterisk-dev] [Code Review] 2792: Make the failing Queue tests pass

wdoekes reviewboard at asterisk.org
Tue Sep 3 13:25:30 CDT 2013


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



/asterisk/trunk/tests/apps/queues/position_priority_maxlen/run-test
<https://reviewboard.asterisk.org/r/2792/#comment18767>

    Unnecessary parentheses.



/asterisk/trunk/tests/apps/queues/position_priority_maxlen/run-test
<https://reviewboard.asterisk.org/r/2792/#comment18768>

    Missing parentheses.


P.S. https://github.com/nvie/vim-flake8

- wdoekes


On Sept. 3, 2013, 2:20 p.m., Mark Michelson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2792/
> -----------------------------------------------------------
> 
> (Updated Sept. 3, 2013, 2:20 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-22315
>     https://issues.asterisk.org/jira/browse/ASTERISK-22315
> 
> 
> Repository: testsuite
> 
> 
> Description
> -------
> 
> The queue_baseline, set_penalty, and position_priority_maxlen tests were failing with Asterisk 12. In each case, the reason was that the tests had not been modified to take into account the new names of manager events or the new names of manager event headers. The queue_baseline and set_penalty tests have small modifications to make them pass.
> 
> The position_priority_maxlen test, on the other hand, has been converted from lua to python. This wasn't necessary to get the test to pass, but the test now runs in ~15 seconds instead of ~80 seconds. The lua script had difficulty terminating Asterisk once the test actually completed, so the test would run until the test timeout was reached. The python test library shuts Asterisk down more quickly once the test cases are complete, and so the test ends much more quickly.
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/tests/apps/queues/position_priority_maxlen/configs/ast1/extensions.conf 4085 
>   /asterisk/trunk/tests/apps/queues/position_priority_maxlen/run-test 4085 
>   /asterisk/trunk/tests/apps/queues/position_priority_maxlen/test.lua 4085 
>   /asterisk/trunk/tests/apps/queues/queue_baseline/run-test 4085 
>   /asterisk/trunk/tests/apps/queues/set_penalty/run-test 4085 
> 
> Diff: https://reviewboard.asterisk.org/r/2792/diff/
> 
> 
> Testing
> -------
> 
> I ran the three tests against both Asterisk 11 and Asterisk 12 after making the changes. Tests pass on both versions of Asterisk.
> 
> 
> Thanks,
> 
> Mark Michelson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130903/494273fd/attachment.htm>


More information about the asterisk-dev mailing list