diff --git a/apps/rp/tests/test_article.py b/apps/rp/tests/test_article.py index ee395fd708948ffc0ba3ed23734a43cd0e578d70..501599bc3323c95d87a75c584aed3505da16b25b 100644 --- a/apps/rp/tests/test_article.py +++ b/apps/rp/tests/test_article.py @@ -122,7 +122,9 @@ class TestArticleViews(TestCase): def test_search_view(self): article = ArticleFactory(title=u'Zog Zog chez les schtroumphs', - lang='FR') + lang='FR', + archive=False, + status='PUBLISHED') article.save() r = self.client.get('/rp/', {'q': 'Zog Zog'}) @@ -131,7 +133,9 @@ class TestArticleViews(TestCase): assert len(r.context['article_list']) == 0 def test_search_view_archived(self): - archive = ArticleFactory(archive=True, lang='FR') + archive = ArticleFactory(archive=True, + lang='FR', + status='PUBLISHED') r = self.client.get('/rp/', {'q': '', 'archive': 'true'}) assert len(r.context['article_list']) == 1 @@ -141,7 +145,10 @@ class TestArticleViews(TestCase): assert len(r.context['article_list']) == 0 def test_search_view_speaks(self): - speak = ArticleFactory(speak=True, lang='FR') + speak = ArticleFactory(speak=True, + archive=False, + lang='FR', + status='PUBLISHED') r = self.client.get('/rp/', {'q': '', 'speak': 'true'}) assert len(r.context['article_list']) == 1 @@ -151,7 +158,10 @@ class TestArticleViews(TestCase): assert len(r.context['article_list']) == 0 def test_search_view_quoted(self): - quote = ArticleFactory(quote=True, lang='FR') + quote = ArticleFactory(quote=True, + archive=False, + lang='FR', + status='PUBLISHED') r = self.client.get('/rp/', {'q': '', 'quote': 'true'}) assert len(r.context['article_list']) == 1 diff --git a/apps/rp/views/articles.py b/apps/rp/views/articles.py index d3807a0081fdb9379de30f23f8826cfb49ab7d91..1352fc260a34d7c28017cb194b3acdfe11d2eecc 100644 --- a/apps/rp/views/articles.py +++ b/apps/rp/views/articles.py @@ -138,9 +138,10 @@ class ArticleDetailView(LoginRequiredMixin, DetailView): return context -class ArticleEdit(PermissionRequiredMixin, UpdateView): +class ArticleEdit(LoginRequiredMixin, PermissionRequiredMixin, UpdateView): model = Article permission_required = 'rp.can_edit' + raise_exception = True fields = ['screenshot', 'url', 'lang', 'title', 'tags', 'extracts', 'quote', 'speak', 'archive']