Closed
Bug 700071
Opened 9 years ago
Closed 9 years ago
test_errorhandler.js ends up talking directly to Mozilla sync servers rather than local test harness
Categories
(Firefox :: Sync, defect)
Firefox
Sync
Tracking
()
RESOLVED
FIXED
mozilla10
People
(Reporter: rnewman, Assigned: rnewman)
References
Details
Attachments
(1 file)
1.00 KB,
patch
|
philikon
:
review+
|
Details | Diff | Splinter Review |
Guess we clear clusterURL and fall back to the default server. Oops.
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → rnewman
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•9 years ago
|
||
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)
Assignee | ||
Updated•9 years ago
|
Attachment #572246 -
Flags: review?(mconnor) → review?(philipp)
Updated•9 years ago
|
Attachment #572246 -
Flags: review?(philipp) → review+
Assignee | ||
Comment 2•9 years ago
|
||
Pushed to inbound to catch more failures in Bug 700051. https://hg.mozilla.org/integration/mozilla-inbound/rev/1ff33773f17e
Assignee | ||
Comment 3•9 years ago
|
||
If this goes green and happy, I will close Bug 700051, in the belief that this will remove the possibility for such failures.
Comment 4•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/1ff33773f17e
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
Updated•2 years ago
|
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.
Description
•