[Asterisk-code-review] pjsip/transfer: Remove obsolete sipp options (testsuite[13])
Jenkins2
asteriskteam at digium.com
Tue Jul 24 09:36:57 CDT 2018
Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/9644 )
Change subject: pjsip/transfer: Remove obsolete sipp options
......................................................................
pjsip/transfer: Remove obsolete sipp options
Some of the blind transfer hold tests were using -min_rtp_port and
-max_rtp_port sipp options. These are now obsolete and cause sipp
to comaplin and the tests to fail.
Removed those options. Tests still work fine.
Change-Id: I3c3f82c7d6aec5fe3acaba1e5dd1d54edba3ba72
---
M tests/channels/pjsip/transfers/blind_transfer/callee_with_hold/test-config.yaml
M tests/channels/pjsip/transfers/blind_transfer/caller_with_hold/test-config.yaml
M tests/channels/pjsip/transfers/blind_transfer/caller_with_hold_drop_options/test-config.yaml
3 files changed, 18 insertions(+), 18 deletions(-)
Approvals:
Jenkins2: Looks good to me, approved; Approved for Submit
diff --git a/tests/channels/pjsip/transfers/blind_transfer/callee_with_hold/test-config.yaml b/tests/channels/pjsip/transfers/blind_transfer/callee_with_hold/test-config.yaml
index 92935ca..c007b3d 100644
--- a/tests/channels/pjsip/transfers/blind_transfer/callee_with_hold/test-config.yaml
+++ b/tests/channels/pjsip/transfers/blind_transfer/callee_with_hold/test-config.yaml
@@ -40,25 +40,25 @@
-
scenarios:
# Charlie receives transfered call and sends audio to Alice.
- - { 'key-args': {'scenario': 'charlie.xml', '-p': '5060', '-i': '127.0.0.4', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.4'},
+ - { 'key-args': {'scenario': 'charlie.xml', '-p': '5060', '-i': '127.0.0.4', '-timeout': '20s', '-mi': '127.0.0.4'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
# Bob receives call from Alice and sends audio. Bob then attempts to blind transfer Alice to Charlie.
- - { 'key-args': {'scenario': 'bob.xml', '-p': '5060', '-i': '127.0.0.3', '-s': 'alice', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.3'},
+ - { 'key-args': {'scenario': 'bob.xml', '-p': '5060', '-i': '127.0.0.3', '-s': 'alice', '-timeout': '20s', '-mi': '127.0.0.3'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
# Alice calls Bob and sends audio.
- - { 'key-args': {'scenario': 'alice.xml', '-p': '5060', '-i': '127.0.0.2', '-s': 'bob', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.2'},
+ - { 'key-args': {'scenario': 'alice.xml', '-p': '5060', '-i': '127.0.0.2', '-s': 'bob', '-timeout': '20s', '-mi': '127.0.0.2'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
# Second iteration
-
scenarios:
# Charlie receives transfered call and sends audio to Alice.
- - { 'key-args': {'scenario': 'charlie.xml', '-p': '5060', '-i': '127.0.0.4', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.4'},
+ - { 'key-args': {'scenario': 'charlie.xml', '-p': '5060', '-i': '127.0.0.4', '-timeout': '20s', '-mi': '127.0.0.4'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
# Bob receives call from Alice and sends audio. Bob then attempts to blind transfer Alice to Charlie.
- - { 'key-args': {'scenario': 'bob.xml', '-p': '5060', '-i': '127.0.0.3', '-s': 'alice', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.3'},
+ - { 'key-args': {'scenario': 'bob.xml', '-p': '5060', '-i': '127.0.0.3', '-s': 'alice', '-timeout': '20s', '-mi': '127.0.0.3'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
# Alice calls Bob and sends audio.
- - { 'key-args': {'scenario': 'alice.xml', '-p': '5060', '-i': '127.0.0.2', '-s': 'bob_two', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.2'},
+ - { 'key-args': {'scenario': 'alice.xml', '-p': '5060', '-i': '127.0.0.2', '-s': 'bob_two', '-timeout': '20s', '-mi': '127.0.0.2'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
ami-config:
diff --git a/tests/channels/pjsip/transfers/blind_transfer/caller_with_hold/test-config.yaml b/tests/channels/pjsip/transfers/blind_transfer/caller_with_hold/test-config.yaml
index 4b77f4b..26765f5 100644
--- a/tests/channels/pjsip/transfers/blind_transfer/caller_with_hold/test-config.yaml
+++ b/tests/channels/pjsip/transfers/blind_transfer/caller_with_hold/test-config.yaml
@@ -41,25 +41,25 @@
-
scenarios:
# Charlie receives transfered call and sends audio to Bob.
- - { 'key-args': {'scenario': 'charlie.xml', '-p': '5060', '-i': '127.0.0.4', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.4'},
+ - { 'key-args': {'scenario': 'charlie.xml', '-p': '5060', '-i': '127.0.0.4', '-timeout': '20s', '-mi': '127.0.0.4'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
# Bob receives call from Alice and sends audio.
- - { 'key-args': {'scenario': 'bob.xml', '-p': '5060', '-i': '127.0.0.3', '-s': 'alice', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.3'},
+ - { 'key-args': {'scenario': 'bob.xml', '-p': '5060', '-i': '127.0.0.3', '-s': 'alice', '-timeout': '20s', '-mi': '127.0.0.3'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
# Alice calls Bob and sends audio. Alice then attempts to blind transfer Bob to Charlie.
- - { 'key-args': {'scenario': 'alice.xml', '-p': '5060', '-i': '127.0.0.2', '-s': 'bob', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.2'},
+ - { 'key-args': {'scenario': 'alice.xml', '-p': '5060', '-i': '127.0.0.2', '-s': 'bob', '-timeout': '20s', '-mi': '127.0.0.2'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
# Second iteration
-
scenarios:
# Charlie receives transfered call and sends audio to Bob.
- - { 'key-args': {'scenario': 'charlie.xml', '-p': '5060', '-i': '127.0.0.4', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.4'},
+ - { 'key-args': {'scenario': 'charlie.xml', '-p': '5060', '-i': '127.0.0.4', '-timeout': '20s', '-mi': '127.0.0.4'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
# Bob receives call from Alice and sends audio.
- - { 'key-args': {'scenario': 'bob.xml', '-p': '5060', '-i': '127.0.0.3', '-s': 'alice', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.3'},
+ - { 'key-args': {'scenario': 'bob.xml', '-p': '5060', '-i': '127.0.0.3', '-s': 'alice', '-timeout': '20s', '-mi': '127.0.0.3'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
# Alice calls Bob and sends audio. Alice then attempts to blind transfer Bob to Charlie.
- - { 'key-args': {'scenario': 'alice.xml', '-p': '5060', '-i': '127.0.0.2', '-s': 'bob_two', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.2'},
+ - { 'key-args': {'scenario': 'alice.xml', '-p': '5060', '-i': '127.0.0.2', '-s': 'bob_two', '-timeout': '20s', '-mi': '127.0.0.2'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
ami-config:
diff --git a/tests/channels/pjsip/transfers/blind_transfer/caller_with_hold_drop_options/test-config.yaml b/tests/channels/pjsip/transfers/blind_transfer/caller_with_hold_drop_options/test-config.yaml
index 23f44a7..4ac14c6 100644
--- a/tests/channels/pjsip/transfers/blind_transfer/caller_with_hold_drop_options/test-config.yaml
+++ b/tests/channels/pjsip/transfers/blind_transfer/caller_with_hold_drop_options/test-config.yaml
@@ -41,25 +41,25 @@
-
scenarios:
# Charlie receives transfered call and sends audio to Bob.
- - { 'key-args': {'scenario': 'charlie.xml', '-p': '5060', '-i': '127.0.0.4', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.4'},
+ - { 'key-args': {'scenario': 'charlie.xml', '-p': '5060', '-i': '127.0.0.4', '-timeout': '20s', '-mi': '127.0.0.4'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
# Bob receives call from Alice and sends audio.
- - { 'key-args': {'scenario': 'bob.xml', '-p': '5060', '-i': '127.0.0.3', '-s': 'alice', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.3'},
+ - { 'key-args': {'scenario': 'bob.xml', '-p': '5060', '-i': '127.0.0.3', '-s': 'alice', '-timeout': '20s', '-mi': '127.0.0.3'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
# Alice calls Bob and sends audio. Alice then attempts to blind transfer Bob to Charlie.
- - { 'key-args': {'scenario': 'alice.xml', '-p': '5060', '-i': '127.0.0.2', '-s': 'bob', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.2'},
+ - { 'key-args': {'scenario': 'alice.xml', '-p': '5060', '-i': '127.0.0.2', '-s': 'bob', '-timeout': '20s', '-mi': '127.0.0.2'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
# Second iteration
-
scenarios:
# Charlie receives transfered call and sends audio to Bob.
- - { 'key-args': {'scenario': 'charlie.xml', '-p': '5060', '-i': '127.0.0.4', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.4'},
+ - { 'key-args': {'scenario': 'charlie.xml', '-p': '5060', '-i': '127.0.0.4', '-timeout': '20s', '-mi': '127.0.0.4'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
# Bob receives call from Alice and sends audio.
- - { 'key-args': {'scenario': 'bob.xml', '-p': '5060', '-i': '127.0.0.3', '-s': 'alice', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.3'},
+ - { 'key-args': {'scenario': 'bob.xml', '-p': '5060', '-i': '127.0.0.3', '-s': 'alice', '-timeout': '20s', '-mi': '127.0.0.3'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
# Alice calls Bob and sends audio. Alice then attempts to blind transfer Bob to Charlie.
- - { 'key-args': {'scenario': 'alice.xml', '-p': '5060', '-i': '127.0.0.2', '-s': 'bob_two', '-timeout': '20s', '-min_rtp_port': '6004', '-max_rtp_port': '6008', '-mi': '127.0.0.2'},
+ - { 'key-args': {'scenario': 'alice.xml', '-p': '5060', '-i': '127.0.0.2', '-s': 'bob_two', '-timeout': '20s', '-mi': '127.0.0.2'},
'ordered-args': ['-timeout_error', '-key', 'custom_media_port', '6004'] }
ami-config:
--
To view, visit https://gerrit.asterisk.org/9644
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: I3c3f82c7d6aec5fe3acaba1e5dd1d54edba3ba72
Gerrit-Change-Number: 9644
Gerrit-PatchSet: 1
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180724/2a870bb2/attachment-0001.html>
More information about the asterisk-code-review
mailing list