<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 2:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't like this really much. I would favour only using the wait() method and handling the exception in the test. Let's keep the amount of APIs we need to maintain in tests as small as possible please.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I looked and can in fact just use wait().</p><p style="white-space: pre-wrap; word-wrap: break-word;">But actually it was *you* who added wait_no_raise to the testenv explicitly as test API in 927344b4.<br>Admittedly three years ago.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I guess you should remove that then<br>(or apply this patch so that wait_no_raise is sanely usable)</p><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21508">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21508">change 21508</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-gsm-tester/+/21508"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-gsm-tester </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I225a175b25c5c148ef5e23d2184f5d1aa2f4188a </div>
<div style="display:none"> Gerrit-Change-Number: 21508 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </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: Sun, 06 Dec 2020 14:53:38 +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>