Closed Bug 1831486 Opened 2 years ago Closed 2 years ago

Break up tests to reduce timeouts

Categories

(Thunderbird :: Add-Ons: Extensions API, task)

Product:
Thunderbird
Component:
Add-Ons: Extensions API
Type:
task

Tracking

(thunderbird114 fixed)

Status:
RESOLVED FIXED
Milestone:
115 Branch
Tracking Flags:
Tracking Status
thunderbird114 --- fixed

People

(Reporter: TbSync, Assigned: TbSync)

Details

(Whiteboard: [snnot3p])

Attachments

(2 files, 3 obsolete files)

Bug 1831486 - Try to fix tests hitting timeout threshold by breaking up tests and requesting longer timeouts. r=#thunderbird-reviewers
48 bytes, text/x-phabricator-request
rjl
: approval-comm-beta+
Details | Review
Bug 1831486 - Double timeout for code coverage runs and fission runs, since they are a lot slower. r=#thunderbird-reviewers
48 bytes, text/x-phabricator-request
Details | Review

There are a couple of tests which exceed timeout. Either try to rewrite them, split them up or request longer timeout.

  • mail/components/extensions/test/browser/browser_ext_mailTabs.js
  • mail/components/extensions/test/browser/browser_ext_browserAction_customized.js
  • mail/components/extensions/test/browser/browser_ext_browserAction_not_customized.js
  • mail/components/extensions/test/browser/browser_ext_messageDisplayAction_popup_click_mv3_event_pages.js
Assignee: nobody → john
Status: NEW → ASSIGNED

Great work, John!

Whiteboard: [snnot3p]
Attachment #9331750 - Attachment description: Bug 1831486 - Try to fix test timeouts by breaking up tests and requesting longer timeouts. r=#thunderbird-reviewers → Bug 1831486 - Try to fix tests hitting timeout threshold by breaking up tests and requesting longer timeouts. r=#thunderbird-reviewers
Keywords: checkin-needed-tb
Target Milestone: --- → 114 Branch
Target Milestone: 114 Branch → 115 Branch

Pushed by john@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/ba126ee28416
Try to fix tests hitting timeout threshold by breaking up tests and requesting longer timeouts. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Keywords: checkin-needed-tb
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attachment #9332563 - Attachment is obsolete: true
Attachment #9332564 - Attachment is obsolete: true
Keywords: leave-open
Keywords: leave-open
Attachment #9332626 - Attachment is obsolete: true
Keywords: checkin-needed-tb

Pushed by alessandro@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/0a72adc6d68c
Double timeout for code coverage runs and fission runs, since they are a lot slower. r=#thunderbird-reviewers

Status: REOPENED → RESOLVED
Closed: 2 years ago → 2 years ago
Keywords: checkin-needed-tb
Resolution: --- → FIXED

Do you want this for beta?

Flags: needinfo?(rob)
Summary: Brake up tests to reduce timeouts → Break up tests to reduce timeouts

Comment on attachment 9331750 [details]
Bug 1831486 - Try to fix tests hitting timeout threshold by breaking up tests and requesting longer timeouts. r=#thunderbird-reviewers

[Triage Comment]
approved

Flags: needinfo?(rob)
Attachment #9331750 - Flags: approval-comm-beta+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: