Closed Bug 1856719 Opened 1 year ago Closed 8 months ago

Fix and re-enable Websockets mochitests on Android

Categories

(Core :: Networking: WebSockets, task, P2)

Unspecified
Android
task

Tracking

()

RESOLVED FIXED
124 Branch
Tracking Status
firefox124 --- fixed

People

(Reporter: RyanVM, Assigned: valentin)

References

Details

(Whiteboard: [necko-triaged][necko-priority-queue] )

Attachments

(1 file)

They were disabled a long time ago by bug 982828. I took a quick look on Try and some do appear to pass these days, but overall the failure rate is still pretty high. I'm wondering if they might be relatively easily fixed, however?
https://searchfox.org/mozilla-central/source/dom/websocket/tests/mochitest.ini#3

Severity: -- → N/A
Priority: -- → P2
Whiteboard: [necko-triaged][necko-priority-new]
Whiteboard: [necko-triaged][necko-priority-new] → [necko-triaged][necko-priority-queue]
See Also: → 1544655

Enabling the tests and running on emulator I can see them erroring out with the message here:
https://searchfox.org/mozilla-central/rev/9bc264fbc5d6e618d8f3b9677a8f5e8550b94dbc/testing/mochitest/ssltunnel/ssltunnel.cpp#843

(" could not open connection to the real server\n"));

Not sure if what we need here is to forward the port or to fix something with where we forward it.

Assignee: nobody → moz.valentin

There is an r+ patch which didn't land and no activity in this bug for 2 weeks.
:valentin, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit BugBot documentation.

Flags: needinfo?(moz.valentin)
Flags: needinfo?(kershaw)
Flags: needinfo?(kershaw)

I did a try push with ssltunnel logging to figure out why it's not working on treeherder
https://treeherder.mozilla.org/jobs?repo=try&revision=4e61d3d755ba661cbf2c61764d110a015aaf3ccb

Flags: needinfo?(valentin.gosu)
Pushed by valentin.gosu@gmail.com: https://hg.mozilla.org/integration/autoland/rev/9793d58574ba Make websocket server listen on all interfaces to make Android tests work r=necko-reviewers,kershaw
Status: NEW → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → 124 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: