The Loop one call functional test fails in websocket code unless the development servers are running

RESOLVED FIXED in Firefox 34

Status

defect
P1
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Paolo, Assigned: standard8)

Tracking

unspecified
mozilla34
Points:
---
Bug Flags:
qe-verify -

Firefox Tracking Flags

(firefox34 fixed)

Details

Attachments

(1 attachment)

The Loop one call functional test starts its own servers on ports 5001 and 3001. It seems, however, that the test fails in websocket code, unless the development servers are also running on ports 5000 and 3000.
This is either a loop server issue, or a test-config setup issue.

In the post to /calls/<token> the server is returning a progress url with port 5000. Hence, the set-up for the server which tells it that it should be using 5001 isn't working.
One line fix to add a public server address setting for the loop-server. This gets the tests passing again.

I did wonder about fixing the server to set the default port for the public address depending on the PORT env var, but that seemed unnecessarily complicated for something that should probably be specified regardless.
Attachment #8478442 - Flags: review?(dmose)
Assignee: nobody → standard8
Priority: -- → P1
Comment on attachment 8478442 [details] [diff] [review]
Fix functional test setup for Loop to set the correct public server address so that websocket connections work correctly.

r=dmose, conditional on filing a bug on having the server use PORT as the basis for binding the websocket
Attachment #8478442 - Flags: review?(dmose) → review+
https://hg.mozilla.org/integration/fx-team/rev/3d3f1028e29d
Target Milestone: --- → mozilla34
https://hg.mozilla.org/mozilla-central/rev/3d3f1028e29d
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Untracking for QA. Please needinfo me to request testing.
Flags: qe-verify-
QA Contact: anthony.s.hughes
You need to log in before you can comment on or make changes to this bug.