Pau Espin Pedrol 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 1: Code-Review+1 (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: count = count + 1 do we really need "count"? It's just len(ms_driver.get_result_values()) -- 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: 1 Gerrit-Owner: Holger Freyther <hol...@freyther.de> Gerrit-Reviewer: Pau Espin Pedrol <pes...@sysmocom.de> Gerrit-Comment-Date: Sat, 23 Feb 2019 22:44:24 +0000 Gerrit-HasComments: Yes Gerrit-HasLabels: Yes