<p style="white-space: pre-wrap; word-wrap: break-word;">I think Neels might have a different opinion, but I agree that the tests in TTCN-3 are much more maintainable than unit tests that use hard-coded byte arrays that are impossible to be parsed/edited by a human being.  Yes, the unit tests execute much quicker.  Which is maybe a reason/motivation to make the TTCN-3 test execute quicker, and to think about moving the tests from the osmo-ttcn3-hacks.git into the individual osmo-* repositories?</p><p style="white-space: pre-wrap; word-wrap: break-word;">In any case, we cannot loose test coverage. So removing the unit tests can only be done once it is ensured that all tests we used to have here are present in SGSN_Tests.ttcn.</p><p><a href="https://gerrit.osmocom.org/9258">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/9258">change 9258</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/9258"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-sgsn </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Iac1c8854b5ea4aa03279990390ebc110c979aac2 </div>
<div style="display:none"> Gerrit-Change-Number: 9258 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 23 May 2018 16:22:36 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>