Bug 1328654 - Clean up after Marionette runner in MarionetteHarness; r=whimboo

This fixes the case where the call to runner.run_tests() raises
an exception before any tests start, thus leaving a
browser instance running.

MozReview-Commit-ID: LFcCZcJJsgf

--HG--
extra : rebase_source : 99a55b534a5260cb791d9dca24364592d24d5d72
This commit is contained in:
Maja Frydrychowicz 2017-01-04 10:09:54 -05:00
parent 6f2a7b8c3a
commit a6ebb1c134

View File

@ -68,7 +68,10 @@ class MarionetteHarness(object):
self.process_args()
tests = self.args.pop('tests')
runner = self._runner_class(**self.args)
runner.run_tests(tests)
try:
runner.run_tests(tests)
finally:
runner.cleanup()
return runner.failed + runner.crashed