Gracefully handle optional properties such as Addon.updateDate

VERIFIED FIXED in mozilla1.9.3a5

Status

()

P1
normal
VERIFIED FIXED
9 years ago
8 years ago

People

(Reporter: Unfocused, Assigned: Unfocused)

Tracking

Trunk
mozilla1.9.3a5
Points:
---
Bug Flags:
in-testsuite +
in-litmus -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [rewrite])

Some properties are optional for a provider to implement/provide.

Optional properties are documented here:
https://wiki.mozilla.org/Extension_Manager:API_Rewrite:API#Optional_Properties
Flags: in-testsuite?
Flags: in-litmus-
Note: The plugins category is currently broken because it assumes updateDate cannot be null.
Priority: -- → P1
http://hg.mozilla.org/projects/addonsmgr/rev/1319aa17d576
Whiteboard: [rewrite] → [rewrite][fixed-in-addonsmgr]
Assignee: nobody → bmcbride
Landed on trunk as a part of bug 554007
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Whiteboard: [rewrite][fixed-in-addonsmgr] → [rewrite]
Target Milestone: --- → mozilla1.9.3a5
Verified the landing by checking the hg log:
http://hg.mozilla.org/mozilla-central/rev/3c6862a3b12f
Status: RESOLVED → VERIFIED
Version: unspecified → Trunk
We have tests that open the plugins list already so the specific case that was broken here is tested, and we'll have a few general tests for missing properties as the mock provider doesn't include all of them. Not sure it is worth doing more than that at this point.
Flags: in-testsuite? → in-testsuite-
(In reply to comment #5)
> We have tests that open the plugins list already so the specific case that was
> broken here is tested, and we'll have a few general tests for missing
> properties as the mock provider doesn't include all of them. Not sure it is
> worth doing more than that at this point.

I assume you meant to set in-testsuite+.
Flags: in-testsuite- → in-testsuite+
You need to log in before you can comment on or make changes to this bug.