This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
Harald Welte gerrit-no-reply at lists.osmocom.orgHarald Welte has submitted this change and it was merged. Change subject: MGCP_Test: be more liberal in some of the error codes we expect ...................................................................... MGCP_Test: be more liberal in some of the error codes we expect Sometimes it's not entirely clear which error code should be returned, so let's accept multiple options, depending on the situation Change-Id: I6d6eaec4d025b9da170f59c759884afe2e768521 --- M mgw/MGCP_Test.ttcn 1 file changed, 4 insertions(+), 5 deletions(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/mgw/MGCP_Test.ttcn b/mgw/MGCP_Test.ttcn index 7c3889a..516944a 100644 --- a/mgw/MGCP_Test.ttcn +++ b/mgw/MGCP_Test.ttcn @@ -212,7 +212,7 @@ /* Send DLCX and expect OK response */ function f_dlcx_ok(MgcpEndpoint ep, template MgcpCallId call_id := omit, template MgcpConnectionId conn_id := omit) runs on dummy_CT { - f_dlcx(ep, "200", "OK", call_id, conn_id); + f_dlcx(ep, ("200","250"), "OK", call_id, conn_id); } /* Send DLCX and accept any response */ @@ -296,7 +296,7 @@ var template MgcpCommand cmd; var MgcpResponse resp; var MgcpEndpoint ep := "2 at mgw"; - var template MgcpResponse rtmpl := tr_MgcpResp_Err("400"); + var template MgcpResponse rtmpl := tr_MgcpResp_Err(("400","516")); f_init(ep); @@ -316,7 +316,7 @@ var MgcpResponse resp; var MgcpEndpoint ep := "2 at mgw"; var MgcpCallId call_id := '1229'H; - var template MgcpResponse rtmpl := tr_MgcpResp_Err("400"); + var template MgcpResponse rtmpl := tr_MgcpResp_Err(("400","517")); f_init(ep); @@ -420,8 +420,7 @@ var MgcpEndpoint ep := "4 at mgw"; var template MgcpResponse rtmpl := { line := { - /* TODO: accept/enforce better error? */ - code := "400", + code := ("400", "515"), string := ? }, params:= { }, -- To view, visit https://gerrit.osmocom.org/5612 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d6eaec4d025b9da170f59c759884afe2e768521 Gerrit-PatchSet: 1 Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Owner: Harald Welte <laforge at gnumonks.org> Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org> Gerrit-Reviewer: Jenkins Builder