[Asterisk-code-review] rest_api: Tweak subscribe-all test timing to allow qualify. (testsuite[master])

Friendly Automation asteriskteam at digium.com
Thu Mar 26 11:01:53 CDT 2020


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

Change subject: rest_api: Tweak subscribe-all test timing to allow qualify.
......................................................................

rest_api: Tweak subscribe-all test timing to allow qualify.

This test would sometimes fail as the qualify would not occur in
a timely fashion due to randomized initial qualifying. This change
tweaks the qualify frequency to 1 second to have it occur sooner
and also prolongs the test some to give more time.

Change-Id: I428669462c22c6f9a19e9aa228fffc67c86bb054
---
M tests/rest_api/applications/subscribe-all/configs/ast1/extensions.conf
M tests/rest_api/applications/subscribe-all/test-config.yaml
2 files changed, 2 insertions(+), 2 deletions(-)

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



diff --git a/tests/rest_api/applications/subscribe-all/configs/ast1/extensions.conf b/tests/rest_api/applications/subscribe-all/configs/ast1/extensions.conf
index 62dea66..0e114bd 100644
--- a/tests/rest_api/applications/subscribe-all/configs/ast1/extensions.conf
+++ b/tests/rest_api/applications/subscribe-all/configs/ast1/extensions.conf
@@ -10,5 +10,5 @@
 
 exten => 1000,1,NoOp()
  same => n,Answer()
- same => n,Wait(1)
+ same => n,Wait(5)
  same => n,Hangup()
diff --git a/tests/rest_api/applications/subscribe-all/test-config.yaml b/tests/rest_api/applications/subscribe-all/test-config.yaml
index d48b310..e8f9c51 100644
--- a/tests/rest_api/applications/subscribe-all/test-config.yaml
+++ b/tests/rest_api/applications/subscribe-all/test-config.yaml
@@ -52,7 +52,7 @@
                 method: 'put'
                 uri: 'asterisk/config/dynamic/res_pjsip/aor/alice'
                 body: { 'fields': [ { 'attribute': 'contact', 'value': 'sip:alice at 127.0.0.1:5060;transport=udp' },
-                                    { 'attribute': 'qualify_frequency', value: '5' } ] }
+                                    { 'attribute': 'qualify_frequency', value: '1' } ] }
                 expect: 200
             -
                 method: 'put'

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

Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Change-Id: I428669462c22c6f9a19e9aa228fffc67c86bb054
Gerrit-Change-Number: 14052
Gerrit-PatchSet: 1
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/20200326/a8cfca95/attachment-0001.html>


More information about the asterisk-code-review mailing list