710912 - Permanent orange on Linux: TEST-UNEXPECTED-FAIL | test-newmailaccount.js | test_can_pref_off_account_provisioner

Status: RESOLVED FIXED

(Thunderbird :: Testing Infrastructure, defect)

Description

[Mike Conley (:mconley) (:⚙️)]

This is fallout from bug 700536.  I thought I'd fixed this - I've had one try build pass these tests just fine.  :/

Anyhow, here's the log:

SUMMARY-UNEXPECTED-FAIL | test-newmailaccount.js | test-newmailaccount.js::test_can_pref_off_account_provisioner
  EXCEPTION: Popup did not close!
    at: utils.js line 429
       TimeoutError("Popup did not close!") utils.js 429
       waitFor((function () elem.state == "closed"),"Popup did not close!",1000,50) utils.js 467
       close_popup([object Object],[object Object]) test-folder-display-helpers.js 1405
       test_can_pref_off_account_provisioner() test-newmailaccount.js 834
            frame.js 557
            frame.js 626
            frame.js 669
            frame.js 497
            frame.js 675
            server.js 179
            server.js 183

Comment 1

[Mike Conley (:mconley) (:⚙️)]

Attachment: Patch v1

I've found a solution within Mozmill's Menu API.  Passes locally, and my try build shows that it passes on all platforms (though the tests are orange due to Test Pilot bustage that got fixed this morning by Standard8).

http://build.mozillamessaging.com/tinderboxpushlog/?tree=ThunderbirdTry&rev=831eae8b22ee

http://build.mozillamessaging.com/tinderboxpushlog/?tree=ThunderbirdTry&rev=be7da37878fc

Comment 2

[Blake Winton (:bwinton) (:☕️)]

Comment on attachment 581946 [details] [diff] [review]
Patch v1

Review of attachment 581946 [details] [diff] [review]:
-----------------------------------------------------------------

Seems reasonable.  r=me.

Comment 3

[Mike Conley (:mconley) (:⚙️)]

Committed to comm-central as http://hg.mozilla.org/comm-central/rev/a3bfe58a7e42

Comment 4

[Mike Conley (:mconley) (:⚙️)]

Committed to comm-aurora as http://hg.mozilla.org/releases/comm-aurora/rev/b14a19d774e5