Commit ea572112 by Muhammad Ammar

Merge pull request #35 from edx/ammar/pagination-fixes-for-annotatorjs

annotatorjs fixes
parents 55467209 39a4c065
...@@ -22,7 +22,7 @@ class NotesPaginator(pagination.PageNumberPagination): ...@@ -22,7 +22,7 @@ class NotesPaginator(pagination.PageNumberPagination):
'current_page': self.page.number, 'current_page': self.page.number,
'next': self.get_next_link(), 'next': self.get_next_link(),
'previous': self.get_previous_link(), 'previous': self.get_previous_link(),
'count': self.page.paginator.count, 'total': self.page.paginator.count,
'num_pages': self.page.paginator.num_pages, 'num_pages': self.page.paginator.num_pages,
'results': data 'rows': data
}) })
...@@ -31,18 +31,18 @@ class UpdateIndexTest(BaseAnnotationViewTests): ...@@ -31,18 +31,18 @@ class UpdateIndexTest(BaseAnnotationViewTests):
end = datetime.datetime.now() end = datetime.datetime.now()
results = self._get_search_results(text='note') results = self._get_search_results(text='note')
self.assertDictContainsSubset({'results': [], 'count': 0}, results) self.assertDictContainsSubset({'rows': [], 'total': 0}, results)
# When second note was created. # When second note was created.
call_command('update_index', start_date=second_start.isoformat(), end_date=second_end.isoformat(), verbosity=0) call_command('update_index', start_date=second_start.isoformat(), end_date=second_end.isoformat(), verbosity=0)
results = self._get_search_results(text='note') results = self._get_search_results(text='note')
self.assertEqual(results['count'], 1) self.assertEqual(results['total'], 1)
self.assertEqual(results['results'][0]['text'], 'Second note') self.assertEqual(results['rows'][0]['text'], 'Second note')
# All notes. # All notes.
call_command('update_index', start_date=start.isoformat(), end_date=end.isoformat(), verbosity=0) call_command('update_index', start_date=start.isoformat(), end_date=end.isoformat(), verbosity=0)
results = self._get_search_results(text='note') results = self._get_search_results(text='note')
self.assertEqual(results['count'], 3) self.assertEqual(results['total'], 3)
@factory.django.mute_signals(signals.post_delete) @factory.django.mute_signals(signals.post_delete)
def test_delete(self): def test_delete(self):
...@@ -55,7 +55,7 @@ class UpdateIndexTest(BaseAnnotationViewTests): ...@@ -55,7 +55,7 @@ class UpdateIndexTest(BaseAnnotationViewTests):
self._create_annotation(text=u'Third note') self._create_annotation(text=u'Third note')
results = self._get_search_results(text='note') results = self._get_search_results(text='note')
self.assertEqual(results['count'], 3) self.assertEqual(results['total'], 3)
# Delete first note. # Delete first note.
url = reverse('api:v1:annotations_detail', kwargs={'annotation_id': first_note['id']}) url = reverse('api:v1:annotations_detail', kwargs={'annotation_id': first_note['id']})
...@@ -75,5 +75,5 @@ class UpdateIndexTest(BaseAnnotationViewTests): ...@@ -75,5 +75,5 @@ class UpdateIndexTest(BaseAnnotationViewTests):
results = self._get_search_results(text='note') results = self._get_search_results(text='note')
# When remove flag is provided, start and end flags do not play any role. # When remove flag is provided, start and end flags do not play any role.
self.assertEqual(results['count'], 1) self.assertEqual(results['total'], 1)
self.assertEqual(results['results'][0]['text'], 'Third note') self.assertEqual(results['rows'][0]['text'], 'Third note')
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment