Closed Bug 700071 Opened 8 years ago Closed 8 years ago

test_errorhandler.js ends up talking directly to Mozilla sync servers rather than local test harness

Categories

(Firefox :: Sync, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla10

People

(Reporter: rnewman, Assigned: rnewman)

References

Details

Attachments

(1 file)

Guess we clear clusterURL and fall back to the default server. Oops.
Assignee: nobody → rnewman
Status: NEW → ASSIGNED
This test harness never set serverURL, so as soon as we handled a 401 by clearing clusterURL, it switched to using production. Oops.

Tests pass with this change, with no access to services.mozilla.org.
Attachment #572246 - Flags: review?(mconnor)
Attachment #572246 - Flags: review?(mconnor) → review?(philipp)
Attachment #572246 - Flags: review?(philipp) → review+
If this goes green and happy, I will close Bug 700051, in the belief that this will remove the possibility for such failures.
https://hg.mozilla.org/mozilla-central/rev/1ff33773f17e
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
Component: Firefox Sync: Backend → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.