<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I mean, we shouldn't rely on what osmo-bsc tells us about it, since we are actually testing it. We should know from the test what to expect without asking. Perhaps passing g_pars.expect_channel_mode_modify there or similar would help?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I actually disagree the way f_establish_fully() currently does all sorts of magic decisions like this.<br>I would much more appreciate a test that expects what should happen 1:1 and not assume/infer important parts of the test in the way f_establish_fully() does.<br>But I am not going to completely refactor f_establish_fully(), we're using that everywhere and it is likely going to cause quite some effort.<br>I'm also not going to duplicate and modify that code, because I'd have to also duplicate a lot of the channel activation testing code and, as ugly to me as it is, it is apparently working quite well for the last years.<br>The quickest way to achieving a working Mode Modify test is asking the BSC for the current channel mode.</p><p style="white-space: pre-wrap; word-wrap: break-word;">At first I thought, indeed, we could also use that mad f_rslem_get_last_act() to recall the last channel activation message and read the mode from that. But in fact we would also have to check whether the last activation was ACKed or NACKed, whether another Channel Mode Modify happened in the meantime and changed the mode, and whether the channel was released or not. So, directly asking the BSC for the current actual mode is the most accurate and least error prone way to keep using f_establish_fully().</p><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/23807">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/+/23807">change 23807</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/+/23807"/><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: If30e2cec65da91cb5899ee29e2afb6696437a4c9 </div>
<div style="display:none"> Gerrit-Change-Number: 23807 </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-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 20 Apr 2021 14:40:26 +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>