<p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/18998">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/18998/1/msc/BSC_ConnectionHandler.ttcn">File msc/BSC_ConnectionHandler.ttcn:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/18998/1/msc/BSC_ConnectionHandler.ttcn@62">Patch Set #1, Line 62:</a> <code style="font-family:monospace,monospace">import from TCCConversion_Functions { function f_strstr };</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">oh I thought last discussion we had about this concluded this partial including was not supported in […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I found this in some ttcn3 explaining PDF,<br>but vadim already told me that partial includes all amount to "all" anyway.<br>I thought it would be nice to indicate why I'm including this file, but if you insist we can make it 'all'</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/18998/1/msc/BSC_ConnectionHandler.ttcn@370">Patch Set #1, Line 370:</a> <code style="font-family:monospace,monospace"> f_verify_vty_lac_ci(verify_vlr := false);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">In general I'm for testing stuff in general for all tests, but I'm not really sure we want to run st […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I didn't see a noticeable change in time the tests take to run. The VTY request and analysis of response is pretty instantaneous AFAICT. Are you having a different experience?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/18998/1/msc/MSC_Tests.ttcn">File msc/MSC_Tests.ttcn:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/18998/1/msc/MSC_Tests.ttcn@411">Patch Set #1, Line 411:</a> <code style="font-family:monospace,monospace">            verify_cell_id := mp_enable_cell_id_test and verify_cell_id</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">So you are actualyl doing what I requested, but you are not checking the var in all places where the […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not following, doing what you requested??</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/18998">change 18998</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/+/18998"/><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: Ie410714a96353f74a52a104c56fa0a08683e0004 </div>
<div style="display:none"> Gerrit-Change-Number: 18998 </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-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 03 Jul 2020 23:36:02 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>