[Asterisk-code-review] testsuite: fix periodic 180 after 183 failures (testsuite[20])

Friendly Automation asteriskteam at digium.com
Mon Sep 26 07:23:02 CDT 2022


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

Change subject: testsuite: fix periodic 180 after 183 failures
......................................................................

testsuite: fix periodic 180 after 183 failures

allow_sending_180_after_183 tests were sending the 183,
180 and 200 all immediately without delay. This can cause
asterisk to receive all three before setting up the A
leg, leading to unpredictable results.

ASTERISK-30231

Change-Id: I78df66edb39452a76408cd9b583f17a4448ad754
---
M tests/channels/pjsip/allow_sending_180_after_183/disabled/sipp/B_PARTY.xml
M tests/channels/pjsip/allow_sending_180_after_183/enabled/sipp/B_PARTY.xml
2 files changed, 18 insertions(+), 0 deletions(-)

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




diff --git a/tests/channels/pjsip/allow_sending_180_after_183/disabled/sipp/B_PARTY.xml b/tests/channels/pjsip/allow_sending_180_after_183/disabled/sipp/B_PARTY.xml
index ff8270b..d6b9c7b 100644
--- a/tests/channels/pjsip/allow_sending_180_after_183/disabled/sipp/B_PARTY.xml
+++ b/tests/channels/pjsip/allow_sending_180_after_183/disabled/sipp/B_PARTY.xml
@@ -50,6 +50,7 @@
 			Content-Length: 0
 
 		]]></send>
+	<pause milliseconds="1000"/>
 	<send retrans="500"><![CDATA[
 
 			SIP/2.0 200 OK
diff --git a/tests/channels/pjsip/allow_sending_180_after_183/enabled/sipp/B_PARTY.xml b/tests/channels/pjsip/allow_sending_180_after_183/enabled/sipp/B_PARTY.xml
index 89fc0ea..eb13958 100644
--- a/tests/channels/pjsip/allow_sending_180_after_183/enabled/sipp/B_PARTY.xml
+++ b/tests/channels/pjsip/allow_sending_180_after_183/enabled/sipp/B_PARTY.xml
@@ -50,6 +50,7 @@
 			Content-Length: 0
 
 			]]></send>
+	<pause milliseconds="1000"/>
 	<send retrans="500"><![CDATA[
 
 			SIP/2.0 200 OK

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

Gerrit-Project: testsuite
Gerrit-Branch: 20
Gerrit-Change-Id: I78df66edb39452a76408cd9b583f17a4448ad754
Gerrit-Change-Number: 19334
Gerrit-PatchSet: 1
Gerrit-Owner: Michael Bradeen <mbradeen at sangoma.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220926/78239f09/attachment.html>


More information about the asterisk-code-review mailing list