> 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 >(;), 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 > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
