The default bug view has changed. See this FAQ.

Port bug 1206133 to Thunderbird - TEST-UNEXPECTED-FAIL | /builds/slave/test/build/tests/mozmill/message-header/test-message-header.js

RESOLVED FIXED in Thunderbird 52.0


6 months ago
6 months ago


(Reporter: Jorg K (GMT+1), Assigned: Jorg K (GMT+1))



Thunderbird 52.0

Firefox Tracking Flags

(Not tracked)



(1 attachment, 2 obsolete attachments)



6 months ago
Looks like bug 1206133 changed popup states here:

This causes Mozmill failures here:

INFO -  SUMMARY-UNEXPECTED-FAIL | test-message-header.js | test-message-header.js::test_address_book_switch_disabled_on_contact_in_mailing_list
INFO -    EXCEPTION: a != b: 'showing' != 'open'.
INFO -      at: test-folder-display-helpers.js line 2858
INFO -         assert_true test-folder-display-helpers.js:2858 11
INFO -         assert_equals test-folder-display-helpers.js:2845 3
INFO -         test_address_book_switch_disabled_on_contact_in_mailing_list test-message-header.js:464 3


6 months ago
Keywords: intermittent-failure
Summary: Port bug 1206133 to Thunderbird → Port bug 1206133 to Thunderbird - TEST-UNEXPECTED-FAIL | /builds/slave/test/build/tests/mozmill/message-header/test-message-header.js

Comment 1

6 months ago
Created attachment 8796608 [details] [diff] [review]

OK, what happens is that instead of "open" we now get "showing".

In total there are four states: "closed", "open", "showing" and "hidden".

Let's be a little more flexible and cater for "open" and "showing" as being open.

With this patch the test passes on current trunk.
Attachment #8796608 - Flags: review?(acelists)

Comment 2

6 months ago
Where is the "positioning" state? Or is that just an event?

Also, why should we consider "showing" as being open, in this particular test only?
We often get popups stuck in "showing" state on Linux for infinite length of time and the test fails. But those problems are intermittent so we attribute them to either mozmill, or some Linux bug in Gtk/XUL backend or something.

So why is this test special?

Comment 3

6 months ago
Kirk, bug 1206133 broke one of our test. We now get the panel state returned as "showing" and not as "open" as it used to be. Most likely this is due to:
Is this expected?
Flags: needinfo?(ksteuber)
Bug 1206133 changed panel behavior. Panels used to open synchronously; now they open asynchronously. This means that tests that do things like this: |; assert(panel.state == "open");| now fail. The fix for this is typically to wait for the popupshown event.
Flags: needinfo?(ksteuber)

Comment 5

6 months ago
Created attachment 8796679 [details] [diff] [review]
Attachment #8796608 - Attachment is obsolete: true
Attachment #8796608 - Flags: review?(acelists)
Attachment #8796679 - Flags: review?(acelists)

Comment 6

6 months ago
Comment on attachment 8796679 [details] [diff] [review]

Review of attachment 8796679 [details] [diff] [review]:

Yes, this seems more correct. Please remove the now redundant assert_equals()s after the waitFor()s (3 occurrences in the patch).
Attachment #8796679 - Flags: review?(acelists) → review+

Comment 7

6 months ago
Created attachment 8796751 [details] [diff] [review]

Addressed review comments and removed those three lines.
Attachment #8796679 - Attachment is obsolete: true
Attachment #8796751 - Flags: review+

Comment 8

6 months ago
(pushed with DONTBUILD since we're waiting for bug 456053 to fix its test failures.)
Last Resolved: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 52.0
You need to log in before you can comment on or make changes to this bug.