Merge pull request #29 from edx/dcs/fix-youtube-lookup

Fixed youtube id lookup (no longer storing youtube urls in db)

edxval/models.py

@@ -93,10 +93,9 @@ class Video(models.Model):
         """
         Look up video by youtube id
         """
-        url = '://youtu.be/%s' % youtube_id
         qset = cls.objects.filter(
             encoded_videos__profile__profile_name='youtube',
-            encoded_videos__url__endswith=url
+            encoded_videos__url=youtube_id
         ).prefetch_related('encoded_videos', 'courses', 'subtitles')
         return qset
 

edxval/tests/test_views.py

@@ -516,7 +516,7 @@ class VideoListTest(APIAuthTestCase):
             'encoded_videos': [
                 {
                     'profile': 'youtube',
-                    'url': 'https://youtu.be/AbcDef',
+                    'url': 'AbcDef',
                     'file_size': 4545,
                     'bitrate': 6767,
                 }