[WIP] import data from an API using an admin view

representatives_votes/contrib/parltrack/import_votes.py

@@ -97,15 +97,18 @@ def parse_proposal_data(self, proposal_data, dossier_pk):
                 changed = True
 
         if changed:
+            logger.debug('Saving proposal %s' %
+                proposal_data.get('epref', proposal_data['title']))
             proposal.save()
 
         responses = vote_pre_import.send(sender=self, vote_data=proposal_data)
 
         for receiver, response in responses:
             if response is False:
                 logger.debug(
-                    'Skipping dossier %s', proposal_data.get(
-                        'epref', proposal_data['title']))
+                    'Skipping votes for dossier %s because of %s',
+                    proposal_data.get( 'epref', proposal_data['title']),
+                    receiver)
                 return
 
         positions = ['For', 'Abstain', 'Against']

representatives_votes/serializers.py

@@ -43,7 +43,6 @@ class ProposalDetailSerializer(ProposalSerializer):
 
     class Meta:
         model = models.Proposal
-        fields = ProposalSerializer.Meta.fields + ('votes',)
 
 
 class DossierSerializer(serializers.HyperlinkedModelSerializer):
@@ -64,8 +63,7 @@ class DossierDetailSerializer(DossierSerializer):
     """
     Dossier serializer that includes proposals and votes.
     """
-
-    proposals = ProposalSerializer(many=True)
+    proposals = ProposalDetailSerializer(many=True)
 
     class Meta:
         model = models.Dossier

representatives_votes/tests.py

@@ -0,0 +1,11 @@
+from django import test
+
+
+class RetrieverSpec(test.TestCase):
+    # Should:
+    # - sync any dossier where title='', dossiers are inserted by a custom
+    # admin feature
+    # - sync any proposal without vote, proposals should be added on
+    # dossier-sync, so any dossier in our db should have votes for every
+    # proposal at some point.
+

representatives_votes/tests/test_views.py

@@ -31,5 +31,8 @@ def test_proposals(self):
     def test_vote(self):
         self.functional_test(1, '/api/votes/1/')
 
+    def test_proposals(self):
+        self.functional_test(1, '/api/proposals/')
+
     def test_votes(self):
         self.functional_test(1, '/api/votes/')