> On Aug. 1, 2014, 4:23 p.m., Jonathan Rose wrote: > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/full_state/full_state.py, > > lines 37-38 > > <https://reviewboard.asterisk.org/r/3873/diff/1/?file=65767#file65767line37> > > > > Alright, this right here turns out to introduce an occasional timing > > based problem where state changes prior to the subscription actually being > > able to react to this. I've managed to fix this using reactor.callLater, > > so I'll be adding that to all of the tests that currently rely on this > > construct.
Ok, so suffice to say the change here is the same as the others... but I'm not updating this particular review since it's exactly the same otherwise and I don't want to deal with the hassle of pulling everything else out of my working folder so that I can make a diff without all the other tests not included here. Just pretend that I use reactor.callLater to queue up the AMI actions in this case instead of calling them immediately if you review this. - Jonathan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3873/#review12964 ----------------------------------------------------------- On July 31, 2014, 12:02 p.m., Jonathan Rose wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3873/ > ----------------------------------------------------------- > > (Updated July 31, 2014, 12:02 p.m.) > > > Review request for Asterisk Developers and Mark Michelson. > > > Bugs: ASTERISK-23870 and ASTERISK-23872 > https://issues.asterisk.org/jira/browse/ASTERISK-23870 > https://issues.asterisk.org/jira/browse/ASTERISK-23872 > > > Repository: testsuite > > > Description > ------- > > Continued from: https://reviewboard.asterisk.org/r/3673/ > > > This changeset implements the nominal resource list tests outlined on this > > page: > > https://wiki.asterisk.org/wiki/display/AST/Resource+List+Subscription+Test+Plan > > > There are six tests: > > 1. Subscription Establishment: Simply ensures that Asterisk responds with a > > 200 OK when we > > subscribe to a resource list and that the 200 OK has a Require: eventlist > > header in it. > > 2. Initial NOTIFY: Validates the initial NOTIFY body that Asterisk sends > > when subscribing > > to a resource list. > > 3. Full State: Establishes a subscription to a resource list and then > > changes the state of > > a resource. Ensures that Asterisk sends a NOTIFY with full state of the > > list. > > 4. Partial State: Establishes a subscription to a resource list and then > > changes the state > > of a resource. Ensures that Asterisk sends a NOTIFY with partial state, > > with only the > > state of the resource whose state was changed. > > 5. Resubscription Full State: Establishes a subscription and then > > resubscribes. Ensures > > that even though partial state is configured, the NOTIFY that Asterisk > > sends in response > > to the resubscription has full state of the list. > > 6. Termination Full State: Establishes a subscription and then terminates > > the > > subscription. Ensures that even though partial state is configured, the > > NOTIFY that > > Asterisk sends in response to the termination has full state of the list. > > Since that review was posted, I've also added support for lists of lists and > MWI bodies to the RLSIntegrity and pcap libraries. > > > Diffs > ----- > > /asterisk/trunk/tests/channels/pjsip/subscriptions/tests.yaml 5316 > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/tests.yaml > PRE-CREATION > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/rls_integrity.py > PRE-CREATION > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/tests.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/tests.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/termination_full_state/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/termination_full_state/termination.py > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/termination_full_state/sipp/termination.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/termination_full_state/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/termination_full_state/configs/ast1/extensions.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/subscription_establishment/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/subscription_establishment/sipp/list_subscribe.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/subscription_establishment/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/subscription_establishment/configs/ast1/extensions.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/resubscribe_full_state/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/resubscribe_full_state/sipp/resubscribe.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/resubscribe_full_state/resubscribe.py > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/resubscribe_full_state/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/resubscribe_full_state/configs/ast1/extensions.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/partial_state/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/partial_state/sipp/list_subscribe.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/partial_state/partial_state.py > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/partial_state/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/partial_state/configs/ast1/extensions.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/initial_notify/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/initial_notify/notify.py > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/initial_notify/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/initial_notify/configs/ast1/extensions.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/full_state/test-config.yaml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/full_state/sipp/list_subscribe.xml > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/full_state/full_state.py > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/full_state/configs/ast1/pjsip.conf > PRE-CREATION > > /asterisk/trunk/tests/channels/pjsip/subscriptions/rls/lists/nominal/presence/full_state/configs/ast1/extensions.conf > PRE-CREATION > /asterisk/trunk/lib/python/asterisk/pcap.py 5316 > > Diff: https://reviewboard.asterisk.org/r/3873/diff/ > > > Testing > ------- > > I've been running the tests against mmichelson's rls-rlmi branch to ensure > that they pass when expected to pass and fail as the resource lists differ > from what is actually received. I've also written a whole set of lists of > lists tests for presence which will be covered in another review and a list > test for MWI that will be part of a separate review as well. > > > 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
