Closed Bug 1601517 Opened 6 years ago Closed 6 years ago

Intermittent comm/mail/test/browser/cloudfile/browser_notifications.js | Uncaught exception - at resource://testing-common/mozmill/utils.jsm:386 - TimeoutError: Timed out waiting for notification with value bigAttachmentUploading to show.

Categories

(Thunderbird :: FileLink, defect, P5)

defect

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 74.0

People

(Reporter: intermittent-bug-filer, Assigned: darktrojan)

References

Details

(Keywords: intermittent-failure)

Attachments

(2 files)

Filed by: geoff [at] darktrojan.net
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=279530636&repo=comm-central
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/eacvaS6tSdm-h9YhDqSYpw/runs/0/artifacts/public/logs/live_backing.log


This is the same problem that has been causing the crash in Z1 tests for some time.

Make sure to wait for the notification to actually appear instead of testing it is already there.

https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=a8d24cceff7115cff0928df0b34bae11297e7b7f&selectedJob=284579438

Assignee: nobody → mkmelin+mozilla
Status: NEW → ASSIGNED
Attachment #9120332 - Flags: review?(geoff)
Comment on attachment 9120332 [details] [diff] [review] bug1601517_test_fix_browser_notifications.patch Review of attachment 9120332 [details] [diff] [review]: ----------------------------------------------------------------- Seems like a good idea. ::: mail/test/browser/cloudfile/browser_notifications.js @@ +30,5 @@ > var mozmill = ChromeUtils.import( > "resource://testing-common/mozmill/mozmill.jsm" > ); > var { > assert_notification_displayed, Do we still need assert_notification_displayed?
Attachment #9120332 - Flags: review?(geoff) → review+

We do, but it's possible the usages should be converted into waiting instead. It's not entirely trivial, so I'll just leave it for now and we can see if it's needed.

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/78ee796295d6
fix comm/mail/test/browser/cloudfile/browser_notifications.js .... Expected the notification with value bigAttachmentUploading to be shown. r=darktrojan

Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 74.0

Still seeing it :(

Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Summary: Intermittent comm/mail/test/browser/cloudfile/browser_notifications.js | Uncaught exception - at resource://testing-common/mozmill/NotificationBoxHelpers.jsm:79 - Error: Expected the notification with value bigAttachmentUploading to be shown → Intermittent comm/mail/test/browser/cloudfile/browser_notifications.js | Uncaught exception - at resource://testing-common/mozmill/utils.jsm:386 - TimeoutError: Timed out waiting for notification with value bigAttachmentUploading to show.
Assignee: mkmelin+mozilla → geoff
Status: REOPENED → ASSIGNED

Generally, the problems with this test are not that the notification we expect doesn't show up, it's that the notification shows up and has already disappeared by the time we expect it. There's a number of timing variables at play here, not least that some of the faked uploads are set to take 500ms, whether that's a good amount of time or not. With this patch I make the upload continue until told to stop, and ensure that happens after we look for the notification that says an upload is in progress. I also turn off animation of the notification bars (probably unnecessary given the real fix, but hey) and reset a pref that gets set during this test.

https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=1000538be12324bef23f8ed9fb1664463cb6f2ce

Attachment #9139916 - Flags: review?(mkmelin+mozilla)
Comment on attachment 9139916 [details] [diff] [review] 1601517-notifications-test-1.diff Review of attachment 9139916 [details] [diff] [review]: ----------------------------------------------------------------- Hoping it will fix it! r=mkmelin
Attachment #9139916 - Flags: review?(mkmelin+mozilla) → review+

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/3758d7aac181
Fix browser_notifications.js by controlling the timing of events. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 6 years ago6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: