[asterisk-dev] [Code Review] Test device state notifications in chan_sip

Mark Michelson reviewboard at asterisk.org
Wed Aug 29 12:33:27 CDT 2012


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

Review request for Asterisk Developers.


Summary
-------

Due to an issue being reported against Asterisk 11 because of RINGING device state notifications not being sent to SIP phones, I have created a simple test that ensures that device state changes result in notifications being sent to SIP subscribers.

This test has a single subscriber subscribe to a custom device state. Then the custom device state is changed to RINGING, to INUSE, and then to NOT_INUSE. We ensure that all three changes result in NOTIFYs being sent to the subscriber.

Note that for this test to pass currently in Asterisk 11 or Asterisk trunk, the patch from ASTERISK-20297 must first be applied. It has not been committed yet.


This addresses bug ASTERISK-20297.
    https://issues.asterisk.org/jira/browse/ASTERISK-20297


Diffs
-----

  /asterisk/trunk/lib/python/asterisk/TestCase.py 3422 
  /asterisk/trunk/tests/channels/SIP/device_state_notification/Originator.py PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/device_state_notification/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/device_state_notification/configs/ast1/sip.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/device_state_notification/sipp/uac-subscribe-unsubscribe.xml PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/device_state_notification/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/tests.yaml 3422 

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


Testing
-------


Thanks,

Mark

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120829/420080fc/attachment.htm>


More information about the asterisk-dev mailing list