<p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15675">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/15675/1/pcu/PCU_Tests_RAW.ttcn">File pcu/PCU_Tests_RAW.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/15675/1/pcu/PCU_Tests_RAW.ttcn@566">Patch Set #1, Line 566:</a> <code style="font-family:monospace,monospace">f_establish_tbf_ul</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I am not against helper functions in TTCN-3, but sometimes it feels like we're loosing flexibility. And in the end it turns that one helper is calling another helper with calls some other helpers. On the other hand, I am also agree that we need to avoid code duplication.</p><p style="white-space: pre-wrap; word-wrap: break-word;">My suggestion is this particular case is to keep TBF establishment aside, and only check if the received Immediate Assignment (first 'in' parameter) is an Uplink TBF assignment (implemented starting from line "Make sure we received an UL TBF Assignment"). This way we avoid duplication of the arguments of f_establish_tbf().</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15675/1/pcu/PCU_Tests_RAW.ttcn@578">Patch Set #1, Line 578:</a> <code style="font-family:monospace,monospace">?</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This can then be parametrized too, so there would be no need to check if it's a dynamic allocation or not (see below). If some test case would need a Single Block Assignment, it will just pass the corresponding template as a parameter.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15675">change 15675</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/+/15675"/><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: I5b8a7ca5a4d0f905bf68949978811b139568c237 </div>
<div style="display:none"> Gerrit-Change-Number: 15675 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 04 Oct 2019 19:35:54 +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>