Commit cfd1fabb by Usman Khalid Committed by muzaffaryousaf

Added type field to bookmark resource.

parent 1616de50
...@@ -6,6 +6,7 @@ DEFAULT_FIELDS = [ ...@@ -6,6 +6,7 @@ DEFAULT_FIELDS = [
'id', 'id',
'course_id', 'course_id',
'usage_id', 'usage_id',
'block_type',
'created', 'created',
] ]
......
...@@ -14,6 +14,7 @@ class BookmarkSerializer(serializers.ModelSerializer): ...@@ -14,6 +14,7 @@ class BookmarkSerializer(serializers.ModelSerializer):
id = serializers.SerializerMethodField('resource_id') # pylint: disable=invalid-name id = serializers.SerializerMethodField('resource_id') # pylint: disable=invalid-name
course_id = serializers.Field(source='course_key') course_id = serializers.Field(source='course_key')
usage_id = serializers.Field(source='usage_key') usage_id = serializers.Field(source='usage_key')
block_type = serializers.Field(source='usage_key.block_type')
path = serializers.Field(source='path') path = serializers.Field(source='path')
def __init__(self, *args, **kwargs): def __init__(self, *args, **kwargs):
...@@ -38,6 +39,7 @@ class BookmarkSerializer(serializers.ModelSerializer): ...@@ -38,6 +39,7 @@ class BookmarkSerializer(serializers.ModelSerializer):
'id', 'id',
'course_id', 'course_id',
'usage_id', 'usage_id',
'block_type',
'display_name', 'display_name',
'path', 'path',
'created', 'created',
......
...@@ -75,6 +75,7 @@ class BookmarksAPITests(ModuleStoreTestCase): ...@@ -75,6 +75,7 @@ class BookmarksAPITests(ModuleStoreTestCase):
self.assertEqual(response_data['id'], '%s,%s' % (self.user.username, unicode(bookmark.usage_key))) self.assertEqual(response_data['id'], '%s,%s' % (self.user.username, unicode(bookmark.usage_key)))
self.assertEqual(response_data['course_id'], unicode(bookmark.course_key)) self.assertEqual(response_data['course_id'], unicode(bookmark.course_key))
self.assertEqual(response_data['usage_id'], unicode(bookmark.usage_key)) self.assertEqual(response_data['usage_id'], unicode(bookmark.usage_key))
self.assertEqual(response_data['block_type'], unicode(bookmark.usage_key.block_type))
self.assertIsNotNone(response_data['created']) self.assertIsNotNone(response_data['created'])
if optional_fields: if optional_fields:
......
...@@ -54,6 +54,7 @@ class BookmarksAPITests(ModuleStoreTestCase): ...@@ -54,6 +54,7 @@ class BookmarksAPITests(ModuleStoreTestCase):
self.assertEqual(response_data['id'], '%s,%s' % (self.user.username, unicode(bookmark.usage_key))) self.assertEqual(response_data['id'], '%s,%s' % (self.user.username, unicode(bookmark.usage_key)))
self.assertEqual(response_data['course_id'], unicode(bookmark.course_key)) self.assertEqual(response_data['course_id'], unicode(bookmark.course_key))
self.assertEqual(response_data['usage_id'], unicode(bookmark.usage_key)) self.assertEqual(response_data['usage_id'], unicode(bookmark.usage_key))
self.assertEqual(response_data['block_type'], unicode(bookmark.usage_key.block_type))
self.assertIsNotNone(response_data['created']) self.assertIsNotNone(response_data['created'])
self.assertEqual(response_data['display_name'], bookmark.display_name) self.assertEqual(response_data['display_name'], bookmark.display_name)
......
...@@ -108,6 +108,7 @@ class BookmarksViewTestsMixin(ModuleStoreTestCase): ...@@ -108,6 +108,7 @@ class BookmarksViewTestsMixin(ModuleStoreTestCase):
self.assertEqual(response_data['id'], '%s,%s' % (self.user.username, unicode(bookmark.usage_key))) self.assertEqual(response_data['id'], '%s,%s' % (self.user.username, unicode(bookmark.usage_key)))
self.assertEqual(response_data['course_id'], unicode(bookmark.course_key)) self.assertEqual(response_data['course_id'], unicode(bookmark.course_key))
self.assertEqual(response_data['usage_id'], unicode(bookmark.usage_key)) self.assertEqual(response_data['usage_id'], unicode(bookmark.usage_key))
self.assertEqual(response_data['block_type'], unicode(bookmark.usage_key.block_type))
self.assertIsNotNone(response_data['created']) self.assertIsNotNone(response_data['created'])
if optional_fields: if optional_fields:
......
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