581921 - Random orange: TEST-UNEXPECTED-FAIL | test-reply-to-list-from-address-selection.js | test_Reply_To_List_From_Address

Status: RESOLVED FIXED

(Thunderbird :: General, defect)

Description

[Mark Banner (:standard8)]

We're getting a fairly frequent random failure of this test on our Windows tinderboxes:

TEST-UNEXPECTED-FAIL | e:\buildbot\win32-comm-central-check\build\mail\test\mozmill\message-header\test-reply-to-list-from-address-selection.js | test_Reply_To_List_From_Address
  EXCEPTION: Timed out waiting for window open!
    at: test-window-helpers.js line 193
       Error("Timed out waiting for window open!")  0
       WindowWatcher_waitForWindowOpen("msgcompose") test-window-helpers.js 193
       wait_for_new_window("msgcompose") test-window-helpers.js 504
       wait_for_compose_window() test-compose-helpers.js 166
       open_compose_with_reply_to_list() test-compose-helpers.js 121
       test_Reply_To_List_From_Address() test-reply-to-list-from-address-selection.js 133
            frame.js 474
            frame.js 530
            frame.js 573
            frame.js 417
            frame.js 579
            server.js 164
            server.js 168
TEST-PASS |  teardownModule

Full log: http://tinderbox.mozilla.org/showlog.cgi?log=Thunderbird/1280116501.1280120592.12218.gz#err0

Ideas/patches would be appreciated to help keep the tree green.

Comment 1

[Mark Banner (:standard8)]

Attachment: The fix

I'm hoping this will fix the issue. From what I can tell, the original version of the test_Reply_To_List_From_Address would do this:

1) Select a folder
2) Select a message in the folder list (no wait for load)
3) Plan for a window that never opens
4) Opens the message in a tab (no wait for load)
5) Try to respond to message.

With the patch we'll do:

1) Select a folder
2) Select a message
3) Wait for the message to finish loading
4) Try to respond to the message.

I've got a rubber-stamp (rs) from bwinton to land this test-only change:

http://hg.mozilla.org/comm-central/rev/b47b4511c5a8

Comment 2

[Mark Banner (:standard8)]

I'm hoping that patch will fix it, so tentatively marking as fixed.