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

[Jonathan Rose]

> On March 24, 2014, 3:33 p.m., Mark Michelson wrote:
> > /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/no_accept_header/sipp/subscribe.xml, line 27
> > <https://reviewboard.asterisk.org/r/3357/diff/1/?file=56030#file56030line27>
> >
> >     You have some problematic XML in this line here. I suggest changing the backslash-escaped double-quotes to """ and the > in the regex to
> >     &
> >     g
> >     t
> >     ;
> >     
> >     Sorry, I couldn't put those characters together because reviewboard actually decodes the string into the the > symbol :)

This was brought in 1:1 from subscriptions/presence/digium_presence's own subscribe xml. I tried swapping the escaped quotes with "e;, but that seems to have broken things.  Worked well enough with &gt(;), but that wasn't escaped anyway. I'll just leave this as it is for now.


- Jonathan


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


On March 14, 2014, 2:13 p.m., Jonathan Rose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3357/
> -----------------------------------------------------------
> 
> (Updated March 14, 2014, 2:13 p.m.)
> 
> 
> Review request for Asterisk Developers, Kevin Harwell and Matt Jordan.
> 
> 
> 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/run-test 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/tests.yaml 4836 
>   /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/unallowed/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/unallowed/sipp/subscribe.xml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/unallowed/run-test PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/unallowed/configs/ast1/pjsip.conf PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/unallowed/configs/ast1/extensions.conf PRE-CREATION 
>   /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/presence/below_min_expiry/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/below_min_expiry/sipp/subscribe.xml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/below_min_expiry/run-test PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/below_min_expiry/configs/ast1/pjsip.conf PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/below_min_expiry/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/run-test 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 
> 
> 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/20140326/ebe346b9/attachment-0001.html>