[asterisk-dev] [Code Review] 3018: testsuite: Add tests for Say application jump behavior with SAY_DTMF_INTERRUPT enabled and disabled

[Jonathan Rose]

> On Dec. 5, 2013, 9:13 p.m., opticron wrote:
> > /asterisk/trunk/tests/apps/say_interrupt/configs/ast1/extensions.conf, lines 20-25
> > <https://reviewboard.asterisk.org/r/3018/diff/2/?file=48959#file48959line20>
> >
> >     These could use Hangup()s.

k, done.


- Jonathan


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


On Dec. 2, 2013, 5:10 p.m., Jonathan Rose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3018/
> -----------------------------------------------------------
> 
> (Updated Dec. 2, 2013, 5:10 p.m.)
> 
> 
> Review request for Asterisk Developers, Mark Michelson and rmudgett.
> 
> 
> Repository: testsuite
> 
> 
> Description
> -------
> 
> https://reviewboard.asterisk.org/r/3011/
> 
> Tests the functionality of this feature, both when set and not set
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/tests/apps/tests.yaml 4343 
>   /asterisk/trunk/tests/apps/say_interrupt/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/apps/say_interrupt/configs/ast1/extensions.conf PRE-CREATION 
> 
> Diff: https://reviewboard.asterisk.org/r/3018/diff/
> 
> 
> Testing
> -------
> 
> It's a test.  It runs and completes successfully on my box with all the expected extensions being hit according to the logs. I changed some things around and was easily able to make it fail in ways that I expected it to fail, which is good. Changing the SAY_DTMF_INTERRUPT value in either test from what it currently is causes failures, which is also good. All DTMF values in the test must be issued for each test in order for the test to be considered successful.
> 
> 
> Thanks,
> 
> Jonathan Rose
> 
>

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