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

Reply via email to