<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 style="white-space: pre-wrap; word-wrap: break-word;">oh I thought last discussion we had about this concluded this partial including was not supported in titan, but looks like it is?</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 style="white-space: pre-wrap; word-wrap: break-word;">In general I'm for testing stuff in general for all tests, but I'm not really sure we want to run stuff through the VTY every time someone establishes a dedicated channel, since this really impacts time and how tests execute 8as in timing of events).</p><p style="white-space: pre-wrap; word-wrap: break-word;">Since the function runs in BSC_ConnHdlr, what about setting a boolean there to call this verify function?</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 style="white-space: pre-wrap; word-wrap: break-word;">So you are actualyl doing what I requested, but you are not checking the var in all places where the verify function is called. Please do so and create an extra test enabling it (or enable it in an existing test for which it makes sense).</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: Mon, 29 Jun 2020 15:27:41 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>