iotests/testrunner.py: move updating last_elapsed to run_tests
We are going to use do_run_test() in multiprocessing environment, where we'll not be able to change original runner object. Happily, the only thing we change is that last_elapsed and it's simple to do it in run_tests() instead. All other accesses to self in do_runt_test() and in run_test() are read-only. Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> Message-Id: <20211203122223.2780098-3-vsementsov@virtuozzo.com> Reviewed-by: John Snow <jsnow@redhat.com> Tested-by: John Snow <jsnow@redhat.com> Signed-off-by: Hanna Reitz <hreitz@redhat.com>
This commit is contained in:
parent
02dd48f859
commit
1f257b70d1
@ -287,7 +287,6 @@ class TestRunner(ContextManager['TestRunner']):
|
||||
diff=diff, casenotrun=casenotrun)
|
||||
else:
|
||||
f_bad.unlink()
|
||||
self.last_elapsed.update(test, elapsed)
|
||||
return TestResult(status='pass', elapsed=elapsed,
|
||||
casenotrun=casenotrun)
|
||||
|
||||
@ -353,6 +352,9 @@ class TestRunner(ContextManager['TestRunner']):
|
||||
print('\n'.join(res.diff))
|
||||
elif res.status == 'not run':
|
||||
notrun.append(name)
|
||||
elif res.status == 'pass':
|
||||
assert res.elapsed is not None
|
||||
self.last_elapsed.update(t, res.elapsed)
|
||||
|
||||
sys.stdout.flush()
|
||||
if res.interrupted:
|
||||
|
Loading…
Reference in New Issue
Block a user