<p>fixeria has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22716">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">msc: finish and enable test case for MT Call T310 timer<br><br>The test suite needs to handle MGCP messages, otherwise the related<br>FSMs in the IUT would tear everything down before T310 is expired.<br><br>Change-Id: I79d9ae3b086d05c3d7c0a1241720d6c3f1e90281<br>Related: SYS#5340<br>---<br>M msc/MSC_Tests.ttcn<br>1 file changed, 41 insertions(+), 15 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/16/22716/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/msc/MSC_Tests.ttcn b/msc/MSC_Tests.ttcn</span><br><span>index e6681bb..206effa 100644</span><br><span>--- a/msc/MSC_Tests.ttcn</span><br><span>+++ b/msc/MSC_Tests.ttcn</span><br><span>@@ -1679,43 +1679,69 @@</span><br><span> </span><br><span> /* Test MT Call T310 timer */</span><br><span> friend function f_tc_mt_t310(charstring id, BSC_ConnHdlrPars pars) runs on BSC_ConnHdlr {</span><br><span style="color: hsl(0, 100%, 40%);">- f_init_handler(pars, 200.0);</span><br><span>         var CallParameters cpars := valueof(t_CallParams('123456'H, 0));</span><br><span style="color: hsl(0, 100%, 40%);">-        var MNCC_PDU mncc;</span><br><span>   var MgcpCommand mgcp_cmd;</span><br><span style="color: hsl(120, 100%, 40%);">+     var PDU_BSSAP bssap;</span><br><span style="color: hsl(120, 100%, 40%);">+  timer T310;</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+       f_init_handler(pars);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+       /* Initiate a MT call, establish connection */</span><br><span>       f_mt_call_start(cpars);</span><br><span> </span><br><span>  /* MS->MSC: CALL CONFIRMED */</span><br><span>     BSSAP.send(ts_PDU_DTAP_MO(ts_ML3_MO_CC_CALL_CONF(cpars.transaction_id)));</span><br><span>    MNCC.receive(tr_MNCC_CALL_CONF_ind(cpars.mncc_callref));</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-    MGCP.receive(tr_CRCX) -> value mgcp_cmd;</span><br><span style="color: hsl(0, 100%, 40%);">-     cpars.mgcp_call_id := f_MgcpCmd_extract_call_id(mgcp_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-      cpars.mgcp_ep := mgcp_cmd.line.ep;</span><br><span style="color: hsl(0, 100%, 40%);">-      /* FIXME: Respond to CRCX */</span><br><span style="color: hsl(120, 100%, 40%);">+  /* NOTE: MSC is expected to start T310 here */</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-      /* old libosmocore T310 default timeout is 180s. so let's wait 190 */</span><br><span style="color: hsl(0, 100%, 40%);">-       timer T := 190.0;</span><br><span style="color: hsl(0, 100%, 40%);">-       T.start;</span><br><span style="color: hsl(120, 100%, 40%);">+      /* MSC->MGW: CRCX (first) */</span><br><span style="color: hsl(120, 100%, 40%);">+       MGCP.receive(tr_CRCX) -> value mgcp_cmd;</span><br><span style="color: hsl(120, 100%, 40%);">+   f_handle_crcx(cpars, mgcp_cmd); /* MSC<-MGW: OK */</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+       /* BSC->BSC: BSSMAP ASSIGNMENT REQ */</span><br><span style="color: hsl(120, 100%, 40%);">+      BSSAP.receive(tr_BSSMAP_AssignmentReq(omit, ?)) -> value bssap;</span><br><span style="color: hsl(120, 100%, 40%);">+    BSSAP.send(ts_BSSMAP_AssignmentComplete(omit,</span><br><span style="color: hsl(120, 100%, 40%);">+         aoip := f_ts_BSSMAP_IE_AoIP_TLA(cpars.bss_rtp_ip, cpars.bss_rtp_port),</span><br><span style="color: hsl(120, 100%, 40%);">+                speechCodec := ts_BSSMAP_IE_SpeechCodec({ ts_CodecFR })));</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+  /* MSC->MGW: MDCX */</span><br><span style="color: hsl(120, 100%, 40%);">+       MGCP.receive(tr_MDCX) -> value mgcp_cmd;</span><br><span style="color: hsl(120, 100%, 40%);">+   MGCP.send(ts_MDCX_ACK(mgcp_cmd.line.trans_id, cpars.mgw_conn_1.mgcp_connection_id,</span><br><span style="color: hsl(120, 100%, 40%);">+            sdp := omit));</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+      /* MSC->MGW: CRCX (second) */</span><br><span style="color: hsl(120, 100%, 40%);">+      MGCP.receive(tr_CRCX) -> value mgcp_cmd;</span><br><span style="color: hsl(120, 100%, 40%);">+   f_handle_crcx(cpars, mgcp_cmd); /* MSC<-MGW: OK */</span><br><span style="color: hsl(120, 100%, 40%);">+ MNCC.receive(tr_MNCC_RTP_CREATE(cpars.mncc_callref));</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+       /* Reschedule the guard timeout */</span><br><span style="color: hsl(120, 100%, 40%);">+    g_Tguard.start(30.0 + 10.0);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+        /* NOTE: the BSC is expected to respond with CC ALERTING at this state, so</span><br><span style="color: hsl(120, 100%, 40%);">+     * the MSC would stop T310.  However, the idea is to verify T310 expiration</span><br><span style="color: hsl(120, 100%, 40%);">+    * here, so grab some popcorn and wait for MNCC DISC.ind. */</span><br><span style="color: hsl(120, 100%, 40%);">+  T310.start(30.0 + 2.0);</span><br><span>      alt {</span><br><span style="color: hsl(0, 100%, 40%);">-   [] T.timeout {</span><br><span style="color: hsl(0, 100%, 40%);">-          setverdict(fail, "Timeout waiting for T310");</span><br><span style="color: hsl(120, 100%, 40%);">+       [] T310.timeout {</span><br><span style="color: hsl(120, 100%, 40%);">+             setverdict(fail, "Timeout waiting for MNCC DISC.ind due to T310");</span><br><span>                 mtc.stop;</span><br><span>            }</span><br><span>    [] MNCC.receive(tr_MNCC_DISC_ind(cpars.mncc_callref)) {</span><br><span>              MNCC.send(ts_MNCC_REL_req(cpars.mncc_callref, valueof(ts_MNCC_cause(23))));</span><br><span style="color: hsl(120, 100%, 40%);">+           log("Rx MNCC DISC.ind, T310.read yelds ", T310.read);</span><br><span style="color: hsl(120, 100%, 40%);">+               setverdict(pass);</span><br><span>            }</span><br><span>    }</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>  BSSAP.receive(tr_PDU_DTAP_MT(tr_ML3_MT_CC_DISC(cpars.transaction_id)));</span><br><span>      BSSAP.receive(tr_PDU_DTAP_MT(tr_ML3_MT_CC_RELEASE(cpars.transaction_id)));</span><br><span style="color: hsl(0, 100%, 40%);">-      /* FIXME: We're sending this with TIflag 0: allocated by sender, which is wrong */</span><br><span style="color: hsl(0, 100%, 40%);">-  BSSAP.send(ts_PDU_DTAP_MO(ts_ML3_MO_CC_REL_COMPL(cpars.transaction_id)));</span><br><span style="color: hsl(120, 100%, 40%);">+     BSSAP.send(ts_PDU_DTAP_MO(ts_ML3_MO_CC_REL_COMPL(cpars.transaction_id, '1'B)));</span><br><span> </span><br><span>  alt {</span><br><span>        [] MGCP.receive(tr_DLCX(?)) -> value mgcp_cmd {</span><br><span>           MGCP.send(ts_DLCX_ACK2(mgcp_cmd.line.trans_id));</span><br><span style="color: hsl(0, 100%, 40%);">-                f_create_mgcp_delete_ep(cpars.mgcp_ep);</span><br><span style="color: hsl(120, 100%, 40%);">+               // FIXME: f_create_mgcp_delete_ep(cpars.mgcp_ep);</span><br><span>            repeat;</span><br><span>              }</span><br><span>    [] as_clear_cmd_compl_disc();</span><br><span>@@ -6342,7 +6368,7 @@</span><br><span>        execute( TC_mo_crcx_ran_reject() );</span><br><span>  execute( TC_mt_crcx_ran_reject() );</span><br><span>  execute( TC_mo_setup_and_dtmf_dup() );</span><br><span style="color: hsl(0, 100%, 40%);">-  //execute( TC_mt_t310() );</span><br><span style="color: hsl(120, 100%, 40%);">+    execute( TC_mt_t310() );</span><br><span>     execute( TC_gsup_cancel() );</span><br><span>         execute( TC_lu_imsi_auth_tmsi_encr_1_13() );</span><br><span>         execute( TC_lu_imsi_auth_tmsi_encr_3_13() );</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22716">change 22716</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22716"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I79d9ae3b086d05c3d7c0a1241720d6c3f1e90281 </div>
<div style="display:none"> Gerrit-Change-Number: 22716 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>