Closed
Bug 1604359
Opened 4 years ago
Closed 4 years ago
Intermittent browser_manager.js | Should be displaying 4 accounts - 4 == 0 - JS frame :: chrome://mochitests/content/browser/comm/mail/test/browser/cloudfile/browser_manager.js :: test_load_accounts_and_properly_order :: line 97
Categories
(Thunderbird :: FileLink, defect, P5)
Thunderbird
FileLink
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 74.0
People
(Reporter: intermittent-bug-filer, Assigned: darktrojan)
Details
(Keywords: intermittent-failure)
Attachments
(1 file)
15.03 KB,
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter Review |
Filed by: geoff [at] darktrojan.net
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=281461814&repo=comm-central
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/at2qj9G9TVmkusy31HFFUA/runs/0/artifacts/public/logs/live_backing.log
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 5•4 years ago
|
||
We should be able to knock this off by rewriting the test in a modern way.
Assignee: nobody → geoff
Status: NEW → ASSIGNED
Attachment #9121457 -
Flags: review?(mkmelin+mozilla)
Comment 6•4 years ago
|
||
Comment on attachment 9121457 [details] [diff] [review] 1604359-cloudfile-manager-test-1.diff Review of attachment 9121457 [details] [diff] [review]: ----------------------------------------------------------------- LGTM, with some nits. r=mkmelin ::: mail/components/preferences/test/browser/browser_cloudfile.js @@ +35,2 @@ > > +let mockPromptService = { Please add @implements {nsIPromptService} @@ +40,5 @@ > + return true; > + }, > + QueryInterface: ChromeUtils.generateQI([Ci.nsIPromptService]), > +}; > +let mockExternalProtocolService = { Can we add some @implements {nsIExternalProtocolService} to make it more clear and less XPCOMy @@ +60,4 @@ > > +let { MockRegistrar } = ChromeUtils.import( > + "resource://testing-common/MockRegistrar.jsm" > +); would be nicer to have the imports on top of the file
Attachment #9121457 -
Flags: review?(mkmelin+mozilla) → review+
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/03dcd29772aa
Rewrite CloudFile preferences test without Mozmill modules; r=mkmelin DONTBUILD
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•4 years ago
|
Target Milestone: --- → Thunderbird 74.0
Assignee | ||
Comment 9•4 years ago
|
||
Flags: needinfo?(geoff)
Comment hidden (Intermittent Failures Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•