Holger Freyther has posted comments on this change. ( https://gerrit.osmocom.org/13029 )
Change subject: nitb_netreg_mass: Add code to declare a run successful/failure ...................................................................... Patch Set 2: (1 comment) https://gerrit.osmocom.org/#/c/13029/1/suites/nitb_netreg_mass/register_default_mass.py File suites/nitb_netreg_mass/register_default_mass.py: https://gerrit.osmocom.org/#/c/13029/1/suites/nitb_netreg_mass/register_default_mass.py@48 PS1, Line 48: if not result.has_lu_time(): > do we really need "count"? It's just len(ms_driver. […] Sure. Good idea. I will store the result in a temp to avoid having to assume that two calls return the same data. -- To view, visit https://gerrit.osmocom.org/13029 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-gsm-tester Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I49a821e8e4656466259e519ef901726cbb76ef86 Gerrit-Change-Number: 13029 Gerrit-PatchSet: 2 Gerrit-Owner: Holger Freyther <hol...@freyther.de> Gerrit-Reviewer: Holger Freyther <hol...@freyther.de> Gerrit-Reviewer: Jenkins Builder (1000002) Gerrit-Reviewer: Pau Espin Pedrol <pes...@sysmocom.de> Gerrit-Comment-Date: Mon, 25 Feb 2019 03:28:08 +0000 Gerrit-HasComments: Yes Gerrit-HasLabels: No