[tor-commits] [ooni-probe/master] Make the director pass Measurements to the report

art at torproject.org art at torproject.org
Sat Sep 28 20:34:29 UTC 2013


commit 0226bdb130827511f1ec7347a2c1e9d1281ff53a
Author: aagbsn <aagbsn at extc.org>
Date:   Wed Sep 18 13:44:31 2013 +0200

    Make the director pass Measurements to the report
    
    Simply passing a failure instance does not provide any contextual
    information, such as the input that was used, or headers. Instead, we
    should always write the report object, even in case of failure, and
    NetTests should try to add useful information to the report in their
    errbacks.
---
 ooni/director.py |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ooni/director.py b/ooni/director.py
index 50323fd..628b812 100644
--- a/ooni/director.py
+++ b/ooni/director.py
@@ -178,7 +178,7 @@ class Director(object):
         log.msg("Successfully completed measurement: %s" % measurement)
         self.totalMeasurementRuntime += measurement.runtime
         self.successfulMeasurements += 1
-        return measurement.testInstance.report
+        return measurement
 
     def measurementFailed(self, failure, measurement):
         log.msg("Failed doing measurement: %s" % measurement)
@@ -186,7 +186,7 @@ class Director(object):
 
         self.failedMeasurements += 1
         self.failures.append((failure, measurement))
-        return failure
+        return measurement
 
     def reporterFailed(self, failure, net_test):
         """





More information about the tor-commits mailing list