[Asterisk-code-review] ast_restart: Give time for handling of NOTIFY response. (testsuite[18])

Friendly Automation asteriskteam at digium.com
Mon Mar 22 10:03:55 CDT 2021


Friendly Automation has submitted this change. ( https://gerrit.asterisk.org/c/testsuite/+/15663 )

Change subject: ast_restart: Give time for handling of NOTIFY response.
......................................................................

ast_restart: Give time for handling of NOTIFY response.

In some cases the SIPp response to the NOTIFY in this test
is not handled quickly resulting in retransmission of the
NOTIFY. This change adds a pause to give time to process
the response.

ASTERISK-29361

Change-Id: I2d791fa14136fce94df3d1405b5a15b31c79c26c
---
M tests/channels/pjsip/subscriptions/ast_restart/sipp/subscribe.xml
1 file changed, 4 insertions(+), 1 deletion(-)

Approvals:
  George Joseph: Looks good to me, approved
  Friendly Automation: Approved for Submit



diff --git a/tests/channels/pjsip/subscriptions/ast_restart/sipp/subscribe.xml b/tests/channels/pjsip/subscriptions/ast_restart/sipp/subscribe.xml
index 1ed154d..5aca80e 100644
--- a/tests/channels/pjsip/subscriptions/ast_restart/sipp/subscribe.xml
+++ b/tests/channels/pjsip/subscriptions/ast_restart/sipp/subscribe.xml
@@ -24,7 +24,7 @@
 		<action>
 			<ereg regexp="(;tag=.*)" header="To:" search_in="hdr" check_it="true" assign_to="to_tag"/>
 		</action>
-	</rcv>
+	</recv>
 
 	<!-- NOTIFY for the initial subscription -->
 	<recv request="NOTIFY" crlf="true">
@@ -107,6 +107,9 @@
 		]]>
 	</send>
 
+	<!-- Wait for Asterisk to process the NOTIFY response -->
+	<pause milliseconds="2000" />
+
 	<!-- Unsubscribe  -->
 	<send retrans="500">
 		<![CDATA[

-- 
To view, visit https://gerrit.asterisk.org/c/testsuite/+/15663
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: testsuite
Gerrit-Branch: 18
Gerrit-Change-Id: I2d791fa14136fce94df3d1405b5a15b31c79c26c
Gerrit-Change-Number: 15663
Gerrit-PatchSet: 2
Gerrit-Owner: Joshua Colp <jcolp at sangoma.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210322/36489bc9/attachment-0001.html>


More information about the asterisk-code-review mailing list