[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [stem/master] Drop unused run_target from test runner



commit 08c290474d9f78c87016420458e2f4e517975c16
Author: Damian Johnson <atagar@xxxxxxxxxxxxxx>
Date:   Tue May 9 10:51:22 2017 -0700

    Drop unused run_target from test runner
    
    Our Runner class provided a run_target attribute but nothing used it. This is
    because testing requirements are against the torrc instead.
---
 run_tests.py   | 2 +-
 test/runner.py | 5 +----
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/run_tests.py b/run_tests.py
index fc6d906..f4f49c0 100755
--- a/run_tests.py
+++ b/run_tests.py
@@ -233,7 +233,7 @@ def main():
       error_tracker.set_category(target)
 
       try:
-        integ_runner.start(target, args.attribute_targets, args.tor_path, extra_torrc_opts = test.util.get_torrc_entries(target))
+        integ_runner.start(args.attribute_targets, args.tor_path, extra_torrc_opts = test.util.get_torrc_entries(target))
 
         println('Running tests...\n', STATUS)
 
diff --git a/test/runner.py b/test/runner.py
index 69af65d..4b4d29d 100644
--- a/test/runner.py
+++ b/test/runner.py
@@ -147,7 +147,6 @@ class _MockChrootFile(object):
 
 class Runner(object):
   def __init__(self):
-    self.run_target = None
     self.attribute_targets = []
 
     self._runner_lock = threading.RLock()
@@ -166,12 +165,11 @@ class Runner(object):
 
     self._original_recv_message = None
 
-  def start(self, run_target, attribute_targets, tor_cmd, extra_torrc_opts):
+  def start(self, attribute_targets, tor_cmd, extra_torrc_opts):
     """
     Makes temporary testing resources and starts tor, blocking until it
     completes.
 
-    :param Target run_target: configuration we're running with
     :param list attribute_targets: **Targets** for our non-configuration attributes
     :param str tor_cmd: command to start tor with
     :param list extra_torrc_opts: additional torrc options for our test instance
@@ -180,7 +178,6 @@ class Runner(object):
     """
 
     with self._runner_lock:
-      self.run_target = run_target
       self.attribute_targets = attribute_targets
 
       # if we're holding on to a tor process (running or not) then clean up after



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits