[Asterisk-code-review] pjsip/ami/show subscriptions: Fill out SIPp scenario more. (testsuite[master])

Anonymous Coward asteriskteam at digium.com
Thu Mar 31 14:38:35 CDT 2016


Anonymous Coward #1000019 has submitted this change and it was merged.

Change subject: pjsip/ami/show_subscriptions: Fill out SIPp scenario more.
......................................................................


pjsip/ami/show_subscriptions: Fill out SIPp scenario more.

In some situations it was possible for the SIPp scenario to end
before the test had really finished, causing Asterisk to shutdown
and the test to fail.

This change fills out the SIPp scenario more to expect another
NOTIFY message and also adds a pause to give the test time to
get inbound subscription details.

Change-Id: I429855159c9f01105c5cc1c111e7c9009fc37d55
---
M tests/channels/pjsip/ami/show_subscriptions/sipp/subscribe.xml
1 file changed, 19 insertions(+), 0 deletions(-)

Approvals:
  Mark Michelson: Looks good to me, but someone else must approve
  Anonymous Coward #1000019: Verified
  Joshua Colp: Looks good to me, approved



diff --git a/tests/channels/pjsip/ami/show_subscriptions/sipp/subscribe.xml b/tests/channels/pjsip/ami/show_subscriptions/sipp/subscribe.xml
index c2ef4a6..328b378 100644
--- a/tests/channels/pjsip/ami/show_subscriptions/sipp/subscribe.xml
+++ b/tests/channels/pjsip/ami/show_subscriptions/sipp/subscribe.xml
@@ -58,4 +58,23 @@
   </send>
 
   <recv response="200" rtd="true" />
+
+  <recv request="NOTIFY" crlf="true" />
+
+  <send>
+    <![CDATA[
+
+      SIP/2.0 200 OK
+      [last_Via:]
+      [last_From:]
+      [last_To:]
+      [last_Call-ID:]
+      [last_CSeq:]
+      Contact: <sip:[local_ip]:[local_port];transport=[transport]>
+      Content-Length: 0
+
+    ]]>
+  </send>
+
+  <pause milliseconds="4000" />
 </scenario>

-- 
To view, visit https://gerrit.asterisk.org/2518
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I429855159c9f01105c5cc1c111e7c9009fc37d55
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>



More information about the asterisk-code-review mailing list