Change in osmo-ttcn3-hacks[master]: MSC_Tests: Make sure only sgsap related tests use the SGs interface

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/.

dexter gerrit-no-reply at lists.osmocom.org
Thu Feb 14 17:56:58 UTC 2019


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


Change subject: MSC_Tests: Make sure only sgsap related tests use the SGs interface
......................................................................

MSC_Tests: Make sure only sgsap related tests use the SGs interface

At the moment the SGs interface is started and connected in all tests in
the testsuite. This may be a problem because the IUT may lack the SGs
support. In this case all tests would fail because of the missing SGs
interface.

Lets initalize and connect the SGs interface only for SGs related tests
to prevent unrelated test failures.

Change-Id: I8e012e3599ad00db2e132b6463499f8a4a2307f1
Related: OS#3614
---
M msc/MSC_Tests.ttcn
1 file changed, 81 insertions(+), 45 deletions(-)



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

diff --git a/msc/MSC_Tests.ttcn b/msc/MSC_Tests.ttcn
index 34fe9ad..00f9bec 100644
--- a/msc/MSC_Tests.ttcn
+++ b/msc/MSC_Tests.ttcn
@@ -119,8 +119,6 @@
 	charstring mp_mme_name := "mmec01.mmegi0001.mme.epc.mnc070.mcc901.3gppnetwork.org";
 	charstring mp_vlr_name := "vlr.example.net";
 
-	boolean mp_sgsap_enable := false;
-
 	BSSAP_Configurations mp_bssap_cfg := {
 		{
 			sccp_service_type := "mtp3_itu",
@@ -254,7 +252,7 @@
 	}
 }
 
-function f_init(integer num_bsc := 1) runs on MTC_CT {
+function f_init(integer num_bsc := 1, boolean sgsap := false) runs on MTC_CT {
 
 	if (g_initialized == true) {
 		return;
@@ -280,7 +278,7 @@
 	f_init_gsup("MSC_Test");
 	f_init_smpp("MSC_Test");
 
-	if (mp_sgsap_enable == true) {
+	if (sgsap == true) {
 		f_init_sgsap("MSC_Test");
 	}
 
@@ -472,7 +470,7 @@
 type function void_fn(charstring id, BSC_ConnHdlrPars pars) runs on BSC_ConnHdlr;
 
 /* FIXME: move into BSC_ConnectionHandler? */
-function f_init_pars(integer imsi_suffix) runs on MTC_CT return BSC_ConnHdlrPars {
+function f_init_pars(integer imsi_suffix, boolean sgsap := false) runs on MTC_CT return BSC_ConnHdlrPars {
 	var BSC_ConnHdlrNetworkPars net_pars := {
 		kc_support := '0A'O,	/* A5/1 and A5/3 enabled */
 		expect_tmsi := true,
@@ -497,7 +495,7 @@
 		ipa_ctrl_port := mp_msc_ctrl_port,
 		ipa_ctrl_enable := true,
 		mm_info := mp_mm_info,
-		sgsap_enable := mp_sgsap_enable
+		sgsap_enable := sgsap
 	};
 	return pars;
 }
@@ -523,7 +521,7 @@
 	connect(vc_conn:SMPP, vc_SMPP:SMPP_CLIENT);
 	connect(vc_conn:SMPP_PROC, vc_SMPP:SMPP_PROC);
 	/* SGs part */
-	if (mp_sgsap_enable == true) {
+	if (pars.sgsap_enable == true) {
 		connect(vc_conn:SGsAP, vc_SGsAP:SGsAP_CLIENT);
 		connect(vc_conn:SGsAP_PROC, vc_SGsAP:SGsAP_PROC);
 	}
@@ -3569,9 +3567,11 @@
 }
 
 testcase TC_sgsap_reset() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
-	vc_conn := f_start_handler(refers(f_tc_sgsap_reset), 11810);
+	f_init(1, true);
+	pars := f_init_pars(11810, true);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_reset), pars);
 	vc_conn.done;
 }
 
@@ -3661,9 +3661,11 @@
 	setverdict(pass);
 }
 testcase TC_sgsap_lu() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
-	vc_conn := f_start_handler(refers(f_tc_sgsap_lu), 11811);
+	f_init(1, true);
+	pars := f_init_pars(11811, true);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_lu), pars);
 	vc_conn.done;
 }
 
@@ -3698,10 +3700,12 @@
 	setverdict(pass);
 }
 testcase TC_sgsap_lu_imsi_reject() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
+	f_init(1, true);
+	pars := f_init_pars(11812, true);
 
-	vc_conn := f_start_handler(refers(f_tc_sgsap_lu_imsi_reject), 11812);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_lu_imsi_reject), pars);
 	vc_conn.done;
 }
 
@@ -3759,10 +3763,12 @@
 	setverdict(pass);
 }
 testcase TC_sgsap_lu_and_nothing() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
+	f_init(1, true);
+	pars := f_init_pars(11813, true);
 
-	vc_conn := f_start_handler(refers(f_tc_sgsap_lu_and_nothing), 11813);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_lu_and_nothing), pars);
 	vc_conn.done;
 }
 
@@ -3782,9 +3788,11 @@
 	setverdict(pass);
 }
 testcase TC_sgsap_expl_imsi_det_eps() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
-	vc_conn := f_start_handler(refers(f_tc_sgsap_expl_imsi_det_eps), 11814);
+	f_init(1, true);
+	pars := f_init_pars(11814, true);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_expl_imsi_det_eps), pars);
 	vc_conn.done;
 }
 
@@ -3805,9 +3813,11 @@
 	setverdict(pass);
 }
 testcase TC_sgsap_expl_imsi_det_noneps() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
-	vc_conn := f_start_handler(refers(f_tc_sgsap_expl_imsi_det_noneps), 11815);
+	f_init(1, true);
+	pars := f_init_pars(11815, true);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_expl_imsi_det_noneps), pars);
 	vc_conn.done;
 }
 
@@ -3854,9 +3864,11 @@
 	setverdict(pass);
 }
 testcase TC_sgsap_paging_rej() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
-	vc_conn := f_start_handler(refers(f_tc_sgsap_paging_rej), 11816);
+	f_init(1, true);
+	pars := f_init_pars(11816, true);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_paging_rej), pars);
 	vc_conn.done;
 }
 
@@ -3901,9 +3913,11 @@
 	setverdict(pass);
 }
 testcase TC_sgsap_paging_subscr_rej() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
-	vc_conn := f_start_handler(refers(f_tc_sgsap_paging_subscr_rej), 11817);
+	f_init(1, true);
+	pars := f_init_pars(11817, true);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_paging_subscr_rej), pars);
 	vc_conn.done;
 }
 
@@ -3943,9 +3957,11 @@
 	setverdict(pass);
 }
 testcase TC_sgsap_paging_ue_unr() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
-	vc_conn := f_start_handler(refers(f_tc_sgsap_paging_ue_unr), 11818);
+	f_init(1, true);
+	pars := f_init_pars(11818, true);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_paging_ue_unr), pars);
 	vc_conn.done;
 }
 
@@ -3982,9 +3998,11 @@
 	setverdict(pass);
 }
 testcase TC_sgsap_paging_and_nothing() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
-	vc_conn := f_start_handler(refers(f_tc_sgsap_paging_and_nothing), 11819);
+	f_init(1, true);
+	pars := f_init_pars(11819, true);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_paging_and_nothing), pars);
 	vc_conn.done;
 }
 
@@ -4038,9 +4056,11 @@
 	setverdict(pass);
 }
 testcase TC_sgsap_paging_and_lu() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
-	vc_conn := f_start_handler(refers(f_tc_sgsap_paging_and_lu), 11820);
+	f_init(1, true);
+	pars := f_init_pars(11820, true);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_paging_and_lu), pars);
 	vc_conn.done;
 }
 
