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)

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: 4 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: