[Piglit] [PATCH v2 2/6] framework/tests/base_tests.py: fix descriptions of two timeout tests

baker.dylan.c at gmail.com baker.dylan.c at gmail.com
Fri Oct 23 18:17:20 PDT 2015

From: Dylan Baker <baker.dylan.c at gmail.com>

These tests claimed to test the timeout mechanism, but what they're
really testing is the status changing ability of the timeout mechanism.

Signed-off-by: Dylan Baker <dylanx.c.baker at intel.com>
 framework/tests/base_tests.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/framework/tests/base_tests.py b/framework/tests/base_tests.py
index caef9e3..105352f 100644
--- a/framework/tests/base_tests.py
+++ b/framework/tests/base_tests.py
@@ -62,7 +62,7 @@ def test_run_return_early():
 def test_timeout():
-    """test.base.Test.run(): kills tests that exceed timeout when set"""
+    """test.base.Test.run(): Sets status to 'timeout' when timeout exceeded"""
     utils.binary_check('sleep', 1)
     class _Test(Test):
@@ -77,7 +77,7 @@ def test_timeout():
 def test_timeout_pass():
-    """test.base.Test.run(): Result is returned when timeout is set but not exceeded
+    """test.base.Test.run(): Doesn't change status when timeout not exceeded

More information about the Piglit mailing list