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

jrose reviewboard at asterisk.org
Thu Sep 12 10:19:58 CDT 2013



> 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.
> 
> jrose wrote:
>     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.

It's worth noting that prior to res_parking going in, the behavior when the template lot didn't exist was that it would use the default instead, which is why this would work for you before. You were using the variable incorrectly though. I discussed it with Matt before making the change and he agreed with my behavior change at the time.


- 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/1b6736d5/attachment.htm>


More information about the asterisk-dev mailing list