[Asterisk-code-review] tests/channels/pjsip/hold: Increase reactor timeout. (testsuite[13])

Friendly Automation asteriskteam at digium.com
Tue Dec 11 09:34:44 CST 2018


Friendly Automation has submitted this change and it was merged. ( https://gerrit.asterisk.org/10783 )

Change subject: tests/channels/pjsip/hold: Increase reactor timeout.
......................................................................

tests/channels/pjsip/hold: Increase reactor timeout.

In some execution runs Jenkins2 takes a long period of
time to start the SIPp application and execute the
scenario. This can run into the 30 second reactor timeout
of the test and cause a failure.

This change bumps the reactor timeout to 60 seconds to
give more time for SIPp to start.

Change-Id: I62c6aec6d3687eec651f4e4393631c54dbd366f0
---
M tests/channels/pjsip/hold/run-test
M tests/channels/pjsip/hold_ice/run-test
M tests/channels/pjsip/hold_inactive/run-test
3 files changed, 3 insertions(+), 0 deletions(-)

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



diff --git a/tests/channels/pjsip/hold/run-test b/tests/channels/pjsip/hold/run-test
index cb46fc2..9b32ffc 100755
--- a/tests/channels/pjsip/hold/run-test
+++ b/tests/channels/pjsip/hold/run-test
@@ -65,6 +65,7 @@
                                   '-i': '127.0.0.3', '-p': '5060',
                                   '-inf': INJECT_FILE}]
 
+        self.reactor_timeout = 60
         self.passed = True
         self.moh_start_events = 0
         self.moh_stop_events = 0
diff --git a/tests/channels/pjsip/hold_ice/run-test b/tests/channels/pjsip/hold_ice/run-test
index 878adbd..fa092fa 100755
--- a/tests/channels/pjsip/hold_ice/run-test
+++ b/tests/channels/pjsip/hold_ice/run-test
@@ -37,6 +37,7 @@
                                       '-p': '6081',
                                       '-inf': INJECT_FILE}
 
+        self.reactor_timeout = 60
         self.moh_start_event = False
         self.moh_stop_event = False
         self.check_list_success_events = 0
diff --git a/tests/channels/pjsip/hold_inactive/run-test b/tests/channels/pjsip/hold_inactive/run-test
index 353eb99..9a1a301 100755
--- a/tests/channels/pjsip/hold_inactive/run-test
+++ b/tests/channels/pjsip/hold_inactive/run-test
@@ -59,6 +59,7 @@
                                   '-i': '127.0.0.3', '-p': '5060',
                                   '-inf': INJECT_FILE}]
 
+        self.reactor_timeout = 60
         self.passed = True
         self.moh_start_events = 0
         self.moh_stop_events = 0

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

Gerrit-Project: testsuite
Gerrit-Branch: 13
Gerrit-MessageType: merged
Gerrit-Change-Id: I62c6aec6d3687eec651f4e4393631c54dbd366f0
Gerrit-Change-Number: 10783
Gerrit-PatchSet: 2
Gerrit-Owner: Joshua C. Colp <jcolp at digium.com>
Gerrit-Reviewer: Friendly Automation (1000185)
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua C. Colp <jcolp at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20181211/2793ba5b/attachment-0001.html>


More information about the asterisk-code-review mailing list