Closed Bug 1659366 Opened 5 months ago Closed 5 months ago

Intermittent toolkit/components/search/tests/xpcshell/test_config_engine_params.js | xpcshell return code: 0


(Thunderbird :: Upstream Synchronization, defect, P2)



(thunderbird_esr78 unaffected)

81 Branch
Tracking Status
thunderbird_esr78 --- unaffected


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


(Keywords: intermittent-failure, Whiteboard: [thunderbird-testfailure: X all])


(1 file)

Filed by: mozilla [at]
Parsed log:
Full log:

Heaps of search tests fail. Looks like it stated at this push but no one reported it :-(
TEST-UNEXPECTED-FAIL | xpcshell.ini:toolkit/components/search/tests/xpcshell/test_config_engine_params.js | xpcshell return code: 0
TEST-UNEXPECTED-FAIL | xpcshell.ini:toolkit/components/search/tests/xpcshell/test_config_engine_params.js | test_get_extension - [test_get_extension : 12] Should have found an engine - null != null
TEST-UNEXPECTED-FAIL | xpcshell.ini:toolkit/components/search/tests/xpcshell/test_identifiers.js | xpcshell return code: 0
TEST-UNEXPECTED-FAIL | xpcshell.ini:toolkit/components/search/tests/xpcshell/test_identifiers.js | test_from_telemetry_id - [test_from_telemetry_id : 32] Should have the correct identifier - "basic" == "telemetry"
TEST-UNEXPECTED-FAIL | xpcshell.ini:toolkit/components/search/tests/xpcshell/test_initialization.js | xpcshell return code: 0
TEST-UNEXPECTED-FAIL | xpcshell.ini:toolkit/components/search/tests/xpcshell/test_initialization.js | test_initialization_failure - [test_initialization_failure : 1] Missing expected exception Should have failed to initialize
TEST-UNEXPECTED-FAIL | xpcshell.ini:toolkit/components/search/tests/xpcshell/test_json_cache.js | xpcshell return code: 0

Forgot to paste the push:
Bad M-C: 605c404fbd80c67e1127ac054b8bec6742bd748f
Good M-C: f46205a42faecda0e5a73bdcde0a8e1caa0126d9
Maybe bug 1657918 since bug 1658646 was backed out. But then that's about the some other search.

Whiteboard: [thunderbird-testfailure: X all]

Mark, you're quite active in TB these days (including doing some sheriffing). Can you tell where this came from?

Flags: needinfo?(standard8)

Hmm, looks like M-C removed and we still have it, so we are months behind, and now it broke.

I'm getting out of practise :-(

Mark Banner — Bug 1658920 - Stop running legacy configuration for search tests. r=daleharvey

Only sheriffing because I landed things ;-)

In any case, when landing the removals I was sure I put in setting the modern config preference to true so that they would still pass for Thunderbird. Unfortunately that seems to have gone missing along the way so I guess I did something wrong somewhere.

I'll get a patch up.

Flags: needinfo?(standard8)

This was intended when we removed the legacy tests in bug 1658920, but was somehow missed/lost.

Assignee: nobody → standard8
Priority: P5 → P2
Pushed by
Force on in test for the time being so that Thunderbird tests past. r=daleharvey
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → 81 Branch
You need to log in before you can comment on or make changes to this bug.