Patch Set 4:

File src/osmo_gsm_tester/

Line 32:     def __init__(self, fail_type='', fail_msg=''):
> * what exactly is the fail_type? is it junit specific? let's document here,
fail_type is just a keyword to define the type of failure, same as in you have 
types of errors or exceptions. JUNit report makes use of this on the output for 

I will make the parameters mandatory then.

Line 123:                 fmsg =  e.fail_msg + '\n' + 
> (double space but whatever)

Line 144:     def set_fail(self, fail_type='', fail_message='', tb=True):
> (again require args instead of default-empty?)

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: Iedf6d912b3cce3333a187a4ac6d5c6b70fe9d5c5
Gerrit-PatchSet: 4
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <>
Gerrit-Reviewer: Pau Espin Pedrol <>
Gerrit-Reviewer: neels <>
Gerrit-HasComments: Yes

Reply via email to