[asterisk-dev] [Code Review] 2815: testsuite: bridge/parkcall_blindxfer test update for res_parking module and related changes.

[jrose]

> On Sept. 11, 2013, 10:56 p.m., jbigelow wrote:
> > /asterisk/trunk/tests/bridge/parkcall_blindxfer/configs/ast1/extensions.conf, line 36
> > <https://reviewboard.asterisk.org/r/2815/diff/1/?file=45484#file45484line36>
> >
> >     I don't understand why this would be changed when this section of dial plan is for the dynamic parking lot tests.

PARKINGDYNAMIC refers to the name of the template lot being used. If the lot doesn't already exist, you will have nothing to base the template on and lot creation will fail.


- jrose


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


On Sept. 2, 2013, 10:07 p.m., jrose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2815/
> -----------------------------------------------------------
> 
> (Updated Sept. 2, 2013, 10:07 p.m.)
> 
> 
> Review request for Asterisk Developers, kmoore and Matt Jordan.
> 
> 
> Bugs: ASTERISK-22328
>     https://issues.asterisk.org/jira/browse/ASTERISK-22328
> 
> 
> Repository: testsuite
> 
> 
> Description
> -------
> 
> This test was originally written by John Bigelow. I've updated it to handle the test events it expects within their own functions rather than by using event parameters to address all of the events.
> 
> In other regards, the problems here were similar to other review tests.  There is a difference in the expectations of events between 11 and 12.
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/tests/bridge/parkcall_blindxfer/Hangup.py 4081 
>   /asterisk/trunk/tests/bridge/parkcall_blindxfer/configs/ast1/extensions.conf 4081 
>   /asterisk/trunk/tests/bridge/parkcall_blindxfer/configs/ast1/res_parking.conf PRE-CREATION 
>   /asterisk/trunk/tests/bridge/parkcall_blindxfer/test-config.yaml 4081 
>   /asterisk/trunk/tests/bridge/tests.yaml 4081 
> 
> Diff: https://reviewboard.asterisk.org/r/2815/diff/
> 
> 
> Testing
> -------
> 
> Tested in Asterisk 11 and 12 to make sure they still passed and processed everything expected.
> 
> 
> Thanks,
> 
> jrose
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130912/612bc5d5/attachment-0001.htm>