The default bug view has changed. See this FAQ.

Frequent Random Orange: TEST-UNEXPECTED-FAIL | test-plugin-outdated.js | test_outdated_plugin_notification (Notification value was not correct: 'null' == 'null')

RESOLVED FIXED in Thunderbird 25.0

Status

Thunderbird
Testing Infrastructure
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: standard8, Assigned: aceman)

Tracking

({intermittent-failure})

Trunk
Thunderbird 25.0
intermittent-failure

Thunderbird Tracking Flags

(thunderbird24+ fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
https://tbpl.mozilla.org/php/getParsedLog.php?id=24849952&tree=Thunderbird-Trunk

TEST-START | /Users/cltbld/talos-slave/test/build/mozmill/content-tabs/test-plugin-outdated.js | test_outdated_plugin_notification
Step Pass: {"function": "controller.click()"}
Step Pass: {"function": "controller.waitFor()"}
Test Failure: Notification value was not correct: 'null' == 'null'.
TEST-UNEXPECTED-FAIL | /Users/cltbld/talos-slave/test/build/mozmill/content-tabs/test-plugin-outdated.js | test-plugin-outdated.js::test_outdated_plugin_notification
(Reporter)

Updated

4 years ago
Keywords: intermittent-failure
(Assignee)

Comment 1

4 years ago
'null' == 'null' ?

This looks like something bug 880798 wants to fix. But that one is for xpcshell and this failure is in mozmill.
Changing platforms as the failure is also seen on Linux (on TB-trunk).
Assignee: nobody → acelists
OS: Mac OS X → All
(Assignee)

Comment 2

4 years ago
Created attachment 771445 [details] [diff] [review]
patch

Aryx, this would need a try run again, all systems (except Win), mozmill only.
Flags: needinfo?(archaeopteryx)
Pushed to Thunderbird-Try: https://tbpl.mozilla.org/?tree=Thunderbird-Try&showall=1&rev=079ff8a50151
Flags: needinfo?(archaeopteryx)
(Assignee)

Comment 4

4 years ago
Comment on attachment 771445 [details] [diff] [review]
patch

There are some build failures on opt targets but in the debug targets where the build succeeded it seems the test no longer fails.

(For the build failures, PR_USEC_PER_SEC is defined in msgCore.h which nsAutoSyncManager.cpp is not including.)
Attachment #771445 - Flags: review?(mbanner)
(Reporter)

Comment 5

4 years ago
Comment on attachment 771445 [details] [diff] [review]
patch

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

Looks great, thanks!
Attachment #771445 - Flags: review?(mbanner) → review+
(Reporter)

Comment 6

4 years ago
https://hg.mozilla.org/comm-central/rev/cb74c0bcdd73
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 25.0
(Reporter)

Updated

4 years ago
tracking-thunderbird24: --- → +
(Reporter)

Comment 7

4 years ago
Comment on attachment 771445 [details] [diff] [review]
patch

[Triage Comment]
Another one bites the dust. So lets fix this on aurora as well :-)
Attachment #771445 - Flags: approval-comm-aurora+
(Reporter)

Comment 8

4 years ago
https://hg.mozilla.org/releases/comm-aurora/rev/3777b122c29d
status-thunderbird24: --- → fixed
You need to log in before you can comment on or make changes to this bug.