pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21508 )
Change subject: wait_no_raise(): set default args ...................................................................... Patch Set 2: 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. -- To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21508 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-gsm-tester Gerrit-Branch: master Gerrit-Change-Id: I225a175b25c5c148ef5e23d2184f5d1aa2f4188a Gerrit-Change-Number: 21508 Gerrit-PatchSet: 2 Gerrit-Owner: neels <nhofm...@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: pespin <pes...@sysmocom.de> Gerrit-Comment-Date: Fri, 04 Dec 2020 17:56:41 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment