diff --git a/common/lib/xmodule/xmodule/video_module/video_module.py b/common/lib/xmodule/xmodule/video_module/video_module.py
index 311c48c567..7603892232 100644
--- a/common/lib/xmodule/xmodule/video_module/video_module.py
+++ b/common/lib/xmodule/xmodule/video_module/video_module.py
@@ -935,6 +935,7 @@ class VideoDescriptor(VideoFields, VideoTranscriptsMixin, VideoStudioViewHandler
xml (lxml object): xml representation of video to be imported
course_id (str): course id
"""
+ edx_video_id = ''
if self.edx_video_id is not None:
edx_video_id = self.edx_video_id.strip()
diff --git a/lms/djangoapps/courseware/tests/test_video_mongo.py b/lms/djangoapps/courseware/tests/test_video_mongo.py
index 10268375af..1c7075cd58 100644
--- a/lms/djangoapps/courseware/tests/test_video_mongo.py
+++ b/lms/djangoapps/courseware/tests/test_video_mongo.py
@@ -1584,35 +1584,6 @@ class VideoDescriptorTest(TestCase, VideoDescriptorTestBase):
expected = etree.XML(expected_str, parser=parser)
self.assertXmlEqual(expected, actual)
- def test_export_val_data_with_external(self):
- """
- Tests exported val data for external video.
- """
- external_video_id = '3_yD_cEKoCk'
- create_or_update_video_transcript(
- video_id=external_video_id,
- language_code='ar',
- metadata={
- 'provider': 'Cielo24',
- 'file_name': 'ext101.srt',
- 'file_format': 'srt'
- }
- )
-
- actual = self.descriptor.definition_to_xml(resource_fs=None)
- expected_str = """
-
- """.format(video_id=external_video_id)
- parser = etree.XMLParser(remove_blank_text=True)
- expected = etree.XML(expected_str, parser=parser)
- self.assertXmlEqual(expected, actual)
-
def test_export_val_data_not_found(self):
self.descriptor.edx_video_id = 'nonexistent'
actual = self.descriptor.definition_to_xml(resource_fs=None)
@@ -1668,33 +1639,6 @@ class VideoDescriptorTest(TestCase, VideoDescriptorTestBase):
self.get_video_transcript_data('test_edx_video_id')
)
- def test_import_val_data_external(self):
- """
- Tests video import with external video.
- """
- external_video_id = 'external_video_id'
- module_system = DummySystem(load_error_modules=True)
-
- xml_data = """
-
- """.format(video_id=external_video_id)
-
- id_generator = Mock()
- id_generator.target_course_id = "test_course_id"
- self.descriptor.from_xml(xml_data, module_system, id_generator)
-
- # verify transcript data
- self.assertDictEqual(
- get_video_transcript(external_video_id, 'ar'),
- self.get_video_transcript_data(external_video_id)
- )
-
def test_import_val_data_invalid(self):
create_profile('mobile')
module_system = DummySystem(load_error_modules=True)
diff --git a/requirements/edx/base.txt b/requirements/edx/base.txt
index 0170f3de47..75f3b582bd 100644
--- a/requirements/edx/base.txt
+++ b/requirements/edx/base.txt
@@ -60,7 +60,7 @@ edx-organizations==0.4.9
edx-rest-api-client==1.7.1
edx-search==1.1.0
edx-submissions==2.0.12
-edxval==0.1.7
+edxval==0.1.8
event-tracking==0.2.4
feedparser==5.1.3
firebase-token-generator==1.3.2