Closed Bug 1058111 Opened 6 years ago Closed 6 years ago

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

Categories

(Hello (Loop) :: Client, defect, P1)

defect

Tracking

(firefox34 fixed)

RESOLVED FIXED
mozilla34
Tracking Status
firefox34 --- fixed

People

(Reporter: Paolo, Assigned: standard8)

Details

Attachments

(1 file)

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/mozilla-central/rev/3d3f1028e29d
Status: NEW → RESOLVED
Closed: 6 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.