[Asterisk-code-review] ast_restart: Give time for handling of NOTIFY response. (testsuite[master])
Friendly Automation
asteriskteam at digium.com
Mon Mar 22 10:05:01 CDT 2021
Friendly Automation has submitted this change. ( https://gerrit.asterisk.org/c/testsuite/+/15684 )
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/+/15684
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Change-Id: I2d791fa14136fce94df3d1405b5a15b31c79c26c
Gerrit-Change-Number: 15684
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/db9778ee/attachment.html>
More information about the asterisk-code-review
mailing list