Closed
Bug 1604359
Opened 6 years ago
Closed 6 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•6 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•6 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: 6 years ago
Resolution: --- → FIXED
| Assignee | ||
Updated•6 years ago
|
Target Milestone: --- → Thunderbird 74.0
| Assignee | ||
Comment 9•6 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
•