[v3,24/28] framework/test/base: Remove timeout parameter

Submitted by Dylan Baker on Oct. 31, 2016, 5:50 p.m.

Details

Message ID 5ef51fb35e4a1d687099995a5a440bd79b7ca4bf.1477936071.git-series.dylan@pnwbakers.com
State New
Headers show
Series "Series without cover letter" ( rev: 1 ) in Piglit

Not browsing as part of any series.

Commit Message

Dylan Baker Oct. 31, 2016, 5:50 p.m.
This doesn't work and never has, setting it would cause an exception,
and I'm not exactly sure how to fix it. It's also unused, so we can
bring it back later if we need it.

Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
---
 framework/test/base.py | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

Patch hide | download patch | download mbox

diff --git a/framework/test/base.py b/framework/test/base.py
index 6b7cab6..4e7c8b2 100644
--- a/framework/test/base.py
+++ b/framework/test/base.py
@@ -175,7 +175,7 @@  class Test(object):
     __slots__ = ['run_concurrent', 'env', 'result', 'cwd', '_command']
     timeout = None
 
-    def __init__(self, command, run_concurrent=False, timeout=None):
+    def __init__(self, command, run_concurrent=False):
         assert isinstance(command, list), command
 
         self.run_concurrent = run_concurrent
@@ -183,9 +183,6 @@  class Test(object):
         self.env = {}
         self.result = TestResult()
         self.cwd = None
-        if timeout is not None:
-            assert isinstance(timeout, int)
-            self.timeout = timeout
 
     def execute(self, path, log, options):
         """ Run a test