Closed
Bug 902950
Opened 12 years ago
Closed 7 years ago
Fix test TODO left over from bug 853388
Categories
(Toolkit :: Add-ons Manager, defect)
Toolkit
Add-ons Manager
Tracking
()
RESOLVED
INACTIVE
People
(Reporter: Irving, Unassigned)
References
Details
Bug 853388 left behind one unresolved test case, involving the intersection between updating addons (without changing the addon version number), rebuilding the addon database and handling compatibility overrides from AMO.
See https://bugzilla.mozilla.org/show_bug.cgi?id=853388#c31 and following comments (mostly #c32 and #c33).
My most recent thought on this issue is: If there is a compatibility update from AMO, shouldn't the cached .json file we create for the staged add-on contain that information? I'm not sure it does in this test case.
Updated•12 years ago
|
Flags: needinfo?(dtownsend+bugmail)
Comment 1•12 years ago
|
||
(In reply to :Irving Reid from comment #0)
> Bug 853388 left behind one unresolved test case, involving the intersection
> between updating addons (without changing the addon version number),
> rebuilding the addon database and handling compatibility overrides from AMO.
>
> See https://bugzilla.mozilla.org/show_bug.cgi?id=853388#c31 and following
> comments (mostly #c32 and #c33).
>
> My most recent thought on this issue is: If there is a compatibility update
> from AMO, shouldn't the cached .json file we create for the staged add-on
> contain that information? I'm not sure it does in this test case.
Yes it should, I'm pretty sure it does too. The question is whether that compatibility update makes it into the DB if the DB is getting rebuilt during the same startup that the add-on is getting installed.
Flags: needinfo?(dtownsend+bugmail)
Comment 2•7 years ago
|
||
Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INACTIVE
You need to log in
before you can comment on or make changes to this bug.
Description
•