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

[Matt Jordan]

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

Ship it!



/asterisk/trunk/lib/python/asterisk/TestCase.py
<https://reviewboard.asterisk.org/r/2090/#comment13614>

    Not that I disagree with this change, but is there a reason why we needed to change this around?



/asterisk/trunk/tests/channels/SIP/device_state_notification/test-config.yaml
<https://reviewboard.asterisk.org/r/2090/#comment13615>

    Python module names should actually be lowercase - so this should be "originator.Originator"
    
    Yes, we violate this willy nilly all over the test suite (and I'm a prime offender on it), but we probably should be consistent with PEP8 guidelines as much as possible going forward.
    
    Also: this may be useful in multiple situations.  If so, we may want to add this to the library and make the module a bit more configurable.  We can defer that to a later code change if you'd prefer.


- Matt


On Aug. 29, 2012, 12:33 p.m., Mark Michelson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2090/
> -----------------------------------------------------------
> 
> (Updated Aug. 29, 2012, 12:33 p.m.)
> 
> 
> 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/20120904/5c43b396/attachment.htm>