<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">[...] but I don't like that instead of introducing a separate test case for (RR) Channel Release EARFCNs, you're integrating more checks into the existing tests.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I see it as a test that has N EARFCNs configured and checks that these show up where they should: during init and during chan rel</p><p style="white-space: pre-wrap; word-wrap: break-word;">If we made these separate tests, we would blow up the number of tests: duplicate the 12 TC_si2quater_N_earfcns(), creating 12 more lines in the test results, 12 more tcpdumps taken, and so on. For what gain?<br>The TC_si2quater_* would configure the EARFCNs and initialize the BTS links; and the 12 new rr_chan_rel tests would literally do the same, and then after that start a DCHAN and release. They simply and logically follow from the si2quater tests.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Of course the test names then are slightly inaccurate, I intended to submit a renaming patch and discuss there whether we'd rather keep test names unchanging to not (temporarily) clutter test results summary in jenkins.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I suggest to make f_expect_chan_rel() return the received Channel Release message (instead of overloading it with even more functionality)</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">hmm, the mode and style of that function seems to be to pass things that it should expect during channel release. If I return the CSFB cells instead, then what about the other expect_* arguments, should I return those as well? What about future additions of expected items, return those? Seems to me that passing an expected cell list for CSFB is the best match here.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19216">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19216">change 19216</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/+/19216"/><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: I882c5e1f70bcc4833fc837a95c900ce291919cc5 </div>
<div style="display:none"> Gerrit-Change-Number: 19216 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </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: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 03 Aug 2020 22:50:12 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>