Merge branch '47-les-methodes-set_priority-et-unset_priority-semblent-ne-rien-faire' into 'rp2'

Resolve "Les méthodes set_priority et unset_priority semblent ne rien faire"

Closes #47

See merge request !39

apps/rp/api/serializers.py

@@ -23,7 +23,8 @@ class ArticleSerializer(serializers.ModelSerializer):
 
     class Meta:
         model = Article
-        fields = ('id', 'url', 'title', 'tags', 'extracts', 'status', 'score')
+        fields = ('id', 'url', 'title', 'tags', 'extracts',
+                  'status', 'score', 'priority')
 
     def create(self, validated_data):
         article = Article.add_new_url(**validated_data)

apps/rp/models.py

@@ -138,13 +138,15 @@ class Article(models.Model):
     @transition(field=status, source='DRAFT', target='DRAFT',
                 permission="rp.can_change_priority")
     def set_priority(self):
-        """ Set the boolean priority of an article to True. """
+        """ Set the boolean priority of an article to True.
+        The article must be a _DRAFT_"""
         self.priority = True
 
     @transition(field=status, source='DRAFT', target='DRAFT',
                 permission="rp.can_change_priority")
     def unset_priority(self):
-        """ Set the boolean priority of an article to False. """
+        """ Set the boolean priority of an article to False. 
+        The article must be a _DRAFT_."""
         self.priority = False
 
     @transition(field=status, source='DRAFT', target='DRAFT')

apps/rp/tests/test_article.py

@@ -194,3 +194,23 @@ class TestArticleApi(TestCase):
         a = ArticleFactory(status='PUBLISHED')
         r = self.client.post('/api/articles/{}/recover/'.format(a.id))
         assert r.status_code == 403
+
+    def test_api_set_priority(self):
+        self.user.user_permissions.add(Permission.objects.get(
+            codename='can_change_priority'))
+        self.client.force_login(user=self.user)
+        a = ArticleFactory(status='DRAFT')
+        assert a.priority is False
+        r = self.client.post('/api/articles/{}/set_priority/'.format(a.id))
+        assert r.status_code == 200
+        assert r.data['priority']
+
+    def test_api_unset_priority(self):
+        self.user.user_permissions.add(Permission.objects.get(
+            codename='can_change_priority'))
+        self.client.force_login(user=self.user)
+        a = ArticleFactory(status='DRAFT', priority=True)
+        assert a.priority
+        r = self.client.post('/api/articles/{}/unset_priority/'.format(a.id))
+        assert r.status_code == 200
+        assert r.data['priority'] is False