fixeria has uploaded this change for review. (
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36772?usp=email )
Change subject: msc: add TC_gsup_mt_sms_lu_x5 for X5 (LU delay) timer
......................................................................
msc: add TC_gsup_mt_sms_lu_x5 for X5 (LU delay) timer
Change-Id: I74fa174ea649adb2112c8e471c0e339a2197a08d
Related: osmo-msc.git Ic519cab55d65e47b2636124427dab1a1d80fab78
Related: SYS#6913
---
M msc/BSC_ConnectionHandler.ttcn
M msc/MSC_Tests.ttcn
M msc/expected-results.xml
3 files changed, 78 insertions(+), 3 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks
refs/changes/72/36772/1
diff --git a/msc/BSC_ConnectionHandler.ttcn b/msc/BSC_ConnectionHandler.ttcn
index 658923e..056a3c0 100644
--- a/msc/BSC_ConnectionHandler.ttcn
+++ b/msc/BSC_ConnectionHandler.ttcn
@@ -833,7 +833,8 @@
}
}
-function f_perform_lu(template (omit) MobileIdentityLV use_mi := omit)
+function f_perform_lu(template (omit) MobileIdentityLV use_mi := omit,
+ boolean expect_clear := true)
runs on BSC_ConnHdlr {
var MobileIdentityLV mi;
if (istemplatekind(use_mi, "omit")) {
@@ -866,8 +867,11 @@
f_msc_lu_hlr();
f_mm_imei();
as_accept_reject_lu(g_pars.net.expect_attach_success);
- /* FIXME: there could be pending SMS or other common procedures by the
MSC, let's ignore them */
- f_expect_clear(verify_vlr_cell_id := g_pars.net.expect_attach_success);
+
+ /* there could be pending SMS or other common procedures by the MSC */
+ if (expect_clear) {
+ f_expect_clear(verify_vlr_cell_id :=
g_pars.net.expect_attach_success);
+ }
setverdict(pass);
}
diff --git a/msc/MSC_Tests.ttcn b/msc/MSC_Tests.ttcn
index 09004e8..1f9bfc4 100644
--- a/msc/MSC_Tests.ttcn
+++ b/msc/MSC_Tests.ttcn
@@ -3279,6 +3279,64 @@
f_vty_config(MSCVTY, "msc", "sms-over-gsup");
vc_conn :=
f_start_handler_with_pars(refers(f_tc_gsup_mt_multi_part_sms), pars);
vc_conn.done;
+ Tf_vty_config(MSCVTY, "msc", "no sms-over-gsup");
+}
+
+/* Test X5 (LU delay) timer allowing to deliver MT SMS over the same
connection */
+friend function f_TC_gsup_mt_sms_lu_x5(charstring id, BSC_ConnHdlrPars pars)
+runs on BSC_ConnHdlr {
+ var SmsParameters spars := valueof(t_SmsPars);
+
+ f_init_handler(pars);
+
+ /* We need to inspect GSUP activity */
+ f_create_gsup_expect(hex2str(g_pars.imsi));
+
+ /* Perform location update */
+ f_perform_lu(expect_clear := false);
+
+ /* Register an 'expect' for given IMSI (+TMSI) */
+ f_ran_register_imsi(g_pars.imsi, g_pars.tmsi);
+
+ var template GSUP_PDU mt_forwardSM_res := tr_GSUP_MT_FORWARD_SM_RES(
+ imsi := g_pars.imsi,
+ /* NOTE: MSC should assign RP-MR itself */
+ sm_rp_mr := ?
+ );
+
+ f_sleep(0.5);
+
+ /* Submit a MT SMS on GSUP */
+ f_gsup_forwardSM_req(spars);
+
+ /* Wait for MT SMS on DTAP */
+ f_mt_sms_expect(spars);
+
+ /* Send RP-ACK and expect MT-forwardSM-Res on GSUP */
+ f_mt_sms_send_rp_ack(spars);
+ alt {
+ [] GSUP.receive(mt_forwardSM_res) {
+ log("RX MT-forwardSM-Res (RP-ACK)");
+ setverdict(pass);
+ }
+ [] GSUP.receive {
+ setverdict(fail, "Rx unexpected GSUP message");
+ mtc.stop;
+ }
+ }
+
+ f_expect_clear();
+}
+testcase TC_gsup_mt_sms_lu_x5() runs on MTC_CT {
+ var BSC_ConnHdlrPars pars;
+ var BSC_ConnHdlr vc_conn;
+ f_init();
+ pars := f_init_pars(92);
+ f_vty_config(MSCVTY, "msc", "sms-over-gsup");
+ f_vty_config(MSCVTY, "msc", "timer geran x5 2");
+ vc_conn := f_start_handler_with_pars(refers(f_TC_gsup_mt_sms_lu_x5),
pars);
+ vc_conn.done;
+ f_vty_config(MSCVTY, "msc", "timer geran x5 0");
f_vty_config(MSCVTY, "msc", "no sms-over-gsup");
}
@@ -7434,6 +7492,7 @@
execute( TC_gsup_mt_sms_rp_mr() );
execute( TC_gsup_mo_mt_sms_rp_mr() );
execute( TC_gsup_mt_multi_part_sms() );
+ execute( TC_gsup_mt_sms_lu_x5() );
execute( TC_lu_and_mo_ussd_single_request() );
execute( TC_lu_and_mt_ussd_notification() );
diff --git a/msc/expected-results.xml b/msc/expected-results.xml
index aad96a7..d5a9462 100644
--- a/msc/expected-results.xml
+++ b/msc/expected-results.xml
@@ -80,6 +80,7 @@
<testcase classname='MSC_Tests' name='TC_gsup_mt_sms_rp_mr' time='MASKED'/>
<testcase classname='MSC_Tests' name='TC_gsup_mo_mt_sms_rp_mr'
time='MASKED'/>
<testcase classname='MSC_Tests' name='TC_gsup_mt_multi_part_sms'
time='MASKED'/>
+ <testcase classname='MSC_Tests' name='TC_gsup_mt_sms_lu_x5' time='MASKED'/>
<testcase classname='MSC_Tests' name='TC_lu_and_mo_ussd_single_request'
time='MASKED'/>
<testcase classname='MSC_Tests' name='TC_lu_and_mt_ussd_notification'
time='MASKED'/>
<testcase classname='MSC_Tests' name='TC_lu_and_mo_ussd_during_mt_call'
time='MASKED'/>
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/36772?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I74fa174ea649adb2112c8e471c0e339a2197a08d
Gerrit-Change-Number: 36772
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-MessageType: newchange
