osmo-trx[master]: tests: Timeval: adapt to have reproducible output and enable...

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Max gerrit-no-reply at lists.osmocom.org
Wed Jan 10 09:35:30 UTC 2018


Patch Set 1:

This is odd - if test output is always empty, how can we be sure that the test was executed at all? I think it's better to log at least what kind of test is executing. Even better would be to log smth in case of error instead of using assert: if we hit assert than we don't know what happens with the test code beyond assert point. If we log error and continue, we might see several errors at once and the test would still fail due to output difference. Although it might be matter of personal preference.

-- 
To view, visit https://gerrit.osmocom.org/5708
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1a79892ba3c934879a171789e0edb357277acae4
Gerrit-PatchSet: 1
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <msuraev at sysmocom.de>
Gerrit-HasComments: No



More information about the gerrit-log mailing list