make latest bounce to watermarked

This commit is contained in:
Andy McKay 2011-09-23 14:13:12 -07:00
Родитель 08ae18c6b2
Коммит 72ff89588c
2 изменённых файлов: 14 добавлений и 4 удалений

Просмотреть файл

@ -214,12 +214,16 @@ class File(amo.models.OnChangeMixin, amo.models.ModelBase):
m.group('suffix'))
def latest_xpi_url(self):
addon = self.version.addon_id
kw = {'addon_id': addon}
addon = self.version.addon
kw = {'addon_id': addon.pk}
if self.platform_id != amo.PLATFORM_ALL.id:
kw['platform'] = self.platform_id
url = reverse('downloads.latest', kwargs=kw)
return os.path.join(url, 'addon-%s-latest%s' % (addon, self.extension))
if addon.is_premium():
url = reverse('downloads.watermarked', args=[self.id])
else:
url = reverse('downloads.latest', kwargs=kw)
return os.path.join(url, 'addon-%s-latest%s' %
(addon.pk, self.extension))
def eula_url(self):
return reverse('addons.eula', args=[self.version.addon_id, self.id])

Просмотреть файл

@ -936,6 +936,12 @@ class TestWatermark(amo.tests.TestCase, amo.tests.AMOPaths):
self.file.version.addon.update(premium_type=amo.ADDON_PREMIUM)
assert url in self.file.get_url_path('test')
def test_get_latest_path(self):
url = reverse('downloads.watermarked', args=[self.file.id])
assert url not in self.file.latest_xpi_url()
self.file.version.addon.update(premium_type=amo.ADDON_PREMIUM)
assert url in self.file.latest_xpi_url()
class TestWatermarkCleanup(amo.tests.TestCase, amo.tests.AMOPaths):
fixtures = ['base/addon_3615', 'base/users']