[Asterisk-code-review] qualify timeout: Increase qualify timeout. (testsuite[master])

[Anonymous Coward]

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

Change subject: qualify_timeout: Increase qualify timeout.
......................................................................


qualify_timeout: Increase qualify timeout.

The qualify_timeout test fails under Jenkins test agents occasionally.
After investigating, this was because the qualify was timing out before
the AMI session was connected. Since the AMI session was not connected,
the required ConnectionStatus event would not get sent, meaning the test
would fail.

By increasing the qualify timeout, we can better ensure that the
qualify_timeout will happen after AMI has been connected, and we will
see the event we expect to see.

Change-Id: If9a011f432e30e406f6dace4c8e2712aedb593b8
---
M tests/channels/pjsip/qualify/qualify_timeout/configs/ast1/pjsip.conf
M tests/channels/pjsip/qualify/qualify_timeout/test-config.yaml
2 files changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Anonymous Coward #1000019: Verified
  Joshua Colp: Looks good to me, approved



diff --git a/tests/channels/pjsip/qualify/qualify_timeout/configs/ast1/pjsip.conf b/tests/channels/pjsip/qualify/qualify_timeout/configs/ast1/pjsip.conf
index 14ad57e..3060582 100644
--- a/tests/channels/pjsip/qualify/qualify_timeout/configs/ast1/pjsip.conf
+++ b/tests/channels/pjsip/qualify/qualify_timeout/configs/ast1/pjsip.conf
@@ -15,5 +15,5 @@
 [sipp]
 type=aor
 contact=sip:127.0.0.1:5061
-qualify_frequency=6000
-qualify_timeout = 2
+qualify_frequency=10000
+qualify_timeout = 5
diff --git a/tests/channels/pjsip/qualify/qualify_timeout/test-config.yaml b/tests/channels/pjsip/qualify/qualify_timeout/test-config.yaml
index a5460d4..59a6372 100644
--- a/tests/channels/pjsip/qualify/qualify_timeout/test-config.yaml
+++ b/tests/channels/pjsip/qualify/qualify_timeout/test-config.yaml
@@ -21,11 +21,11 @@
 
 test-object-config:
     fail-on-any: False
-    reactor-timeout: 15
+    reactor-timeout: 25
     test-iterations:
         -
             scenarios:
-                - { 'key-args': {'scenario': 'options.xml', '-i': '127.0.0.1', '-d': '6000', '-p': '5061'} }
+                - { 'key-args': {'scenario': 'options.xml', '-i': '127.0.0.1', '-d': '10000', '-p': '5061'} }
 
 ami-config:
     -

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

Gerrit-MessageType: merged
Gerrit-Change-Id: If9a011f432e30e406f6dace4c8e2712aedb593b8
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>