----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3357/#review11391 -----------------------------------------------------------
The tests look good, but Matt said to go ahead and add an MWI version of the no_accept_header test. - Mark Michelson On March 26, 2014, 8:04 p.m., Jonathan Rose wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3357/ > ----------------------------------------------------------- > > (Updated March 26, 2014, 8:04 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/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/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 > >
-- _____________________________________________________________________ -- 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
