diff options
| author | Philipp Maier <pmaier@sysmocom.de> | 2018-06-21 10:03:13 +0200 |
|---|---|---|
| committer | Harald Welte <laforge@gnumonks.org> | 2018-06-23 11:40:04 +0000 |
| commit | 76a8d0160e6d53b5c709ae6a48942cc53987e384 (patch) | |
| tree | df13f764cb076dcc13e5385ec6ecfa0360ef0e7d | |
| parent | 2a978b9fd8e90e23523b22dfac94047f361d1211 (diff) | |
MGCP_Test: ts_CRCX_no_lco: check media description instead of media attribute
The testcase ts_CRCX_no_lco looks at the media attributes to see
if it findes the expected default codec there. In this testcase
we expect PCMU/8000/1 as media attribute, but this is a codec from
the fixed payload type domain. The MGW may not list this info inside
the media attributes. Listing the payload type number in the media
description is sufficient. We should check this instead.
- Remove media attribute check
- Check meida description for PCMU (0)
Change-Id: I69600a1025e68011e8fc5d8bf22d842d9c63bf53
Related: OS#2658
| -rw-r--r-- | mgw/MGCP_Test.ttcn | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mgw/MGCP_Test.ttcn b/mgw/MGCP_Test.ttcn index 26a07012..68684050 100644 --- a/mgw/MGCP_Test.ttcn +++ b/mgw/MGCP_Test.ttcn @@ -326,7 +326,7 @@ module MGCP_Test { /* See also OS#2658: Even when we omit the LCO information, we expect the MGW to pick a sane payload type for us. This payload type should be visible in the SDP of the response. */ - if (resp.sdp.media_list[0].attributes[0].rtpmap.attr_value != "0 PCMU/8000/1") { + if (resp.sdp.media_list[0].media_field.fmts[0] != "0") { setverdict(fail, "SDP contains unexpected codec"); } |