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

Joshua Colp asteriskteam at digium.com
Mon Mar 22 08:28:51 CDT 2021


Joshua Colp has uploaded this change for review. ( 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(-)



  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/63/15663/1

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: 1
Gerrit-Owner: Joshua Colp <jcolp at sangoma.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210322/7f7b00f7/attachment.html>


More information about the asterisk-code-review mailing list