[asterisk-dev] [Code Review] 3357: testsuite: Add off-nominal subscription tests for PJSIP.

Jonathan Rose reviewboard at asterisk.org
Fri Apr 11 17:12:16 CDT 2014


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

(Updated April 11, 2014, 5:12 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers, Kevin Harwell and Matt Jordan.


Changes
-------

Committed in revision 4952


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


Repository: testsuite


Description
-------

No Accept header
This would set up the subscription, but use the default type for the event package being subscribed for

Disallowed subscriptions
A SIP UA subscribes for a valid event package with Asterisk, but the endpoint doesn't allow subscriptions
Asterisk responds with a 603

MinExpiry not met
A SIP UA sends a subscription with an expiration time that is less than the configured minexpiry for the endpoint
Asterisk responds with a 423

No Event Header
A SIP UA sends a subscription but fails to provide an Event header
Asterisk responds with a 489

Unknown Event Package
A SIP UA sends a subscription for an unknown event package
Asterisk responds with a 489


Each of these tests is based on kharwell's Digium Presence test. As such, the No Accept Header test does require some digium phone specific stuff to be loaded in order to work.  For all the other tests though, the tests are fairly general and will just fail for the reasons you would expect.


Diffs
-----

  /asterisk/trunk/tests/channels/pjsip/subscriptions/unknown_event_package/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/unknown_event_package/sipp/subscribe.xml PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/unknown_event_package/configs/ast1/pjsip.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/unknown_event_package/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/unallowed/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/unallowed/sipp/subscribe.xml PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/unallowed/configs/ast1/pjsip.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/unallowed/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/tests.yaml 4836 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/tests.yaml 4836 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/no_accept_header/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/no_accept_header/sipp/subscribe.xml PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/no_accept_header/run-test PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/no_accept_header/configs/ast1/pjsip.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/no_accept_header/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/no_event_header/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/no_event_header/sipp/subscribe.xml PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/no_event_header/configs/ast1/pjsip.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/no_event_header/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/mwi/tests.yaml 4836 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/mwi/no_accept_header/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/mwi/no_accept_header/sipp/subscribe.xml PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/mwi/no_accept_header/run-test PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/mwi/no_accept_header/configs/ast1/pjsip.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/mwi/no_accept_header/configs/ast1/modules.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/below_min_expiry/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/below_min_expiry/sipp/subscribe.xml PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/below_min_expiry/configs/ast1/pjsip.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/pjsip/subscriptions/below_min_expiry/configs/ast1/extensions.conf PRE-CREATION 

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


Testing
-------

Ran tests to determine that the sipp scenarios were received the expected responses for every scenario.


Thanks,

Jonathan Rose

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


More information about the asterisk-dev mailing list