"An engine with that name already exists!" message whitelisting doesn't work anymore - xpcshell.ini:toolkit/components/search/tests/xpcshell/test_webextensions_upgrade.js | test_upgrade_to_existing_name_not_allowed - [test_upgrade_to_existing_name_not_all
Categories
(Firefox :: Search, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr102 | --- | unaffected |
firefox113 | --- | unaffected |
firefox114 | --- | unaffected |
firefox115 | --- | fixed |
People
(Reporter: mkmelin, Assigned: mkmelin)
References
(Regression)
Details
(Keywords: intermittent-failure, regression)
Attachments
(1 file)
Thunderbird tests are failing with
TEST-UNEXPECTED-FAIL | xpcshell.ini:toolkit/components/search/tests/xpcshell/test_webextensions_upgrade.js | test_upgrade_to_existing_name_not_allowed - [test_upgrade_to_existing_name_not_allowed : 1] Unexpected console message received during test: An engine called engine1 already exists! - false == true
I don't know why Firefox doesn't hit this case, but test_webextensions_upgrade.js was (and is) specifically allowing that message, so it needs to be updated after the message was updated in https://hg.mozilla.org/mozilla-central/rev/ce23c5fc9dc44b2fa666356b9534907675e1b36e#l1.35
Assignee | ||
Comment 1•1 year ago
|
||
Same for test_webextensions_install.js
E.g. https://treeherder.mozilla.org/jobs?repo=comm-central&selectedTaskRun=bB_dJleyQH--V-U7bLm0dQ.0
Assignee | ||
Comment 2•1 year ago
|
||
Comment 3•1 year ago
|
||
Set release status flags based on info from the regressing bug 1833848
Comment hidden (Intermittent Failures Robot) |
Comment 5•1 year ago
|
||
It took a little while for me to find why, but got there. Firefox's addSearchEngine uses Cu.reportError
, Thunderbird's clone uses console.error
.
At the moment, the console tracking is obviously not picking up Cu.reportError
. I'm not sure that's worth fixing at the moment, as we're working towards removing `Cu.reportError (bug 877389).
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/integration/autoland/rev/7182040ba453 update allowed console message for test_webextensions_upgrade.js after the message was updated in bug 1833848. r=Standard8
Comment hidden (Intermittent Failures Robot) |
Comment 8•11 months ago
|
||
bugherder |
Comment hidden (Intermittent Failures Robot) |
Description
•