----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3288/#review11102 -----------------------------------------------------------
/asterisk/trunk/tests/channels/pjsip/subscriptions/presence/resubscribe/configs/ast1/pjsip.conf <https://reviewboard.asterisk.org/r/3288/#comment20750> These lines are unnecessary. /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/resubscribe/sipp/subscribe.xml <https://reviewboard.asterisk.org/r/3288/#comment20752> You can't reuse the "state" variable. It needs to have a different name. /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/unsubscribe/unsubscribe.py <https://reviewboard.asterisk.org/r/3288/#comment20754> This will should be dropped once the path modificaiton is gone. /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/unsubscribe/unsubscribe.py <https://reviewboard.asterisk.org/r/3288/#comment20753> This is unnecessary. This patch may also be missing some files that were moved. - opticron On March 4, 2014, 12:25 a.m., jbigelow wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3288/ > ----------------------------------------------------------- > > (Updated March 4, 2014, 12:25 a.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-23344 > https://issues.asterisk.org/jira/browse/ASTERISK-23344 > > > Repository: testsuite > > > Description > ------- > > These tests cover re-subscribe and un-subscribe for presence using PJSIP. > > Note: This review request also re-organizes existing subscription tests. > > Tests: > * unsubscribe - Nominal test using the pjsua pluggable module where "alice" > subscribes to "bob. Alice then un-subscribes (expiration of '0') and the pass > result is set based upon the subscription state being reported as terminated. > Otherwise the fail result is set. > > * resubscribe - Nominal test that uses SIPp (aka. "alice") to subscribe to > "bob" with an expiration of 300. Upon successfully subscribing, alice then > subscribes to bob again but with an expiration of 600. This ensures the > re-subscribe is successful per receiving a 200 response and a Notify with the > expected expiration in the Subscription-State header. > > > Diffs > ----- > > /asterisk/trunk/tests/channels/pjsip/tests.yaml 4765 > /asterisk/trunk/tests/channels/pjsip/subscriptions/tests.yaml PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/unsubscribe/unsubscribe.py > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/unsubscribe/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/unsubscribe/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/unsubscribe/configs/ast1/modules.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/unsubscribe/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/tests.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/resubscribe/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/resubscribe/sipp/subscribe.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/resubscribe/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/resubscribe/configs/ast1/modules.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/presence/resubscribe/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/tests/channels/pjsip/subscriptions/mwi/tests.yaml > PRE-CREATION > /asterisk/trunk/tests/channels/pjsip/subscribe/tests.yaml 4765 > /asterisk/trunk/tests/channels/pjsip/subscribe/missing_aor/test-config.yaml > 4765 > > /asterisk/trunk/tests/channels/pjsip/subscribe/missing_aor/sipp/subscribe.xml > 4765 > /asterisk/trunk/tests/channels/pjsip/subscribe/missing_aor/run-test 4765 > > /asterisk/trunk/tests/channels/pjsip/subscribe/missing_aor/configs/ast1/pjsip.conf > 4765 > > /asterisk/trunk/tests/channels/pjsip/subscribe/missing_aor/configs/ast1/modules.conf > 4765 > > /asterisk/trunk/tests/channels/pjsip/subscribe/digium_presence/test-config.yaml > 4765 > > /asterisk/trunk/tests/channels/pjsip/subscribe/digium_presence/sipp/subscribe.xml > 4765 > /asterisk/trunk/tests/channels/pjsip/subscribe/digium_presence/run-test > 4765 > > /asterisk/trunk/tests/channels/pjsip/subscribe/digium_presence/configs/ast1/pjsip.conf > 4765 > > /asterisk/trunk/tests/channels/pjsip/subscribe/digium_presence/configs/ast1/extensions.conf > 4765 > /asterisk/trunk/tests/channels/pjsip/presence_xpidf/test-config.yaml 4765 > /asterisk/trunk/tests/channels/pjsip/presence_xpidf/state_check.py 4765 > /asterisk/trunk/tests/channels/pjsip/presence_xpidf/configs/ast1/pjsip.conf > 4765 > > /asterisk/trunk/tests/channels/pjsip/presence_xpidf/configs/ast1/modules.conf > 4765 > > /asterisk/trunk/tests/channels/pjsip/presence_xpidf/configs/ast1/extensions.conf > 4765 > /asterisk/trunk/tests/channels/pjsip/presence_pidf/test-config.yaml 4765 > /asterisk/trunk/tests/channels/pjsip/presence_pidf/state_check.py 4765 > /asterisk/trunk/tests/channels/pjsip/presence_pidf/configs/ast1/pjsip.conf > 4765 > > /asterisk/trunk/tests/channels/pjsip/presence_pidf/configs/ast1/modules.conf > 4765 > > /asterisk/trunk/tests/channels/pjsip/presence_pidf/configs/ast1/extensions.conf > 4765 > /asterisk/trunk/tests/channels/pjsip/mwi/test-config.yaml 4765 > /asterisk/trunk/tests/channels/pjsip/mwi/mwi_check.py 4765 > /asterisk/trunk/tests/channels/pjsip/mwi/configs/ast1/pjsip.conf 4765 > /asterisk/trunk/tests/channels/pjsip/mwi/configs/ast1/modules.conf 4765 > > Diff: https://reviewboard.asterisk.org/r/3288/diff/ > > > Testing > ------- > > * Ensured tests pass on multiple executions > * Ensured the testsuite & Asterisk logs looked good. > > > Thanks, > > jbigelow > >
-- _____________________________________________________________________ -- 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
