[asterisk-dev] [Code Review] 3842: Tests the forward and reverse playback ARI functions for both channels and bridges

opticron reviewboard at asterisk.org
Tue Jul 29 12:05:22 CDT 2014


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

Ship it!


Your test descriptions need editing to remove/fix sentence fragments, but other than that this looks good to go.

- opticron


On July 24, 2014, 12:44 p.m., Christopher Wolfe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3842/
> -----------------------------------------------------------
> 
> (Updated July 24, 2014, 12:44 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-24092
>     https://issues.asterisk.org/jira/browse/ASTERISK-24092
> 
> 
> Repository: testsuite
> 
> 
> Description
> -------
> 
> Test 1: Puts a local channel half into Stasis, and then plays back a sound on it (tt-monkeys).  Upon starting the sound, the playback gets reversed using /control?operation=reverse and a TestEvent gets released that signifies a reversal has occurred.
> 
> Test 2: Same thing as Test 1 except that you are using the /control?operation=forward ARI command and looking for a FastForward TestEvent.
> 
> Test 3: Same thing as Test 1 except that a channel gets put into a bridge, and the bridge instead is the one that starts the playback.
> 
> Test 4: Same thing as Test 3 except that you are testing the /control?operation=forward command on the bridge.
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/tests/rest_api/channels/playback/tests.yaml 5249 
>   /asterisk/trunk/tests/rest_api/channels/playback/reverse/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/channels/playback/reverse/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/channels/playback/forward/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/channels/playback/forward/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/bridges/playback/tests.yaml 5249 
>   /asterisk/trunk/tests/rest_api/bridges/playback/reverse/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/bridges/playback/reverse/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/bridges/playback/forward/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/bridges/playback/forward/configs/ast1/extensions.conf PRE-CREATION 
> 
> Diff: https://reviewboard.asterisk.org/r/3842/diff/
> 
> 
> Testing
> -------
> 
> Made sure that expected values were received.  These tests were pretty straightforward otherwise.
> 
> 
> Thanks,
> 
> Christopher Wolfe
> 
>

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


More information about the asterisk-dev mailing list