It is helpful when the timeout happens and the tester can tune it
Signed-off-by: Shu Ming <shuming(a)linux.vnet.ibm.com>
---
tests/test_model.py | 11 +++++++++--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/tests/test_model.py b/tests/test_model.py
index 4a78fbc..650b298 100644
--- a/tests/test_model.py
+++ b/tests/test_model.py
@@ -640,6 +640,11 @@ class ModelTests(unittest.TestCase):
if not inst.get_capabilities()['system_report_tool']:
raise unittest.SkipTest("Without debug report tool")
+ try:
+ timeout = int(os.environ['TEST_REPORT_TIMEOUT'])
+ except (ValueError, KeyError):
+ timeout = 120
+
namePrefix = 'unitTestReport'
# sosreport always deletes unsual letters like '-' and '_' in
the
# generated report file name.
@@ -656,12 +661,14 @@ class ModelTests(unittest.TestCase):
task = inst.debugreports_create({'name': reportName})
rollback.prependDefer(inst.debugreport_delete, reportName)
taskid = task['id']
- self._wait_task(inst, taskid, 60)
+ self._wait_task(inst, taskid, timeout)
self.assertEquals('finished',
inst.task_lookup(taskid)['status'],
"It is not necessary an error. "
"You may need to increase the "
- "timeout number in _wait_task()")
+ "timeout number by "
+ "TEST_REPORT_TIMEOUT=200 "
+ "./run_tests.sh test_model")
report_list = inst.debugreports_get_list()
self.assertTrue(reportName in report_list)
except OperationFailed, e:
--
1.8.1.4