Closed Bug 1845239 Opened 1 year ago Closed 4 months ago

Fix the broken/disabled tests in browser_rightClickMiddleClickMessages.js

Categories

(Thunderbird :: General, task)

Tracking

(thunderbird128 fixed)

RESOLVED FIXED
129 Branch
Tracking Status
thunderbird128 --- fixed

People

(Reporter: darktrojan, Assigned: darktrojan)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

This test file has a bunch of tests for middle-clicking and right-clicking on things. Some of them have been explicitly skipped since bug 1787094, and some of them appear to have not run since they were converted from Mozmill – the old way of naming test functions "testSomething" and having them magically work was in use, but that doesn't work in Mochitest.

Assignee: geoff → nobody
Status: ASSIGNED → NEW
Assignee: nobody → geoff
Status: NEW → ASSIGNED
Attachment #9406892 - Attachment description: Bug 1845239 - Enable working tests in browser_rightClickMessages.js. r=#thunderbird-reviewers → Bug 1845239 - Fix and enable tests in browser_rightClickMessages.js. r=#thunderbird-reviewers

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/0c0ae3ae9993
Fix and enable tests in browser_rightClickMessages.js. r=aleca,mkmelin

Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → 129 Branch
Blocks: tb128found

Comment on attachment 9406892 [details]
Bug 1845239 - Fix and enable tests in browser_rightClickMessages.js. r=#thunderbird-reviewers

[Approval Request Comment]
Regression caused by (bug #): supernova, probably
User impact if declined: mostly a test fix, but also a slight weirdness when deleting messages by context menu
Testing completed (on c-c, etc.):
Risk to taking this patch (and alternatives if risky):

Attachment #9406892 - Flags: approval-comm-beta?

Comment on attachment 9406892 [details]
Bug 1845239 - Fix and enable tests in browser_rightClickMessages.js. r=#thunderbird-reviewers

[Triage Comment]
Approved for beta

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

Attachment

General

Created:
Updated:
Size: