[Asterisk-code-review] res pjsip - chan is avail fails if running with sipp version... (testsuite[master])

Jenkins2 asteriskteam at digium.com
Mon Dec 11 05:28:51 CST 2017


Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/7490 )

Change subject: res_pjsip - chan_is_avail fails if running with sipp version < 3.5.0
......................................................................

res_pjsip - chan_is_avail fails if running with sipp version < 3.5.0

sipp 3.5.0 made it so the "-aa" option responds to OPTION messages. However,
versions prior to that would not. The test config incorrectly listed the
sipp an older version for the dependency.

This patch bumps the sipp dependency version to the appropriate value, so now
anyone running this test will skip the test instead of failing.

Change-Id: I5147b726cc1a07aa7293e579bbb0e6041d1c1b66
---
M tests/channels/pjsip/dialplan_functions/chan_is_avail/test-config.yaml
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Corey Farrell: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved
  Jenkins2: Approved for Submit



diff --git a/tests/channels/pjsip/dialplan_functions/chan_is_avail/test-config.yaml b/tests/channels/pjsip/dialplan_functions/chan_is_avail/test-config.yaml
index 136fca4..5c03775 100644
--- a/tests/channels/pjsip/dialplan_functions/chan_is_avail/test-config.yaml
+++ b/tests/channels/pjsip/dialplan_functions/chan_is_avail/test-config.yaml
@@ -9,7 +9,7 @@
     dependencies:
         - buildoption: 'TEST_FRAMEWORK'
         - sipp :
-            version : 'v3.0'
+            version : 'v3.5.0'
         - asterisk : 'res_pjsip'
         - asterisk : 'chan_pjsip'
         - asterisk : 'res_pjsip_session'

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

Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I5147b726cc1a07aa7293e579bbb0e6041d1c1b66
Gerrit-Change-Number: 7490
Gerrit-PatchSet: 1
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20171211/54aaa38d/attachment.html>


More information about the asterisk-code-review mailing list