Closed
Bug 963865
Opened 9 years ago
Closed 9 years ago
browser_946320_tabs_from_other_computers.js test fails with Fxa enabled.
Categories
(Firefox :: Sync, defect)
Firefox
Sync
Tracking
()
RESOLVED
FIXED
Firefox 29
People
(Reporter: markh, Assigned: ttaubert)
References
Details
(Whiteboard: [qa?])
Attachments
(1 file)
5.18 KB,
patch
|
markh
:
review+
|
Details | Diff | Splinter Review |
The test browser_946320_tabs_from_other_computers.js checks that after simulating a user being signed into Sync, the relevant sync menu items appears. This user simulation isn't suitable for Firefox Accounts, and thus the test fails. We'll need to fix this before we can enable Fxa by default.
Updated•9 years ago
|
Whiteboard: [qa?]
Comment 1•9 years ago
|
||
(In reply to Mark Hammond [:markh] from comment #0) > The test browser_946320_tabs_from_other_computers.js checks that after > simulating a user being signed into Sync, the relevant sync menu items > appears. This user simulation isn't suitable for Firefox Accounts, and thus > the test fails. We'll need to fix this before we can enable Fxa by default. What exactly goes wrong in fxa? Do you have a try run that demonstrates the problem? There are various sync tests that use a similar pattern...
Assignee | ||
Updated•9 years ago
|
Assignee | ||
Comment 2•9 years ago
|
||
Attachment #8368320 -
Flags: review?(mhammond)
Reporter | ||
Comment 3•9 years ago
|
||
Comment on attachment 8368320 [details] [diff] [review] 0002-Bug-963865-Fix-browser_946320_tabs_from_other_comput.patch Review of attachment 8368320 [details] [diff] [review]: ----------------------------------------------------------------- One day we should invent a testing .jsm that the xpcshell tests can share with the b-c tests, but not this day!
Attachment #8368320 -
Flags: review?(mhammond) → review+
Assignee | ||
Comment 4•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/08d8f9e97fdd
Comment 5•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/08d8f9e97fdd
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 29
You need to log in
before you can comment on or make changes to this bug.
Description
•