[Asterisk-code-review] channels/pjsip/qualify/qualify timeout: Increase sipp wait t... (testsuite[master])

Anonymous Coward asteriskteam at digium.com
Sat Nov 21 17:59:50 CST 2015


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

Change subject: channels/pjsip/qualify/qualify_timeout: Increase sipp wait time.
......................................................................


channels/pjsip/qualify/qualify_timeout: Increase sipp wait time.

Some agents can be quite slow at times causing things to not
exactly execute when they are expected to. In the case of the
qualify_timeout test there was little time left between marking a
contact as unreachable and receiving the 200 OK. This change
increases the time to give Asterisk more time to mark it as
unreachable.

Change-Id: I7dd0591453ebce18514894bf46ed23285afc6705
---
M tests/channels/pjsip/qualify/qualify_timeout/test-config.yaml
1 file changed, 3 insertions(+), 3 deletions(-)

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



diff --git a/tests/channels/pjsip/qualify/qualify_timeout/test-config.yaml b/tests/channels/pjsip/qualify/qualify_timeout/test-config.yaml
index eebd158..a5460d4 100644
--- a/tests/channels/pjsip/qualify/qualify_timeout/test-config.yaml
+++ b/tests/channels/pjsip/qualify/qualify_timeout/test-config.yaml
@@ -1,7 +1,7 @@
 testinfo:
     summary:     'Test qualify timeout'
     description: |
-        'With qualify_timeout = 2, and sipp pausing for 3 seconds, send a qualify to a
+        'With qualify_timeout = 2, and sipp pausing for 6 seconds, send a qualify to a
         contact and verify that the contact is marked Unavailable.
 
 test-modules:
@@ -21,11 +21,11 @@
 
 test-object-config:
     fail-on-any: False
-    reactor-timeout: 10
+    reactor-timeout: 15
     test-iterations:
         -
             scenarios:
-                - { 'key-args': {'scenario': 'options.xml', '-i': '127.0.0.1', '-d': '3000', '-p': '5061'} }
+                - { 'key-args': {'scenario': 'options.xml', '-i': '127.0.0.1', '-d': '6000', '-p': '5061'} }
 
 ami-config:
     -

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7dd0591453ebce18514894bf46ed23285afc6705
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