[Asterisk-code-review] rest_api: Tweak subscribe-all test timing to allow qualify. (testsuite[17])
Friendly Automation
asteriskteam at digium.com
Thu Mar 26 11:01:49 CDT 2020
Friendly Automation has submitted this change. ( https://gerrit.asterisk.org/c/testsuite/+/14051 )
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/+/14051
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings
Gerrit-Project: testsuite
Gerrit-Branch: 17
Gerrit-Change-Id: I428669462c22c6f9a19e9aa228fffc67c86bb054
Gerrit-Change-Number: 14051
Gerrit-PatchSet: 2
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/6d797960/attachment.html>
More information about the asterisk-code-review
mailing list