907143 - Intermittent test-message-header.js | test-message-header.js::test_clicking_star_opens_inline_contact_editor

Status: RESOLVED FIXED

(Thunderbird :: Testing Infrastructure, defect)

Description

[Ryan VanderMeulen [:RyanVM]]

https://tbpl.mozilla.org/php/getParsedLog.php?id=26739604&tree=Thunderbird-Trunk

TB Rev3 Fedora 12 comm-central debug test mozmill on 2013-08-19 16:23:02 PDT for push c2ca0b8f4be9
slave: talos-r3-fed-024

TEST-START | /home/cltbld/talos-slave/test/build/mozmill/message-header/test-message-header.js | test_clicking_star_opens_inline_contact_editor
Step Pass: {"function": "controller.click()"}
Step Pass: {"function": "controller.click()"}
WARNING: NS_ENSURE_TRUE(parentObject) failed: file ../../../../../mozilla/accessible/src/atk/AccessibleWrap.cpp, line 1267
++DOMWINDOW == 77 (0xd358a40) [serial = 185] [outer = 0xd423820]
WARNING: gsignal.c:3078: signal name `selection_changed' is invalid for instance `0xc989390': 'glib warning', file ../../../../mozilla/toolkit/xre/nsSigHandlers.cpp, line 127

(thunderbird-bin:3187): GLib-GObject-WARNING **: gsignal.c:3078: signal name `selection_changed' is invalid for instance `0xc989390'
Test Failure: a != b: 'closed' != 'open'.
++DOMWINDOW == 78 (0xbf8bd90) [serial = 186] [outer = 0xd423820]
TEST-UNEXPECTED-FAIL | /home/cltbld/talos-slave/test/build/mozmill/message-header/test-message-header.js | test-message-header.js::test_clicking_star_opens_inline_contact_editor

Comment 1

[Magnus Melin [:mkmelin]]

Attachment: bug907143_test_clicking_star_opens_inline_contact_editor_orange.patch

Probably need to wait... but i guess it's usually very fast.

Comment 2

[Legacy TBPL/Treeherder Robot]

RyanVM
https://tbpl.mozilla.org/php/getParsedLog.php?id=27006876&tree=Thunderbird-Trunk
TB Rev3 Fedora 12x64 comm-central debug test mozmill on 2013-08-26 08:08:41
revision: 4c2345e03ee4
slave: talos-r3-fed64-059

TEST-UNEXPECTED-FAIL | /home/cltbld/talos-slave/test/build/mozmill/message-header/test-message-header.js | test-message-header.js::test_clicking_star_opens_inline_contact_editor
TEST-UNEXPECTED-FAIL | (runtestlist.py) | Exited with code 1 during directory run

Comment 3

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

Comment on attachment 793949 [details] [diff] [review]
bug907143_test_clicking_star_opens_inline_contact_editor_orange.patch

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

Looks good to me - just a removal suggestion.

::: mail/test/mozmill/message-header/test-message-header.js
@@ +8,3 @@
>   */
> +
> +// make SOLO_TEST=message-header/test-message-header.js mozmill-one

I think we can probably remove this.

Comment 4

[Magnus Melin [:mkmelin]]

https://hg.mozilla.org/comm-central/rev/1cb4f9344d8a -> FIXED

I've kept the make comment. Granted it's not necessary but in reality to run just one test you almost have to write down the command somewhere since autocomplete doesn't work as tests aren't directly under the objdir. So it does more good than harm imho.

Comment 5

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

(In reply to Magnus Melin from comment #4)
> https://hg.mozilla.org/comm-central/rev/1cb4f9344d8a -> FIXED
> 
> I've kept the make comment. Granted it's not necessary but in reality to run
> just one test you almost have to write down the command somewhere since
> autocomplete doesn't work as tests aren't directly under the objdir. So it
> does more good than harm imho.

Okie doke, thanks Magnus.