Closed Bug 1904400 Opened 3 months ago Closed 3 months ago

[wpt-sync] Sync PR 46881 - Don't honor per-request timeout for now

Categories

(Testing :: web-platform-tests, task, P4)

task

Tracking

(firefox129 fixed)

RESOLVED FIXED
129 Branch
Tracking Status
firefox129 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 46881 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/46881
Details from upstream follow.

Jonathan Lee <jonathanjlee@google.com> wrote:

Don't honor per-request timeout for now

This is a partial revert of #46869. Using a nondefault timeout fails with:

  ▶ ERROR [expected OK] external/wpt/IndexedDB/idbindex-query-exception-order.html
  └   → WebDriverRun.run_func didn't set a result

Exception in thread Thread-4:
Traceback (most recent call last):
  File "/tmp/wpt/tools/webdriver/webdriver/transport.py", line 259, in _request
    previous_timeout = self._conn.gettimeout()
AttributeError: 'HTTPConnection' object has no attribute 'gettimeout'

... because {get,set}timeout are methods on socket.socket, not HTTP(S)Connection.

Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → INVALID
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/530e2b02d9da [wpt PR 46881] - [wptrunner] Don't honor per-request timeout for now, a=testonly
Test result changes from PR not available.
Status: REOPENED → RESOLVED
Closed: 3 months ago3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 129 Branch
You need to log in before you can comment on or make changes to this bug.