Dave Townsend [:mossop] Assignee
	Description • 14 years ago

      No description provided.

	Dave Townsend [:mossop] Assignee
	Comment 1 • 14 years ago

We probably already have one but a litmus test verifying that installing add-ons that need to ping AMO for a compatibility update works both from the website and from a downloaded copy of the file.

This is fixed in http://hg.mozilla.org/projects/addonsmgr/rev/a51fa458df43

	Dave Townsend [:mossop] Assignee
	Comment 2 • 14 years ago

This does the compatibility update check before passing the new AddonInstall instance out to the caller.

	Robert Strong (they/them - no direct email)
	Comment 3 • 14 years ago

Comment on attachment 435785 [details] [diff] [review]
patch rev 1

Looks good

	Dave Townsend [:mossop] Assignee
	Comment 4 • 14 years ago

http://hg.mozilla.org/mozilla-central/rev/3a1781f2c403

	Henrik Skupin [:whimboo][⌚️UTC+2]
	Comment 5 • 14 years ago

Dave, for the manual test we would need something like:

1. Finding an add-on which is not compatible with the current version (e.g. take an outdated version)
2. Trying to install that add-on from the local disk
3. Trying to install that add-on from a website (should this be different from AMO?)

As result we have to check for existing updates for that add-on and install those instead?

Do we need step 2 (local disk) or is that covered by the automation script?

	Dave Townsend [:mossop] Assignee
	Comment 6 • 14 years ago

We do have automated tests that should cover this so maybe litmus tests for it are a low priority, but it is a case where I think the added coverage of a manual test would be useful. All your three points are right though for point 3 it can be from AMO.

	raymond [:retornam] (inactive)
	Comment 7 • 14 years ago

verified fixed based on hg push log and passing automated tests

	Henrik Skupin [:whimboo][⌚️UTC+2]
	Comment 8 • 14 years ago

(In reply to comment #6)
> We do have automated tests that should cover this so maybe litmus tests for it
> are a low priority, but it is a case where I think the added coverage of a
> manual test would be useful. All your three points are right though for point 3
> it can be from AMO.

I wouldn't target a Litmus test for this particular bug. Instead we should have a Mozmill test which automatically can make the example add-on incompatible by updating the install.rdf's maxversion entry.

	Henrik Skupin [:whimboo][⌚️UTC+2]
	Comment 9 • 13 years ago

This manual test is kinda hard to test because it will not work with older versions of the extensions, which have been marked as compatible on AMO but not in their install.rdf file. So I'm not sure how useful this test is at the current state.

Added Litmus test:
https://litmus.mozilla.org/show_test.cgi?id=15222