Closed Bug 1476929 Opened 6 years ago Closed 6 years ago

Perma xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_storage_idb_data_migration.js | xpcshell return code: 0 when Gecko 63 merges to Beta on 2018-08-27

Categories

(WebExtensions :: Storage, defect)

defect
Not set
normal

Tracking

(firefox-esr52 unaffected, firefox-esr60 unaffected, firefox61 unaffected, firefox62 unaffected, firefox63+ verified)

VERIFIED FIXED
mozilla63
Tracking Status
firefox-esr52 --- unaffected
firefox-esr60 --- unaffected
firefox61 --- unaffected
firefox62 --- unaffected
firefox63 + verified

People

(Reporter: RaulG, Assigned: rpl)

References

Details

(Keywords: regression)

Attachments

(1 file)

[Tracking Requested - why for this release]: Central as Beta sim: https://treeherder.mozilla.org/#/jobs?repo=try&revision=20a04997cec50341d5f8fc16b9d1a9e981c0c94f&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-classifiedState=unclassified&selectedJob=188930687 Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=188930757&repo=try&lineNumber=2113 Log snippet: [task 2018-07-19T11:34:09.599Z] 11:34:09 INFO - TEST-START | xpcshell-e10s.ini:toolkit/components/extensions/test/xpcshell/test_ext_contentscript.js [task 2018-07-19T11:34:10.770Z] 11:34:10 INFO - TEST-PASS | xpcshell-e10s.ini:toolkit/components/extensions/test/xpcshell/test_ext_contentscript.js | took 1175ms [task 2018-07-19T11:34:10.791Z] 11:34:10 INFO - TEST-START | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_storage_idb_data_migration.js [task 2018-07-19T11:34:11.060Z] 11:34:11 WARNING - TEST-UNEXPECTED-FAIL | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_storage_idb_data_migration.js | xpcshell return code: 0 This is a regression from Bug 1475306.
Flags: needinfo?(lgreco)
I looked at the failure logs, and it is actually due to this error: ``` addons.manager ERROR startup failed: TypeError: Services.appinfo.version is undefined (resource://gre/modules/AddonManager.jsm:734:1) ``` (https://treeherder.mozilla.org/logviewer.html#?job_id=188930757&repo=try&lineNumber=2134) And so the issue is that in this test file we need to call AddonTestUtils.createAppInfo before importing ExtensionStorage.jsm, because it internally imports Services.jsm and it would make the following check to fail because Services.appinfo has been cached with an undefined version property when the test file runs on beta: ``` if (!MOZ_COMPATIBILITY_NIGHTLY) { PREF_EM_CHECK_COMPATIBILITY = PREF_EM_CHECK_COMPATIBILITY_BASE + "." + Services.appinfo.version.replace(BRANCH_REGEXP, "$1"); } ``` (https://searchfox.org/mozilla-central/rev/88199de427d3c5762b7f3c2a4860c10734abd867/toolkit/mozapps/extensions/AddonManager.jsm#732-735)
Flags: needinfo?(lgreco)
Attachment #8993402 - Flags: review?(mixedpuppy)
Assignee: nobody → lgreco
Status: NEW → ASSIGNED
Comment on attachment 8993402 [details] Bug 1476929 - Fix test_ext_storage_idb_data_migration.js failure when running on beta. https://reviewboard.mozilla.org/r/258160/#review266322 add a commit message about why this change fixes the test.
Attachment #8993402 - Flags: review?(mixedpuppy) → review+
Pushed by luca.greco@alcacoop.it: https://hg.mozilla.org/integration/autoland/rev/bf63176221f0 Fix test_ext_storage_idb_data_migration.js failure when running on beta. r=mixedpuppy
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: