Change in osmo-ttcn3-hacks[master]: gbproxy: Use g_parse.sgsn_idx for paging

[daniel]

daniel has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22299 )


Change subject: gbproxy: Use g_parse.sgsn_idx for paging
......................................................................

gbproxy: Use g_parse.sgsn_idx for paging

The ConnHdlr test stores the sgsn_idx in g_pars. Use that in the paging
tests to the correct SGSN pages the MS.
This commit fixes what 4332044c missed and is required for the ptp paging
reject test to pass.

Change-Id: Id85ffa57af943ae98c0f022b261ec24d557c236d
Related: OS#4951
---
M gbproxy/GBProxy_Tests.ttcn
1 file changed, 11 insertions(+), 11 deletions(-)



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

diff --git a/gbproxy/GBProxy_Tests.ttcn b/gbproxy/GBProxy_Tests.ttcn
index f5248aa..81627f4 100644
--- a/gbproxy/GBProxy_Tests.ttcn
+++ b/gbproxy/GBProxy_Tests.ttcn
@@ -1384,7 +1384,7 @@
 {
 	/* doesn't really make sense: Sending to a single BVCI means the message ends up
 	 * at that BVC (cell) only, and paging all over the BSS area is not possible */
-	f_send_paging_ps_exp_one_bss(ts_BssgpP4BssArea, 0, false, 0);
+	f_send_paging_ps_exp_one_bss(ts_BssgpP4BssArea, g_pars.sgsn_idx, false, 0);
 }
 testcase TC_paging_ps_ptp_bss() runs on test_CT {
 	f_init();
@@ -1398,7 +1398,7 @@
 	var template (present) PDU_BSSGP exp_rx;
 	/* doesn't really make sense: Sending to a single BVCI means the message ends up
 	 * at that BVC (cell) only, and paging all over the BSS area is not possible */
-	f_send_paging_ps_exp_one_bss(ts_BssgpP4LAC(pcu_bvc_cfg[0].cell_id.ra_id.lai), 0, false, 0);
+	f_send_paging_ps_exp_one_bss(ts_BssgpP4LAC(pcu_bvc_cfg[0].cell_id.ra_id.lai), g_pars.sgsn_idx, false, 0);
 }
 testcase TC_paging_ps_ptp_lac() runs on test_CT {
 	f_init();
@@ -1414,7 +1414,7 @@
 		lac := 33333
 	};
 	/* as it's sent on the PTP BVC, we expect it to pass even for unknown LAC */
-	f_send_paging_ps_exp_one_bss(ts_BssgpP4LAC(unknown_la), 0, false, 0);
+	f_send_paging_ps_exp_one_bss(ts_BssgpP4LAC(unknown_la), g_pars.sgsn_idx, false, 0);
 }
 testcase TC_paging_ps_ptp_lac_unknown() runs on test_CT {
 	f_init();
@@ -1427,7 +1427,7 @@
 {
 	/* doesn't really make sense: Sending to a single BVCI means the message ends up
 	 * at that BVC (cell) only, and paging all over the BSS area is not possible */
-	f_send_paging_ps_exp_one_bss(ts_BssgpP4RAC(pcu_bvc_cfg[0].cell_id.ra_id), 0, false, 0);
+	f_send_paging_ps_exp_one_bss(ts_BssgpP4RAC(pcu_bvc_cfg[0].cell_id.ra_id), g_pars.sgsn_idx, false, 0);
 }
 testcase TC_paging_ps_ptp_rac() runs on test_CT {
 	f_init();
@@ -1446,7 +1446,7 @@
 		rac := 254
 	};
 	/* as it's sent on the PTP BVC, we expect it to pass even for unknown RAC */
-	f_send_paging_ps_exp_one_bss(ts_BssgpP4RAC(unknown_ra), 0, false, 0);
+	f_send_paging_ps_exp_one_bss(ts_BssgpP4RAC(unknown_ra), g_pars.sgsn_idx, false, 0);
 }
 testcase TC_paging_ps_ptp_rac_unknown() runs on test_CT {
 	f_init();
@@ -1458,7 +1458,7 @@
 private function f_TC_paging_ps_ptp_bvci(charstring id) runs on BSSGP_ConnHdlr
 {
 	/* this should be the normal case for MS in READY MM state after a lower layer failure */
-	f_send_paging_ps_exp_one_bss(ts_BssgpP4Bvci(pcu_bvc_cfg[0].bvci), 0, false, 0);
+	f_send_paging_ps_exp_one_bss(ts_BssgpP4Bvci(pcu_bvc_cfg[0].bvci), g_pars.sgsn_idx, false, 0);
 }
 testcase TC_paging_ps_ptp_bvci() runs on test_CT {
 	f_init();
@@ -1485,7 +1485,7 @@
 private function f_TC_paging_ps_reject_ptp_bvci(charstring id) runs on BSSGP_ConnHdlr
 {
 	/* first send the PS-PAGING from SGSN -> PCU */
-	f_send_paging_ps_exp_one_bss(ts_BssgpP4Bvci(pcu_bvc_cfg[0].bvci), 0, false, 0);
+	f_send_paging_ps_exp_one_bss(ts_BssgpP4Bvci(pcu_bvc_cfg[0].bvci), g_pars.sgsn_idx, false, 0);
 	/* then simulate the PS-PAGING-REJECT from the PCU */
 	f_send_paging_ps_rej(use_sig:=false);
 }
@@ -1499,7 +1499,7 @@
 private function f_TC_paging_ps_ptp_bvci_unknown(charstring id) runs on BSSGP_ConnHdlr
 {
 	/* as it's sent on the PTP BVC, we expect it to pass even for unknown BVCI */
-	f_send_paging_ps_exp_one_bss(ts_BssgpP4Bvci(33333), 0, false, 0);
+	f_send_paging_ps_exp_one_bss(ts_BssgpP4Bvci(33333), g_pars.sgsn_idx, false, 0);
 }
 testcase TC_paging_ps_ptp_bvci_unknown() runs on test_CT {
 	f_init();
@@ -1659,7 +1659,7 @@
 		mcc_mnc := '567F99'H,
 		lac := 33333
 	};
-	f_send_paging_ps_exp_no_bss(ts_BssgpP4LAC(unknown_la), 0, true);
+	f_send_paging_ps_exp_no_bss(ts_BssgpP4LAC(unknown_la), g_pars.sgsn_idx, true);
 }
 testcase TC_paging_ps_sig_lac_unknown() runs on test_CT {
 	f_init();
@@ -1696,7 +1696,7 @@
 		},
 		rac := 254
 	};
-	f_send_paging_ps_exp_no_bss(ts_BssgpP4RAC(unknown_ra), 0, true);
+	f_send_paging_ps_exp_no_bss(ts_BssgpP4RAC(unknown_ra), g_pars.sgsn_idx, true);
 }
 testcase TC_paging_ps_sig_rac_unknown() runs on test_CT {
 	f_init();
@@ -1747,7 +1747,7 @@
 /* PS-PAGING on SIG-BVC for unknown BVCI */
 private function f_TC_paging_ps_sig_bvci_unknown(charstring id) runs on BSSGP_ConnHdlr
 {
-	f_send_paging_ps_exp_no_bss(ts_BssgpP4Bvci(33333), 0, true);
+	f_send_paging_ps_exp_no_bss(ts_BssgpP4Bvci(33333), g_pars.sgsn_idx, true);
 }
 testcase TC_paging_ps_sig_bvci_unknown() runs on test_CT {
 	f_init();

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22299
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: Id85ffa57af943ae98c0f022b261ec24d557c236d
Gerrit-Change-Number: 22299
Gerrit-PatchSet: 1
Gerrit-Owner: daniel <dwillmann at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210118/7c1b1011/attachment.htm>