Bug 1483766 - Use timeout multiplier for Marionette-backed reftests. r=jgraham

timeout_multiplier is set in InternalRefTestImplementation's
constructor but not used.  Instead it appears to only be using the
test's timeout value.

According to the bug report by roc, reftests time out after 10
seconds, when it should be 40.

Thanks-to: Robert O'Callahan <roc@ocallahan.org>
This commit is contained in:
Andreas Tolfsen 2018-08-16 10:24:52 +01:00
Родитель 5296c85fd5
Коммит 9cb0e90464
1 изменённых файлов: 2 добавлений и 1 удалений

Просмотреть файл

@ -816,11 +816,12 @@ class InternalRefTestImplementation(object):
def run_test(self, test):
references = self.get_references(test)
timeout = (test.timeout * 1000) * self.timeout_multiplier
rv = self.executor.protocol.marionette._send_message("reftest:run",
{"test": self.executor.test_url(test),
"references": references,
"expected": test.expected(),
"timeout": test.timeout * 1000})["value"]
"timeout": timeout})["value"]
return rv
def get_references(self, node):