Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
neels has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 ) Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. add bsc/BSC_Tests_VAMOS.ttcn BSC_Tests_VAMOS.ttcn is separate from BSC_Tests.ttcn in order to instruct osmo-bts-omldummy to pass BTS_FEAT_VAMOS == true in the OML BTS attributes. Add tests: TC_chan_act_to_vamos() TC_mode_modify_to_vamos_fr() TC_mode_modify_to_vamos_hr() TC_assign_to_secondary_lchan_fr() TC_assign_to_secondary_lchan_hr() TC_vamos_multiplex_tch_f_tch_f() TC_vamos_multiplex_tch_h_tch_h_tch_h_tch_h() Change-Id: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 --- M bsc/BSC_Tests.ttcn A bsc/BSC_Tests_VAMOS.ttcn M library/L3_Templates.ttcn M library/RSL_Types.ttcn 4 files changed, 859 insertions(+), 6 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn index 2b3703f..41deea1 100644 --- a/bsc/BSC_Tests.ttcn +++ b/bsc/BSC_Tests.ttcn @@ -20,6 +20,8 @@ * level testing. */ +friend module BSC_Tests_VAMOS; + import from Misc_Helpers all; import from General_Types all; import from Osmocom_Types all; @@ -673,7 +675,7 @@ uint8_t mp_exp_ms_power_level := 7; } -private function f_gen_test_hdlr_pars(integer bssap_idx := 0) return TestHdlrParams { +friend function f_gen_test_hdlr_pars(integer bssap_idx := 0) return TestHdlrParams { var TestHdlrParams pars := valueof(t_def_TestHdlrPars); if (mp_bssap_cfg[bssap_idx].transport == BSSAP_TRANSPORT_AoIP) { @@ -799,7 +801,7 @@ } -private function f_shutdown_helper() runs on test_CT { +friend function f_shutdown_helper() runs on test_CT { all component.stop; setverdict(pass); mtc.stop; @@ -1001,7 +1003,7 @@ "sccp-timer iar " & int2str(g_bsc_sccp_timer_iar)}); } -private function f_logp(TELNETasp_PT pt, charstring log_msg) +friend function f_logp(TELNETasp_PT pt, charstring log_msg) { // log on TTCN3 log output log(log_msg); @@ -6602,7 +6604,7 @@ } } -private function f_perform_clear(RSL_DCHAN_PT rsl, template PDU_ML3_NW_MS exp_rr_rel_tmpl := tr_RRM_RR_RELEASE) +friend function f_perform_clear(RSL_DCHAN_PT rsl, template PDU_ML3_NW_MS exp_rr_rel_tmpl := tr_RRM_RR_RELEASE) runs on MSC_ConnHdlr { f_logp(BSCVTY, "MSC instructs BSC to clear channel"); BSSAP.send(ts_BSSMAP_ClearCommand(0)); diff --git a/bsc/BSC_Tests_VAMOS.ttcn b/bsc/BSC_Tests_VAMOS.ttcn new file mode 100644 index 000..bf487cf --- /dev/null +++ b/bsc/BSC_Tests_VAMOS.ttcn @@ -0,0 +1,738 @@ +module BSC_Tests_VAMOS { + +/* Integration Tests for OsmoBSC + * (C) 2021 by sysmocom - s.f.m.c. GmbH + * All rights reserved. + * + * Released under the terms of GNU General Public License, Version 2 or + * (at your option) any later version. + * + * SPDX-License-Identifier: GPL-2.0-or-later + * + * This test suite tests OsmoBSC while emulating both multiple BTS + MS as + * well as the MSC. See README for more details. + * + * There are test cases that run in so-called 'handler mode' and test cases + * that run directly on top of the BSSAP and RSL CodecPorts. The "handler mode" + * tests abstract the multiplexing/demultiplexing of multiple SCCP connections + * and/or RSL channels and are hence suitable for higher-level test cases, while + * the "raw" tests directly on top of the CodecPorts are more suitable for lower- + * level testing. + */ + +import from BSC_Tests all; + +import from Misc_Helpers all; +import from General_Types all; +import from Osmocom_Types all; +import from GSM_Types all; +import from IPL4asp_Types all; + +import from BSSAP_Types all; +import from RAN_Adapter all; +import from BSSAP_LE_Adapter all; +import from BSSAP_LE_CodecPort all; +import from BSSAP_LE_Types all; +import from BSSLAP_Types all; +import from BSSAP_CodecPort all; +import from BSSMAP_Templates all; +import from IPA_Emulation all; +import from IPA_CodecPort all; +import from IPA_Types all; +import from IPA_Testing all; +import from RSL_Types all; +import from RSL_Emulation all; +import from MGCP_Emulation all; +import from MGCP_Templates all; +import from MGCP_Types all; +import from MGCP_CodecPort all; + +import from Osmocom_CTRL_Functions all; +import from Osmocom_CTRL_Types all; +import from Osmocom_CTRL_Adapter all; + +import from StatsD_Types all; +import from StatsD_CodecPort all; +import from StatsD_CodecPort_CtrlFunct all; +import from StatsD_Checker all; + +import from Osmocom_VTY_Functions all; +import from TELNETasp_PortType all; + +import from MobileL3_CommonIE_Types all; +import from MobileL3_Types all; +import from MobileL3_RRM_Types all; +import from L3_Templates all; +import from GSM_RR_Types all; + +import from SCCP_Templates all; +import from BSSMAP_Templates all; +import from BSSMAP_LE_Templates all; + +import from SCCPasp_Types all; + +import from GSM_SystemInformation all;
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 ) Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. Patch Set 14: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 14 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: laforge Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-Comment-Date: Sat, 05 Jun 2021 14:52:15 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 ) Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. Patch Set 13: (1 comment) https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/10/bsc/BSC_Tests_VAMOS.ttcn File bsc/BSC_Tests_VAMOS.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/10/bsc/BSC_Tests_VAMOS.ttcn@89 PS10, Line 89: var integer subslot; > I would set it to 0, otherwise you may return an unbound value. if a test calls this function on an unimplemented channel type, we should rather get an error message, so added that instead. -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 13 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: laforge Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-Comment-Date: Thu, 03 Jun 2021 12:22:31 + Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria Gerrit-MessageType: comment
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
Hello Jenkins Builder, laforge, fixeria, pespin, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 to look at the new patch set (#13). Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. add bsc/BSC_Tests_VAMOS.ttcn BSC_Tests_VAMOS.ttcn is separate from BSC_Tests.ttcn in order to instruct osmo-bts-omldummy to pass BTS_FEAT_VAMOS == true in the OML BTS attributes. Add tests: TC_chan_act_to_vamos() TC_mode_modify_to_vamos_fr() TC_mode_modify_to_vamos_hr() TC_assign_to_secondary_lchan_fr() TC_assign_to_secondary_lchan_hr() TC_vamos_multiplex_tch_f_tch_f() TC_vamos_multiplex_tch_h_tch_h_tch_h_tch_h() Change-Id: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 --- M bsc/BSC_Tests.ttcn A bsc/BSC_Tests_VAMOS.ttcn M library/L3_Templates.ttcn M library/RSL_Types.ttcn 4 files changed, 859 insertions(+), 6 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/11/24411/13 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 13 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: laforge Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-MessageType: newpatchset
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
Hello Jenkins Builder, laforge, fixeria, pespin, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 to look at the new patch set (#11). Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. add bsc/BSC_Tests_VAMOS.ttcn BSC_Tests_VAMOS.ttcn is separate from BSC_Tests.ttcn in order to instruct osmo-bts-omldummy to pass BTS_FEAT_VAMOS == true in the OML BTS attributes. Add tests: TC_chan_act_to_vamos() TC_mode_modify_to_vamos_fr() TC_mode_modify_to_vamos_hr() TC_assign_to_secondary_lchan_fr() TC_assign_to_secondary_lchan_hr() TC_vamos_multiplex_tch_f_tch_f() TC_vamos_multiplex_tch_h_tch_h_tch_h_tch_h() Change-Id: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 --- M bsc/BSC_Tests.ttcn A bsc/BSC_Tests_VAMOS.ttcn M library/L3_Templates.ttcn M library/RSL_Types.ttcn 4 files changed, 843 insertions(+), 6 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/11/24411/11 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 11 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: laforge Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-MessageType: newpatchset
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 ) Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. Patch Set 10: Code-Review-1 (5 comments) https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/10/bsc/BSC_Tests_VAMOS.ttcn File bsc/BSC_Tests_VAMOS.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/10/bsc/BSC_Tests_VAMOS.ttcn@4 PS10, Line 4: * (C) 2017-2018 by Harald Welte Copyright: should be sysmocom, it's an entirely new file. https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/10/bsc/BSC_Tests_VAMOS.ttcn@89 PS10, Line 89: var integer subslot; I would set it to 0, otherwise you may return an unbound value. https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/10/bsc/BSC_Tests_VAMOS.ttcn@114 PS10, Line 114: var BIT5 rsl_cbits; Same here, one can easily get a DTE with this function. https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/10/bsc/BSC_Tests_VAMOS.ttcn@135 PS10, Line 135: var BIT5 rr_cbits; + https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/10/bsc/BSC_Tests_VAMOS.ttcn@158 PS10, Line 158: var boolean fr; + -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 10 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: laforge Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-Comment-Date: Tue, 01 Jun 2021 02:12:59 + Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 ) Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. Patch Set 10: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 10 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: laforge Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-Comment-Date: Mon, 31 May 2021 08:11:03 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 ) Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. Patch Set 10: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 10 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: laforge Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-Comment-Date: Sun, 30 May 2021 09:14:50 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
Hello Jenkins Builder, fixeria, pespin, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 to look at the new patch set (#10). Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. add bsc/BSC_Tests_VAMOS.ttcn BSC_Tests_VAMOS.ttcn is separate from BSC_Tests.ttcn in order to instruct osmo-bts-omldummy to pass BTS_FEAT_VAMOS == true in the OML BTS attributes. Add tests: TC_chan_act_to_vamos() TC_mode_modify_to_vamos_fr() TC_mode_modify_to_vamos_hr() TC_assign_to_secondary_lchan_fr() TC_assign_to_secondary_lchan_hr() TC_vamos_multiplex_tch_f_tch_f() TC_vamos_multiplex_tch_h_tch_h_tch_h_tch_h() Change-Id: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 --- M bsc/BSC_Tests.ttcn A bsc/BSC_Tests_VAMOS.ttcn M library/L3_Templates.ttcn M library/RSL_Types.ttcn 4 files changed, 836 insertions(+), 6 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/11/24411/10 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 10 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-MessageType: newpatchset
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
Hello Jenkins Builder, fixeria, pespin, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 to look at the new patch set (#9). Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. add bsc/BSC_Tests_VAMOS.ttcn BSC_Tests_VAMOS.ttcn is separate from BSC_Tests.ttcn in order to instruct osmo-bts-omldummy to pass BTS_FEAT_VAMOS == true in the OML BTS attributes. Add tests: TC_mode_modify_to_vamos() TC_chan_act_to_vamos() TC_assign_to_secondary_lchan() TC_vamos_multiplex_tch_f_tch_f() Change-Id: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 --- M bsc/BSC_Tests.ttcn A bsc/BSC_Tests_VAMOS.ttcn M library/L3_Templates.ttcn M library/RSL_Types.ttcn 4 files changed, 627 insertions(+), 6 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/11/24411/9 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 9 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-MessageType: newpatchset
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
Hello Jenkins Builder, fixeria, pespin, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 to look at the new patch set (#8). Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. add bsc/BSC_Tests_VAMOS.ttcn BSC_Tests_VAMOS.ttcn is separate from BSC_Tests.ttcn in order to instruct osmo-bts-omldummy to pass BTS_FEAT_VAMOS == true in the OML BTS attributes. Add tests: TC_mode_modify_to_vamos() TC_chan_act_to_vamos() TC_assign_to_secondary_lchan() TC_vamos_multiplex_tch_f_tch_f() Change-Id: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 --- M bsc/BSC_Tests.ttcn A bsc/BSC_Tests_VAMOS.ttcn M library/L3_Templates.ttcn M library/RSL_Types.ttcn 4 files changed, 627 insertions(+), 6 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/11/24411/8 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 8 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-MessageType: newpatchset
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
Hello Jenkins Builder, fixeria, pespin, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 to look at the new patch set (#7). Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. add bsc/BSC_Tests_VAMOS.ttcn BSC_Tests_VAMOS.ttcn is separate from BSC_Tests.ttcn in order to instruct osmo-bts-omldummy to pass BTS_FEAT_VAMOS == true in the OML BTS attributes. Add tests: TC_mode_modify_to_vamos() TC_chan_act_to_vamos() TC_assign_to_secondary_lchan() TC_vamos_multiplex_tch_f_tch_f() Change-Id: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 --- M bsc/BSC_Tests.ttcn A bsc/BSC_Tests_VAMOS.ttcn M library/L3_Templates.ttcn M library/RSL_Types.ttcn 4 files changed, 625 insertions(+), 6 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/11/24411/7 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 7 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-MessageType: newpatchset
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 ) Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. Patch Set 6: (2 comments) https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/4/bsc/BSC_Tests_VAMOS.ttcn File bsc/BSC_Tests_VAMOS.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/4/bsc/BSC_Tests_VAMOS.ttcn@116 PS4, Line 116: var RslChannelNr chan_nr := rsl.ies[0].body.chan_nr; > With the recent RSL chan_nr changes, we should probably also do: […] misunderstanding, this simply activates a primary lchan in VAMOS speech mode https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/4/bsc/BSC_Tests_VAMOS.ttcn@174 PS4, Line 174: RSL.receive(tr_RSL_MODE_MODIFY_REQ(g_chan_nr, ?)) -> value rsl; > Would also be good to match chan_nr against t_RslChanNr_Osmo_VAMOS_Bm. same here, this modifies a primary lchan into VAMOS speech mode -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 6 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-Comment-Date: Fri, 28 May 2021 12:54:13 + Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria Gerrit-MessageType: comment
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
Hello Jenkins Builder, fixeria, pespin, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 to look at the new patch set (#6). Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. add bsc/BSC_Tests_VAMOS.ttcn BSC_Tests_VAMOS.ttcn is separate from BSC_Tests.ttcn in order to instruct osmo-bts-omldummy to pass BTS_FEAT_VAMOS == true in the OML BTS attributes. Add tests: TC_mode_modify_to_vamos() TC_chan_act_to_vamos() TC_assign_to_secondary_lchan() TC_vamos_multiplex_tch_f_tch_f() Change-Id: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 --- M bsc/BSC_Tests.ttcn A bsc/BSC_Tests_VAMOS.ttcn 2 files changed, 384 insertions(+), 4 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/11/24411/6 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 6 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-MessageType: newpatchset
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 ) Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. Patch Set 5: Code-Review+1 (2 comments) https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/4/bsc/BSC_Tests_VAMOS.ttcn File bsc/BSC_Tests_VAMOS.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/4/bsc/BSC_Tests_VAMOS.ttcn@116 PS4, Line 116: var RslChannelNr chan_nr := rsl.ies[0].body.chan_nr; With the recent RSL chan_nr changes, we should probably also do: if (not match(chan_nr, t_RslChanNr_Osmo_VAMOS_Bm(1)) { ... } https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/4/bsc/BSC_Tests_VAMOS.ttcn@174 PS4, Line 174: RSL.receive(tr_RSL_MODE_MODIFY_REQ(g_chan_nr, ?)) -> value rsl; Would also be good to match chan_nr against t_RslChanNr_Osmo_VAMOS_Bm. -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 5 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-Comment-Date: Thu, 27 May 2021 16:13:55 + Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
Hello Jenkins Builder, pespin, fixeria, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 to look at the new patch set (#5). Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. add bsc/BSC_Tests_VAMOS.ttcn BSC_Tests_VAMOS.ttcn is separate from BSC_Tests.ttcn in order to instruct osmo-bts-omldummy to pass BTS_FEAT_VAMOS == true in the OML BTS attributes. Add tests: TC_mode_modify_to_vamos() TC_chan_act_to_vamos() TC_assign_to_secondary_lchan() TC_vamos_multiplex_tch_f_tch_f() Change-Id: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 --- M bsc/BSC_Tests.ttcn A bsc/BSC_Tests_VAMOS.ttcn 2 files changed, 384 insertions(+), 4 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/11/24411/5 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 5 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-MessageType: newpatchset
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 ) Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. Patch Set 4: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 4 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-Comment-Date: Thu, 27 May 2021 10:47:47 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
Hello Jenkins Builder, pespin, fixeria, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 to look at the new patch set (#4). Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. add bsc/BSC_Tests_VAMOS.ttcn BSC_Tests_VAMOS.ttcn is separate from BSC_Tests.ttcn in order to instruct osmo-bts-omldummy to pass BTS_FEAT_VAMOS == true in the OML BTS attributes. Add tests: TC_mode_modify_to_vamos() TC_chan_act_to_vamos() TC_assign_to_secondary_lchan() TC_vamos_multiplex_tch_f_tch_f() Change-Id: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 --- M bsc/BSC_Tests.ttcn A bsc/BSC_Tests_VAMOS.ttcn 2 files changed, 384 insertions(+), 4 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/11/24411/4 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 4 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-MessageType: newpatchset
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 ) Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. Patch Set 3: Code-Review+1 re-add pau's +1 after whitespace fix -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 3 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-Comment-Date: Wed, 26 May 2021 15:25:32 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
Hello Jenkins Builder, pespin, fixeria, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 to look at the new patch set (#3). Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. add bsc/BSC_Tests_VAMOS.ttcn BSC_Tests_VAMOS.ttcn is separate from BSC_Tests.ttcn in order to instruct osmo-bts-omldummy to pass BTS_FEAT_VAMOS == true in the OML BTS attributes. Add tests: TC_mode_modify_to_vamos() TC_chan_act_to_vamos() TC_assign_to_secondary_lchan() TC_vamos_multiplex_tch_f_tch_f() Change-Id: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 --- M bsc/BSC_Tests.ttcn A bsc/BSC_Tests_VAMOS.ttcn 2 files changed, 382 insertions(+), 3 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/11/24411/3 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 3 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: pespin Gerrit-MessageType: newpatchset
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 ) Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 2 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: pespin Gerrit-Comment-Date: Wed, 26 May 2021 15:22:54 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 ) Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. Patch Set 2: thanks for reducing this file -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 2 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: pespin Gerrit-Comment-Date: Wed, 26 May 2021 15:21:24 + Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
neels has removed a vote from this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 ) Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. Removed Code-Review-2 by fixeria -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 2 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: pespin Gerrit-MessageType: deleteVote
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
Hello Jenkins Builder, fixeria, pespin, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 to look at the new patch set (#2). Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. add bsc/BSC_Tests_VAMOS.ttcn BSC_Tests_VAMOS.ttcn is separate from BSC_Tests.ttcn in order to instruct osmo-bts-omldummy to pass BTS_FEAT_VAMOS == true in the OML BTS attributes. Add tests: TC_mode_modify_to_vamos() TC_chan_act_to_vamos() TC_assign_to_secondary_lchan() TC_vamos_multiplex_tch_f_tch_f() Change-Id: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 --- M bsc/BSC_Tests.ttcn A bsc/BSC_Tests_VAMOS.ttcn 2 files changed, 382 insertions(+), 3 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/11/24411/2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 2 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: pespin Gerrit-MessageType: newpatchset
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 ) Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. Patch Set 1: Code-Review-2 (2 comments) This change introduces a lot of duplicated records/functions. You can avoid this using the 'friend'-imports, see ETSI ES 201 873-1, section 8.2.4 "Definition of friend modules". We already use this feature a lot in BTS_Tests[SMSCB,OML,perf].ttcn. https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/1/bsc/BSC_Tests_VAMOS.ttcn File bsc/BSC_Tests_VAMOS.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/1/bsc/BSC_Tests_VAMOS.ttcn@94 PS1, Line 94: SystemInformationConfig This record can be imported from BSC_Tests (check 'friend' keyword), no need to duplicate it here. https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/1/bsc/BSC_Tests_VAMOS.ttcn@109 PS1, Line 109: SystemInformationConfig_omit Same here and down below. Maintaining copies of the same records/functions is going to be painful, so let's avoid it as much as possible. -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 1 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria Gerrit-Reviewer: pespin Gerrit-Comment-Date: Wed, 26 May 2021 14:12:26 + Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in osmo-ttcn3-hacks[master]: add bsc/BSC_Tests_VAMOS.ttcn
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 ) Change subject: add bsc/BSC_Tests_VAMOS.ttcn .. Patch Set 1: Code-Review+1 (1 comment) https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/1/bsc/BSC_Tests_VAMOS.ttcn File bsc/BSC_Tests_VAMOS.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411/1/bsc/BSC_Tests_VAMOS.ttcn@1327 PS1, Line 1327: whitespace -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/24411 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: I2c504099163a30ea102cbd26d3615ca2e5ce1e64 Gerrit-Change-Number: 24411 Gerrit-PatchSet: 1 Gerrit-Owner: neels Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin Gerrit-Comment-Date: Wed, 26 May 2021 10:06:55 + Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment