<p>neels has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/25660">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">bsc: add const list of TSC expected per BTS<br><br>Reduce code (comment) dup by having one const definition for default<br>TSCs instead of magic numbers.<br><br>Will add another use of this in upcoming patch testing correct TSC in<br>handover.<br><br>Related: SYS#4895<br>Change-Id: I3733ebbbfd74630e095a08b83023974aad177b34<br>---<br>M bsc/BSC_Tests.ttcn<br>1 file changed, 16 insertions(+), 6 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/60/25660/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn</span><br><span>index a4def7b..9e4ddff 100644</span><br><span>--- a/bsc/BSC_Tests.ttcn</span><br><span>+++ b/bsc/BSC_Tests.ttcn</span><br><span>@@ -75,6 +75,8 @@</span><br><span> import from GSM_RestOctets all;</span><br><span> import from TCCConversion_Functions all;</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+type record of integer integer_list;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> const integer NUM_BTS := 3;</span><br><span> const integer NUM_BTS_CFG := 4; /* we have 4 BTS in the osmo-bsc.cfg (for inter-BSC HO tests) but use only 3 */</span><br><span> const integer NUM_MSC := 3;</span><br><span>@@ -85,6 +87,18 @@</span><br><span> const integer NUM_TCHF_PER_BTS := 4;</span><br><span> const integer NUM_SDCCH_PER_BTS := 3;</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+/* Default Training Sequence Code expected for bts[i]:</span><br><span style="color: hsl(120, 100%, 40%);">+ * BTS 0 has BSIC 10 (and no explicit timeslot training_sequence_code config), so expecting TSC = (BSIC & 7) = 2.</span><br><span style="color: hsl(120, 100%, 40%);">+ * BTS 1 has BSIC 11, TSC = (BSIC & 7) = 3.</span><br><span style="color: hsl(120, 100%, 40%);">+ * BTS 2 has BSIC 12, TSC = (BSIC & 7) = 4.</span><br><span style="color: hsl(120, 100%, 40%);">+ * BTS 2 has BSIC 12, TSC = (BSIC & 7) = 4.</span><br><span style="color: hsl(120, 100%, 40%);">+ */</span><br><span style="color: hsl(120, 100%, 40%);">+const integer_list BTS_TSC := {</span><br><span style="color: hsl(120, 100%, 40%);">+       2,</span><br><span style="color: hsl(120, 100%, 40%);">+    3,</span><br><span style="color: hsl(120, 100%, 40%);">+    4,</span><br><span style="color: hsl(120, 100%, 40%);">+    4</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span> </span><br><span> /* per-BTS state which we keep */</span><br><span> type record BTS_State {</span><br><span>@@ -699,9 +713,7 @@</span><br><span>       }</span><br><span>    pars.exp_ms_power_level := mp_exp_ms_power_level;</span><br><span>    pars.mscpool.bssap_idx := bssap_idx;</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-    /* BTS 0 has BSIC 10 (and no explicit timeslot training_sequence_code config), so expecting TSC = (BSIC & 7) = 2 */</span><br><span style="color: hsl(0, 100%, 40%);">- pars.expect_tsc := 2;</span><br><span style="color: hsl(120, 100%, 40%);">+ pars.expect_tsc := BTS_TSC[0];</span><br><span> </span><br><span>   return pars;</span><br><span> }</span><br><span>@@ -9735,9 +9747,7 @@</span><br><span>    var MSC_ConnHdlr vc_conn2;</span><br><span>   pars2.imsi := pars1.imsi;</span><br><span>    pars2.media_nr := 2;</span><br><span style="color: hsl(0, 100%, 40%);">-    /* f_tc_cm_reestablishment_2 uses 'bts 1'.</span><br><span style="color: hsl(0, 100%, 40%);">-       * BTS 1 has BSIC 11 (and no explicit timeslot training_sequence_code config), so expecting TSC = (BSIC & 7) = 3 */</span><br><span style="color: hsl(0, 100%, 40%);">- pars2.expect_tsc := 3;</span><br><span style="color: hsl(120, 100%, 40%);">+        pars2.expect_tsc := BTS_TSC[1];</span><br><span> </span><br><span>  f_init(2, true, guard_timeout := 40.0);</span><br><span>      f_sleep(1.0);</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/25660">change 25660</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/+/25660"/><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: I3733ebbbfd74630e095a08b83023974aad177b34 </div>
<div style="display:none"> Gerrit-Change-Number: 25660 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>