Closed Bug 1814884 Opened 3 years ago Closed 3 years ago

Add generic marionette test that checks startup with a clean profile and verifies no error console messages are created

Categories

(Firefox :: General, task, P3)

Desktop
All
task

Tracking

()

RESOLVED FIXED
113 Branch
Tracking Status
firefox111 --- wontfix
firefox112 --- wontfix
firefox113 --- fixed

People

(Reporter: Gijs, Assigned: Gijs)

References

Details

Attachments

(1 file)

We keep having issues where people accidentally add console spam. This is annoying when debugging issues with users (and trying to find the salient bits), and often reflects errors in implementation/testing if they were not otherwise caught before landing in nightly.

To remedy this, I think it'd help if we added a marionette test that starts the browser with a clean new profile, and with an existing profile, and waits for startup to be complete (including all the idle tasks etc.) and then checks that there are no (pending) console messages.

We'll initially need an allowlist of things that already exist, but hopefully this would clamp down on patches that introduce new errors/message spam.

Recent examples: bug 1813320, bug 1814688, bug 1743495, etc.

The Bugbug bot thinks this bug should belong to the 'DevTools::Console' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: General → Console
Product: Firefox → DevTools
Component: Console → General
Product: DevTools → Firefox
See Also: → 1821644
See Also: → 1821648
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
See Also: → 1825431
Summary: Add generic marionette test that checks startup with a clean profile and an upgraded profile and verifies no error console messages are created → Add generic marionette test that checks startup with a clean profile and verifies no error console messages are created
Attachment #9325866 - Attachment description: WIP: Bug 1814884 - add marionette test to ensure we catch newly introduced startup console spew, r?#devtools-reviewers,bgrins! → Bug 1814884 - add marionette test to ensure we catch newly introduced startup console spew, r?#devtools-reviewers,bgrins!
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/fc859e1a124c add marionette test to ensure we catch newly introduced startup console spew, r=webdriver-reviewers,devtools-reviewers,bgrins,nchevobbe,whimboo
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 113 Branch

Since nightly and release are affected, beta will likely be affected too.
For more information, please visit auto_nag documentation.

Severity: -- → S3
Priority: -- → P3
Duplicate of this bug: 1781285
See Also: → 1936426
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: