<p>Harald Welte <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/10707">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">bts_tests: do not define bs_ag_blks_res in multiple locations<br><br>The parameter bs_ag_blks_res is currently defined in ts_SI3_default and<br>in each PagingTestCfg record. When the parameter is changed in<br>PagingTestCfg it must be changed in ts_SI3_default as well in order to<br>have coherent settings in the stestsuit and in the BTS. This can be<br>quite confusing so lets use the setting from ts_SI3_default when<br>populating PagingTestCfg to have the value of the constant in one place<br>only.<br><br>Change-Id: I5cbdb31c8398aa241faca34e850388d5e1682167<br>Related: OS#1575<br>---<br>M bts/BTS_Tests.ttcn<br>1 file changed, 8 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/bts/BTS_Tests.ttcn b/bts/BTS_Tests.ttcn</span><br><span>index 890be91..fd2e50f 100644</span><br><span>--- a/bts/BTS_Tests.ttcn</span><br><span>+++ b/bts/BTS_Tests.ttcn</span><br><span>@@ -1692,9 +1692,10 @@</span><br><span>  *  - that CCCH LOAD IND (PCH) are being generated</span><br><span>  *  - that CCCH LOAD IND (PCH) [no load] is received after paging flood is over */</span><br><span> testcase TC_paging_imsi_80percent() runs on test_CT {</span><br><span style="color: hsl(120, 100%, 40%);">+     var SystemInformation si3 := valueof(ts_SI3_default);</span><br><span>        var PagingTestCfg cfg := {</span><br><span>           combined_ccch := true,</span><br><span style="color: hsl(0, 100%, 40%);">-          bs_ag_blks_res := 1,</span><br><span style="color: hsl(120, 100%, 40%);">+          bs_ag_blks_res := si3.payload.si3.ctrl_chan_desc.bs_ag_blks_res,</span><br><span>             load_factor := 0.8,</span><br><span>          exp_load_ind := true,</span><br><span>                exp_overload := false,</span><br><span>@@ -1715,9 +1716,10 @@</span><br><span>  *  - that CCCH LOAD IND (PCH) are being generated</span><br><span>  *  - that CCCH LOAD IND (PCH) [no load] is received after paging flood is over */</span><br><span> testcase TC_paging_tmsi_80percent() runs on test_CT {</span><br><span style="color: hsl(120, 100%, 40%);">+  var SystemInformation si3 := valueof(ts_SI3_default);</span><br><span>        var PagingTestCfg cfg := {</span><br><span>           combined_ccch := true,</span><br><span style="color: hsl(0, 100%, 40%);">-          bs_ag_blks_res := 1,</span><br><span style="color: hsl(120, 100%, 40%);">+          bs_ag_blks_res := si3.payload.si3.ctrl_chan_desc.bs_ag_blks_res,</span><br><span>             load_factor := 0.8,</span><br><span>          exp_load_ind := true,</span><br><span>                exp_overload := false,</span><br><span>@@ -1738,9 +1740,10 @@</span><br><span>  *  - that CCCH LOAD IND (PCH) are being generated and reach 0 at some point</span><br><span>  *  - that CCCH LOAD IND (PCH) [no load] is received after paging flood is over */</span><br><span> testcase TC_paging_imsi_200percent() runs on test_CT {</span><br><span style="color: hsl(120, 100%, 40%);">+       var SystemInformation si3 := valueof(ts_SI3_default);</span><br><span>        var PagingTestCfg cfg := {</span><br><span>           combined_ccch := true,</span><br><span style="color: hsl(0, 100%, 40%);">-          bs_ag_blks_res := 1,</span><br><span style="color: hsl(120, 100%, 40%);">+          bs_ag_blks_res := si3.payload.si3.ctrl_chan_desc.bs_ag_blks_res,</span><br><span>             load_factor := 2.0,</span><br><span>          exp_load_ind := true,</span><br><span>                exp_overload := true,</span><br><span>@@ -1763,9 +1766,10 @@</span><br><span>  *  - that CCCH LOAD IND (PCH) are being generated and reach 0 at some point</span><br><span>  *  - that CCCH LOAD IND (PCH) [no load] is received after paging flood is over */</span><br><span> testcase TC_paging_tmsi_200percent() runs on test_CT {</span><br><span style="color: hsl(120, 100%, 40%);">+        var SystemInformation si3 := valueof(ts_SI3_default);</span><br><span>        var PagingTestCfg cfg := {</span><br><span>           combined_ccch := true,</span><br><span style="color: hsl(0, 100%, 40%);">-          bs_ag_blks_res := 1,</span><br><span style="color: hsl(120, 100%, 40%);">+          bs_ag_blks_res := si3.payload.si3.ctrl_chan_desc.bs_ag_blks_res,</span><br><span>             load_factor := 2.0,</span><br><span>          exp_load_ind := true,</span><br><span>                exp_overload := true,</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/10707">change 10707</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/10707"/><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-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: I5cbdb31c8398aa241faca34e850388d5e1682167 </div>
<div style="display:none"> Gerrit-Change-Number: 10707 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>