Closed Bug 1893146 Opened 1 year ago Closed 11 months ago

Intermittent comm/mail/base/test/widgets/browser_treeView.js | Uncaught exception in test bound testKeyboardAndMouse - 'select' event should get fired - timed out after 50 tries.

Categories

(Thunderbird :: Mail Window Front End, defect, P5)

Tracking

(thunderbird_esr128 fixed)

RESOLVED FIXED
133 Branch
Tracking Status
thunderbird_esr128 --- fixed

People

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

References

Details

(Keywords: intermittent-failure, intermittent-testcase)

Attachments

(2 files)

Filed by: brendan [at] thunderbird.net
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=455558190&repo=comm-central
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/BhwxjTN4SFm4dT_wVWuN0A/runs/0/artifacts/public/logs/live_backing.log


First observed in this push: https://treeherder.mozilla.org/jobs?repo=comm-central&revision=5dbebc71617ffc3951a2c7163869aae9f240319c

So far, all occurrences I've observed have been on OSX, so might be a failure specific to that platform.

This is easy to reproduce locally on linux with --headless. Reproduces in 15sec. For non-headless, can't reproduce.

See Also: → 1885320, 1913364

Obviously not the ideal fix. It does seem the row is there, but clicking on in with --headless does not work for some reason. It doesn't appear to be a timing issue either.
The normal way works in general - for me at least the failing case is index 12 - https://searchfox.org/comm-central/rev/cb79cdf1d4fccd774ca03a05539640eba9bb6f24/mail/base/test/widgets/browser_treeView.js#670
I don't know if it's a code issue or some oddness in headless mode. Nevertheless, it makes finding issues with (--headless --verify) impossible.

Assignee: nobody → mkmelin+mozilla
Status: NEW → ASSIGNED

Adding leave-open. Let's see how it goes.

Target Milestone: --- → 133 Branch

Pushed by alessandro@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/c93bd0c9069b
Try to fix intermittent comm/mail/base/test/widgets/browser_treeView.js | Uncaught exception in test bound testKeyboardAndMouse - 'select' event should get fired - timed out after 50 tries. r=tobyp

Looks like it worked! \o/

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED

Comment on attachment 9432786 [details]
Bug 1893146 - Try to fix intermittent comm/mail/base/test/widgets/browser_treeView.js | Uncaught exception in test bound testKeyboardAndMouse - 'select' event should get fired - timed out after 50 tries. r=#thunderbird-reviewers

[Approval Request Comment]
User impact if declined: none
Testing completed (on c-c, etc.): c-c, beta
Risk to taking this patch (and alternatives if risky): test only, totally safe

Attachment #9432786 - Flags: approval-comm-esr128?

Comment on attachment 9432786 [details]
Bug 1893146 - Try to fix intermittent comm/mail/base/test/widgets/browser_treeView.js | Uncaught exception in test bound testKeyboardAndMouse - 'select' event should get fired - timed out after 50 tries. r=#thunderbird-reviewers

[Triage Comment]
Approved for esr128

Attachment #9432786 - Flags: approval-comm-esr128? → approval-comm-esr128+

Seems to be happening again

Status: RESOLVED → REOPENED
Resolution: FIXED → ---

(In reply to Ben Campbell from comment #25)

Seems to be happening again

Seems fairly permanent on windows opt. I can also reproduce it locally with --verify on Linux. I've tried looking through our and the m-c pushlog from when it started happening again and haven't spotted anything obvious.

The test window is too small in headless mode and seems to have focus issues permanently on Windows and intermittently elsewhere.

Assignee: mkmelin+mozilla → geoff
Status: REOPENED → RESOLVED
Closed: 1 year ago11 months ago
Resolution: --- → FIXED

This is happening on esr128 again as well.

Geoff, uplift the second patch to 128 esr and 137 release?

Flags: needinfo?(geoff)

Comment on attachment 9474556 [details]
Bug 1893146 - Fix browser_treeView.js. r=#thunderbird-reviewers

Uplift Approval Request

  • Please state case for uplift consideration and ensure bug severity is set: Fix for a frequently failing test. I haven't checked this lands cleanly on other trees but I think it should.
  • User impact if declined:
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Daily?: Yes
  • Has the fix been verified in Beta?: No
  • Needs manual test from QA?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky):
  • String changes made/needed:
Flags: needinfo?(geoff)
Attachment #9474556 - Flags: approval-comm-release?
Attachment #9474556 - Flags: approval-comm-esr128?

Comment on attachment 9474556 [details]
Bug 1893146 - Fix browser_treeView.js. r=#thunderbird-reviewers

[Triage Comment]
Approved for beta, per discussion w/ Wayne and Toby on Zoom/Matrix

Attachment #9474556 - Flags: approval-comm-beta+

Comment on attachment 9474556 [details]
Bug 1893146 - Fix browser_treeView.js. r=#thunderbird-reviewers

[Triage Comment]
Approved for esr128
Approved for release

Attachment #9474556 - Flags: approval-comm-release?
Attachment #9474556 - Flags: approval-comm-release+
Attachment #9474556 - Flags: approval-comm-esr128?
Attachment #9474556 - Flags: approval-comm-esr128+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: