[asterisk-dev] [Code Review] 3277: testsuite: ARI Originate test for assigning UniqueId on channel creation

Scott Griepentrog reviewboard at asterisk.org
Thu Mar 6 14:58:09 CST 2014



> On March 6, 2014, 2:54 p.m., Matt Jordan wrote:
> > /asterisk/trunk/tests/rest_api/bridges/bridge_by_id/test-config.yaml, line 60
> > <https://reviewboard.asterisk.org/r/3277/diff/3/?file=55244#file55244line60>
> >
> >     Which event are you referring to?

Really just a note to make it clear that the action of creating bridge does not result in a separate ARI event that you can detect and trigger the next test step on, thus, the next step is run off the same event.


- Scott


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


On March 4, 2014, 11:21 a.m., Scott Griepentrog wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3277/
> -----------------------------------------------------------
> 
> (Updated March 4, 2014, 11:21 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-23120
>     https://issues.asterisk.org/jira/browse/ASTERISK-23120
> 
> 
> Repository: testsuite
> 
> 
> Description
> -------
> 
> Tests included:
> 
> * rest_api/channels/originate_with_id: call origination with a specified Unique ID via ARI.
> 
> * rest_api/bridges/bridge_by_id: create bridge with Uniqueid ID and place channel in it.
> 
> * rest_api/playback/basic: play audio on channel and control it using unique id
> 
> * rest_api/channels/snoop_id: start and stop spy channel via unique id
> 
> Notes:
> 
> * originate tests borrow basic test plan and AMI/CEL checks from r3274 (AMI version of this same test).
> 
> * New class AriOriginateTestObject extends AriTestObject and overrides AMI channel origination with an ARI method.
> 
> * New configuration section 'requests' for WebSocketEventModule, which provides ability to issue post/delete/etc request in addition to or in place of a callback.
> 
> * Request uri's can now be constructed using variables from matched event, as in DELETE 'channel/{channel.id}'
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/tests/rest_api/tests.yaml 4735 
>   /asterisk/trunk/tests/rest_api/playback/tests.yaml PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/playback/basic/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/playback/basic/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/channels/tests.yaml 4735 
>   /asterisk/trunk/tests/rest_api/channels/snoop_id/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/channels/snoop_id/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/channels/originate_with_id/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/channels/originate_with_id/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/bridges/tests.yaml 4735 
>   /asterisk/trunk/tests/rest_api/bridges/bridge_by_id/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/bridges/bridge_by_id/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/lib/python/asterisk/ari.py 4735 
> 
> Diff: https://reviewboard.asterisk.org/r/3277/diff/
> 
> 
> Testing
> -------
> 
> All tests passed against r3191.  Changing any specified id is detected by the test.
> 
> 
> Thanks,
> 
> Scott Griepentrog
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140306/89235b9b/attachment-0001.html>


More information about the asterisk-dev mailing list