dexter has uploaded this change for review. ( https://gerrit.osmocom.org/9691


Change subject: MGCP_Test: ts_CRCX_no_lco: check media description instead of 
media attribute
......................................................................

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
---
M mgw/MGCP_Test.ttcn
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks 
refs/changes/91/9691/1

diff --git a/mgw/MGCP_Test.ttcn b/mgw/MGCP_Test.ttcn
index 26a0701..6868405 100644
--- a/mgw/MGCP_Test.ttcn
+++ b/mgw/MGCP_Test.ttcn
@@ -326,7 +326,7 @@
                /* 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");
                }


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

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I69600a1025e68011e8fc5d8bf22d842d9c63bf53
Gerrit-Change-Number: 9691
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pma...@sysmocom.de>

Reply via email to