[asterisk-dev] [Code Review] 4232: Testsuite: Tests for AMI NewConnectedLine and ARI ChannelConnectedLine events

Mark Michelson reviewboard at asterisk.org
Thu Dec 4 15:59:05 CST 2014


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

(Updated Dec. 4, 2014, 9:59 p.m.)


Review request for Asterisk Developers.


Changes
-------

Made the suggested changes by opticron. This allowed for the connected_line.py file to be removed completely.


Bugs: ASTERISK-24554
    https://issues.asterisk.org/jira/browse/ASTERISK-24554


Repository: testsuite


Description
-------

This adds two tests to the testsuite to ensure that connected line events are emitted when the connected line on the channel changes.

In tests/manager/connected_line, a channel places a call to an extension that uses the CONNECTEDLINE() dialplan function to change connected line information. We ensure that an AMI NewConnectedLine event is emitted and that it has the expected values for connected line name and number.

In tests/rest_api/channels/connected_line, a channel calls a Stasis application. When the channel enters Stasis, the Stasis application updates the connected line on the channel using the CONNECTEDLINE() dialplan function. We ensure that a ChannelConnectedLine event is emitted and that the connected line name and number have the expected values.


Diffs (updated)
-----

  /asterisk/trunk/tests/rest_api/channels/tests.yaml 5983 
  /asterisk/trunk/tests/rest_api/channels/connected_line_update/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/rest_api/channels/connected_line_update/configs/ast1/extensions.conf PRE-CREATION 

Diff: https://reviewboard.asterisk.org/r/4232/diff/


Testing
-------

The tests pass. In addition, I tested for false positives by removing the calls to the CONNECTEDLINE() dialplan function and ensuring that the tests fail due to the events not being emitted.


Thanks,

Mark Michelson

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


More information about the asterisk-dev mailing list