[Asterisk-code-review] pjsip/transfers/blind transfer/off nominal/transferer reinvi... (testsuite[master])

Anonymous Coward asteriskteam at digium.com
Mon Sep 5 11:54:08 CDT 2016


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

Change subject: pjsip/transfers/blind_transfer/off_nominal/transferer_reinvite: Add pause.
......................................................................


pjsip/transfers/blind_transfer/off_nominal/transferer_reinvite: Add pause.

This test exercises specific code that sends a BYE
if a re-INVITE is received after a channel has been torn
down. Unfortunately when the re-INVITE is sent in the test
there is no guarantee that this condition has actually
happened yet, in particular on slower systems. This change
introduces a pause so that the channel reaches the proper
state. If it has not yet reached the state a BYE will
still occur but it can take longer than the test expects.

Change-Id: Ifd23662fb6a42a6e9311a816875ba2d188df0851
---
M tests/channels/pjsip/transfers/blind_transfer/off_nominal/transferer_reinvite/sipp/transferer.xml
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Anonymous Coward #1000019: Verified
  Matt Jordan: Looks good to me, approved



diff --git a/tests/channels/pjsip/transfers/blind_transfer/off_nominal/transferer_reinvite/sipp/transferer.xml b/tests/channels/pjsip/transfers/blind_transfer/off_nominal/transferer_reinvite/sipp/transferer.xml
index 7558880..1767aef 100644
--- a/tests/channels/pjsip/transfers/blind_transfer/off_nominal/transferer_reinvite/sipp/transferer.xml
+++ b/tests/channels/pjsip/transfers/blind_transfer/off_nominal/transferer_reinvite/sipp/transferer.xml
@@ -121,6 +121,8 @@
     ]]>
   </send>
 
+  <pause milliseconds="2000" />
+
   <send retrans="500">
     <![CDATA[
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifd23662fb6a42a6e9311a816875ba2d188df0851
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list