Make sure we can handle empty youtube attributes.

common/lib/xmodule/xmodule/tests/test_video_xml.py

@@ -110,3 +110,14 @@ class VideoModuleLogicTest(LogicTest):
         youtube_str = '1.00:p2Q6BrNhdh8'
         youtube_str_hack = '1.0:p2Q6BrNhdh8'
         self.assertEqual(_parse_youtube(youtube_str), _parse_youtube(youtube_str_hack))
+
+    def test_parse_youtube_empty(self):
+        """
+        Some courses have empty youtube attributes, so we should handle
+        that well.
+        """
+        self.assertEqual(_parse_youtube(''),
+                         {'0.75': '',
+                          '1.00': '',
+                          '1.25': '',
+                          '1.50': ''})

common/lib/xmodule/xmodule/video_module.py

@@ -168,6 +168,8 @@ def _parse_youtube(data):
     XML-based courses.
     """
     ret = {'0.75': '', '1.00': '', '1.25': '', '1.50': ''}
+    if data == '':
+        return ret
     videos = data.split(',')
     for video in videos:
         pieces = video.split(':')