Add add-on slug to discovery API (fix #2751)

docs/topics/api/discovery.rst

@@ -21,4 +21,4 @@ Firefox (about:addons).
     :>json array results: The array containing the results for this query.
     :>json string results[].heading: The heading for this item. May contain the following sub-strings, that clients need to use to format the string as they desire: ``{start_sub_heading}``, ``{end_sub_heading}`` and ``{addon_name}``.
     :>json string|null results[].description: The description for this item, if any. May contain some HTML tags.
-    :>json object results[].addon: The :ref:`add-on <addon-detail-object>` for this item. Only a subset of fields are present: ``id``, ``current_version`` (with only the ``compatibility`` and ``files`` fields present), ``guid``, ``icon_url``, ``name``, ``theme_data``, ``type`` and ``url``.
+    :>json object results[].addon: The :ref:`add-on <addon-detail-object>` for this item. Only a subset of fields are present: ``id``, ``current_version`` (with only the ``compatibility`` and ``files`` fields present), ``guid``, ``icon_url``, ``name``, ``slug``, ``theme_data``, ``type`` and ``url``.

src/olympia/discovery/serializers.py

@@ -16,7 +16,7 @@ class DiscoveryAddonSerializer(AddonSerializer):
 
     class Meta:
         fields = ('id', 'current_version', 'guid', 'icon_url', 'name',
-                  'theme_data', 'type', 'url',)
+                  'slug', 'theme_data', 'type', 'url',)
         model = Addon
 
 

src/olympia/discovery/tests/test_views.py

@@ -41,6 +41,7 @@ class TestDiscoveryViewList(TestCase):
                 assert result['heading'] == unicode(addons[item.addon_id].name)
             assert result['description'] == item.description
             assert result['addon']['current_version']
+            assert result['addon']['slug'] == addons[item.addon_id].slug
 
     def test_missing_addon(self):
         addon_factory(id=discopane_items[0].addon_id, type=amo.ADDON_PERSONA)