@@ -4067,9 +4087,11 @@
 	setverdict(pass);
 }
 testcase TC_sgsap_unexp_ud() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
-	vc_conn := f_start_handler(refers(f_tc_sgsap_unexp_ud), 11821);
+	f_init(1, true);
+	pars := f_init_pars(11821, true);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_unexp_ud), pars);
 	vc_conn.done;
 }
 
@@ -4091,9 +4113,11 @@
 	setverdict(pass);
 }
 testcase TC_sgsap_unsol_ud() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
-	vc_conn := f_start_handler(refers(f_tc_sgsap_unsol_ud), 11822);
+	f_init(1, true);
+	pars := f_init_pars(11822, true);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_unsol_ud), pars);
 	vc_conn.done;
 }
 
@@ -4237,9 +4261,11 @@
 	setverdict(pass);
 }
 testcase TC_sgsap_mt_sms() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
-	vc_conn := f_start_handler(refers(f_tc_sgsap_mt_sms), 11823);
+	f_init(1, true);
+	pars := f_init_pars(11823, true);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_mt_sms), pars);
 	vc_conn.done;
 }
 
@@ -4265,9 +4291,11 @@
 	f_sgsap_bssmap_screening()
 }
 testcase TC_sgsap_mo_sms() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
-	vc_conn := f_start_handler(refers(f_tc_sgsap_mo_sms), 11824);
+	f_init(1, true);
+	pars := f_init_pars(11824, true);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_mo_sms), pars);
 	vc_conn.done;
 }
 
@@ -4337,9 +4365,11 @@
 	setverdict(pass);
 }
 testcase TC_sgsap_mt_sms_and_nothing() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
-	vc_conn := f_start_handler(refers(f_tc_sgsap_mt_sms_and_nothing), 11825);
+	f_init(1, true);
+	pars := f_init_pars(11825, true);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_mt_sms_and_nothing), pars);
 	vc_conn.done;
 }
 
@@ -4397,9 +4427,11 @@
 	setverdict(pass);
 }
 testcase TC_sgsap_mt_sms_and_reject() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
-	vc_conn := f_start_handler(refers(f_tc_sgsap_mt_sms_and_reject), 11826);
+	f_init(1, true);
+	pars := f_init_pars(11826, true);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_mt_sms_and_reject), pars);
 	vc_conn.done;
 }
 
@@ -4471,10 +4503,12 @@
 	f_mt_lu_and_csfb_call(id, pars, true);
 }
 testcase TC_bssap_lu_sgsap_lu_and_mt_call() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
+	f_init(1, true);
+	pars := f_init_pars(118139, true);
 
-	vc_conn := f_start_handler(refers(f_tc_bssap_lu_sgsap_lu_and_mt_call), 139);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_bssap_lu_sgsap_lu_and_mt_call), pars);
 	vc_conn.done;
 }
 
@@ -4484,10 +4518,12 @@
 	f_mt_lu_and_csfb_call(id, pars, false);
 }
 testcase TC_sgsap_lu_and_mt_call() runs on MTC_CT {
+	var BSC_ConnHdlrPars pars;
 	var BSC_ConnHdlr vc_conn;
-	f_init();
+	f_init(1, true);
+	pars := f_init_pars(11827, true);
 
-	vc_conn := f_start_handler(refers(f_tc_sgsap_lu_and_mt_call), 11827);
+	vc_conn := f_start_handler_with_pars(refers(f_tc_sgsap_lu_and_mt_call), pars);
 	vc_conn.done;
 }
 

-- 
To view, visit https://gerrit.osmocom.org/12913
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: I8e012e3599ad00db2e132b6463499f8a4a2307f1
Gerrit-Change-Number: 12913
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier at sysmocom.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190214/88a0bb71/attachment.htm>


More information about the gerrit-log mailing list