<p>dexter has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/11131">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">BSC_Tests: use consistant AMR S0-S15 bits<br><br>At the moment we use the default S0-S15 bits for the AMR config,<br>regardless what RSL_IE_Body mr_conf or osmo-bsc.cfg sets.<br><br>- Make sure consistant S0-S15 bits are used for AMR related tests.<br><br>This is a re-submit of change I794e6d4fe8abc67337428cbe0bcc8802fae37a6e,<br>which had to be reverted because the depending patch in osmo-bsc is not<br>yet merged into master. This caused TC_assignment_codec_amr_f and<br>TC_assignment_codec_amr_h to fail.<br><br>Change-Id: Ia98f18ba2c17c85ed01488734dc6df67f5b60d41<br>Depends: osmo-bsc: I2d8ded51b3eb4c003fe2da6f2d6f48d001b73737<br>Related: OS#3529<br>---<br>M bsc/BSC_Tests.ttcn<br>1 file changed, 16 insertions(+), 4 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/31/11131/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 25df52d..4a34310 100644</span><br><span>--- a/bsc/BSC_Tests.ttcn</span><br><span>+++ b/bsc/BSC_Tests.ttcn</span><br><span>@@ -1923,6 +1923,10 @@</span><br><span> testcase TC_assignment_codec_amr_f() runs on test_CT {</span><br><span>         var TestHdlrParams pars := valueof(t_def_TestHdlrPars);</span><br><span>      var MSC_ConnHdlr vc_conn;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+   /* Note: This setups the codec configuration. The parameter payload in</span><br><span style="color: hsl(120, 100%, 40%);">+         * mr_conf must be consistant with the parameter codecElements in pars</span><br><span style="color: hsl(120, 100%, 40%);">+         * and also must match the amr-config in osmo-bsc.cfg! */</span><br><span>    var RSL_IE_Body mr_conf := {</span><br><span>                 other := {</span><br><span>                   len := 2,</span><br><span>@@ -1930,11 +1934,14 @@</span><br><span>          }</span><br><span>    };</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+        pars.ass_codec_list := valueof(ts_BSSMAP_IE_CodecList({ts_CodecAMR_F}));</span><br><span style="color: hsl(120, 100%, 40%);">+      pars.ass_codec_list.codecElements[0].s0_7 := '00000110'B;</span><br><span style="color: hsl(120, 100%, 40%);">+     pars.ass_codec_list.codecElements[0].s8_15 := '01010111'B;</span><br><span style="color: hsl(120, 100%, 40%);">+    pars.expect_mr_conf_ie := mr_conf;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>         f_init(1, true);</span><br><span>     f_sleep(1.0);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-       pars.ass_codec_list := valueof(ts_BSSMAP_IE_CodecList({ts_CodecAMR_F}));</span><br><span style="color: hsl(0, 100%, 40%);">-        pars.expect_mr_conf_ie := mr_conf;</span><br><span>   vc_conn := f_start_handler(refers(f_TC_assignment_codec), pars);</span><br><span>     vc_conn.done;</span><br><span> }</span><br><span>@@ -1942,6 +1949,8 @@</span><br><span> testcase TC_assignment_codec_amr_h() runs on test_CT {</span><br><span>         var TestHdlrParams pars := valueof(t_def_TestHdlrPars);</span><br><span>      var MSC_ConnHdlr vc_conn;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+   /* See note above */</span><br><span>         var RSL_IE_Body mr_conf := {</span><br><span>                 other := {</span><br><span>                   len := 2,</span><br><span>@@ -1949,11 +1958,14 @@</span><br><span>          }</span><br><span>    };</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+        pars.ass_codec_list := valueof(ts_BSSMAP_IE_CodecList({ts_CodecAMR_H}));</span><br><span style="color: hsl(120, 100%, 40%);">+      pars.ass_codec_list.codecElements[0].s0_7 := '00000110'B;</span><br><span style="color: hsl(120, 100%, 40%);">+     pars.ass_codec_list.codecElements[0].s8_15 := '00000111'B;</span><br><span style="color: hsl(120, 100%, 40%);">+    pars.expect_mr_conf_ie := mr_conf;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>         f_init(1, true);</span><br><span>     f_sleep(1.0);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-       pars.ass_codec_list := valueof(ts_BSSMAP_IE_CodecList({ts_CodecAMR_H}));</span><br><span style="color: hsl(0, 100%, 40%);">-        pars.expect_mr_conf_ie := mr_conf;</span><br><span>   vc_conn := f_start_handler(refers(f_TC_assignment_codec), pars);</span><br><span>     vc_conn.done;</span><br><span> }</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/11131">change 11131</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/11131"/><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: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Ia98f18ba2c17c85ed01488734dc6df67f5b60d41 </div>
<div style="display:none"> Gerrit-Change-Number: 11131 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>