Revert "nitb_netreg_mass: Add code to declare a run successful/failure"
This reverts commit b4ad8d72431df2336b9c7fbc6eb9597daa80a35c.
Reason for revert: There was a true dependency on earlier changes (len(mobiles)). Rollback until the resource configuration is merged.
Change-Id: If0f5e24cec2f0d1d144b464180b341cacbdf0ebb
diff --git a/src/osmo_ms_driver/location_update_test.py b/src/osmo_ms_driver/location_update_test.py
index f14bd28..8500607 100644
--- a/src/osmo_ms_driver/location_update_test.py
+++ b/src/osmo_ms_driver/location_update_test.py
@@ -23,11 +23,9 @@
from datetime import timedelta
-import collections
import time
class LUResult(Results):
- """Representation of a Location Updating Result."""
def __init__(self, name):
super().__init__(name)
@@ -46,10 +44,6 @@
def lu_delay(self):
return self.lu_time() - self.start_time()
-
-LUStats = collections.namedtuple("LUStats", ["num_attempted", "num_completed",
- "min_latency", "max_latency"])
-
class MassUpdateLocationTest(log.Origin):
"""
A test to launch a configurable amount of MS and make them
@@ -213,24 +207,9 @@
max_value = result.lu_delay()
return min_value, max_value
- def get_result_values(self):
- """
- Returns the raw result values of the test run in any order.
- """
- return self._results.values()
-
- def get_stats(self):
- """
- Returns a statistical summary of the test.
- """
- attempted = self._number_of_ms
- completed = attempted - self._outstanding
- min_latency, max_latency = self.find_min_max(filter(lambda x: x.has_lu_time(), self._results.values()))
- return LUStats(attempted, completed, min_latency, max_latency)
-
def print_stats(self):
- stats = self.get_stats()
- all_completed = stats.num_attempted == stats.num_completed
+ all_completed = self.all_completed()
+ min_value, max_value = self.find_min_max(filter(lambda x: x.has_lu_time(), self._results.values()))
self.log("Tests done", all_completed=all_completed,
- min=stats.min_latency, max=stats.max_latency)
+ min=min_value, max=max_value)