Closed Bug 1483766 Opened 2 years ago Closed 2 years ago
Ref Test Implementation doesn't apply timeout _multiplier
In testing/web-platform/tests/tools/wptrunner/wptrunner/executors/executormarionette.py, InternalRefTestImplementation sets self.self.timeout_multiplier but doesn't seem to use it for anything. It calls self.executor.protocol.marionette._send_message(... "timeout": test.timeout * 1000 ...) ... I think it should be multiplying the timeout by self.timeout_multiplier. And in practice I see that debug WPT reftests time out after 10s, when I think it should be 40s (according to testing/web-platform/tests/tools/wptrunner/wptrunner/browsers/firefox.py).
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 < firstname.lastname@example.org>
Attachment #9001546 - Flags: review?(james)
Great response, thanks!
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/43de4e0d1745 Use timeout multiplier for Marionette-backed reftests. r=jgraham
No, thank _you_! (-:
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/12524 for changes under testing/web-platform/tests
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Upstream PR merged
You need to log in before you can comment on or make changes to this bug.