[asterisk-dev] [Code Review]: Test Queue(F)

[Matt Jordan]

> On March 8, 2012, 8:53 a.m., Matt Jordan wrote:
> > /asterisk/trunk/tests/queues/queue_transfer_callee/test-config.yaml, line 6
> > <https://reviewboard.asterisk.org/r/1789/diff/3/?file=25745#file25745line6>
> >
> >     I might be missing something here, but based on your description, the callee is a member of a queue, while the caller is the person who actually called the Queue application.  The description you have here states that the F option is triggered when the caller hangs up, i.e., the channel that actually entered the Queue application.  Based on that description, I would expect the test to have the channel that entered the Queue application hang up first, and then have the actions associated with the F option exercised on the channel that is a member of the queue.
> >     
> >     In your test, however, it appears as if the channel that hangs up first is not the channel that entered the Queue application, but rather the channel that enters test at test_context.  In that case, the description doesn't seem to match the test.

Disregard this.  I had to reread through the test again to see what was happening here and where the queue member was involved.  This description is fine.


- Matt


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


On March 5, 2012, 2:02 p.m., jrose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1789/
> -----------------------------------------------------------
> 
> (Updated March 5, 2012, 2:02 p.m.)
> 
> 
> Review request for Asterisk Developers, Mark Michelson, Paul Belanger, and Matt Jordan.
> 
> 
> Summary
> -------
> 
> Adds a test for new Queue Option F.
> 
> This is basically just like the Baseline Queue Test, only there are 4 scenarios tested and they all use QueueF with different arguments.
> 
> 
> This addresses bug ASTERISK-19283.
>     https://issues.asterisk.org/jira/browse/ASTERISK-19283
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/tests/queues/queue_transfer_callee/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/queues/queue_transfer_callee/configs/ast1/queues.conf PRE-CREATION 
>   /asterisk/trunk/tests/queues/queue_transfer_callee/run-test PRE-CREATION 
>   /asterisk/trunk/tests/queues/queue_transfer_callee/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/queues/tests.yaml 3072 
> 
> Diff: https://reviewboard.asterisk.org/r/1789/diff
> 
> 
> Testing
> -------
> 
> It's a test.  I ran it a number of times, checked log messages, etc.  Nothing too special.
> 
> 
> Thanks,
> 
> jrose
> 
>

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