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)

defect

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 74.0

People

(Reporter: intermittent-bug-filer, Assigned: darktrojan)

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

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 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
Target Milestone: --- → Thunderbird 74.0

Where did test_external_link() go?

Flags: needinfo?(geoff)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: