<p>dexter has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/11129">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Revert "BSC_Tests: use consistant AMR S0-S15 bits"<br><br>The change depends on another change in osmo-bsc, which<br>is not in master yet. Because of this TC_assignment_codec_amr_f<br>and TC_assignment_codec_amr_h are currently failing. So lets<br>revert this patch and re-submit it later.<br><br>See also: osmo-bsc change I2d8ded51b3eb4c003fe2da6f2d6f48d001b73737<br><br>This reverts commit 7f5609ad3e65098cca3d79565a1aa80460c49bed.<br><br>Change-Id: Ib16d14c723773ce67508c7e6028e594c15779506<br>---<br>M bsc/BSC_Tests.ttcn<br>1 file changed, 4 insertions(+), 16 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/29/11129/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 4a34310..25df52d 100644</span><br><span>--- a/bsc/BSC_Tests.ttcn</span><br><span>+++ b/bsc/BSC_Tests.ttcn</span><br><span>@@ -1923,10 +1923,6 @@</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(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-       /* Note: This setups the codec configuration. The parameter payload in</span><br><span style="color: hsl(0, 100%, 40%);">-   * mr_conf must be consistant with the parameter codecElements in pars</span><br><span style="color: hsl(0, 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>@@ -1934,14 +1930,11 @@</span><br><span>          }</span><br><span>    };</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.ass_codec_list.codecElements[0].s0_7 := '00000110'B;</span><br><span style="color: hsl(0, 100%, 40%);">-       pars.ass_codec_list.codecElements[0].s8_15 := '01010111'B;</span><br><span style="color: hsl(0, 100%, 40%);">-      pars.expect_mr_conf_ie := mr_conf;</span><br><span style="color: hsl(0, 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(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.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>@@ -1949,8 +1942,6 @@</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(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 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>@@ -1958,14 +1949,11 @@</span><br><span>          }</span><br><span>    };</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.ass_codec_list.codecElements[0].s0_7 := '00000110'B;</span><br><span style="color: hsl(0, 100%, 40%);">-       pars.ass_codec_list.codecElements[0].s8_15 := '00000111'B;</span><br><span style="color: hsl(0, 100%, 40%);">-      pars.expect_mr_conf_ie := mr_conf;</span><br><span style="color: hsl(0, 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(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.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/11129">change 11129</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/11129"/><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: Ib16d14c723773ce67508c7e6028e594c15779506 </div>
<div style="display:none"> Gerrit-Change-Number: 11129 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>