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)
Tracking
(Not tracked)
People
(Reporter: intermittent-bug-filer, Assigned: darktrojan)
References
Details
(Keywords: intermittent-failure)
Attachments
(2 files)
|
4.14 KB,
patch
|
darktrojan
:
review+
|
Details | Diff | Splinter Review |
|
12.80 KB,
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter Review |
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.
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
Comment 7•6 years ago
|
||
Make sure to wait for the notification to actually appear instead of testing it is already there.
| Assignee | ||
Comment 8•6 years ago
|
||
| Comment hidden (Intermittent Failures Robot) |
Comment 10•6 years ago
|
||
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.
Comment 11•6 years ago
|
||
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
Updated•6 years ago
|
Updated•6 years ago
|
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Comment hidden (Intermittent Failures Robot) |
| Assignee | ||
Updated•6 years ago
|
| Assignee | ||
Comment 25•6 years ago
|
||
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.
| Comment hidden (Intermittent Failures Robot) |
Comment 27•6 years ago
|
||
Comment 28•6 years ago
|
||
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
Description
•