Commit d5448f7c by Gregory Martin Committed by muzaffaryousaf

Green HEAL tests

parent d435c213
...@@ -59,6 +59,10 @@ class HealTests(TestCase): ...@@ -59,6 +59,10 @@ class HealTests(TestCase):
product_spec='mobile_low', product_spec='mobile_low',
encode_destination=Destination.objects.create(destination_name='destination_name') encode_destination=Destination.objects.create(destination_name='destination_name')
) )
self.hls_encode = Encode.objects.create(
product_spec='hls',
encode_destination=Destination.objects.create(destination_name='destination_name')
)
url = URL( url = URL(
videoID=self.video, videoID=self.video,
...@@ -140,7 +144,6 @@ class HealTests(TestCase): ...@@ -140,7 +144,6 @@ class HealTests(TestCase):
}, },
) )
@unpack @unpack
@skip("Failing from day 1 https://github.com/edx/edx-video-pipeline/pull/26")
def test_determine_fault(self, edx_id, video_trans_status, video_trans_start, video_active): def test_determine_fault(self, edx_id, video_trans_status, video_trans_start, video_active):
""" """
Tests that determine_fault works in various video states. Tests that determine_fault works in various video states.
...@@ -150,8 +153,10 @@ class HealTests(TestCase): ...@@ -150,8 +153,10 @@ class HealTests(TestCase):
video_trans_status=video_trans_status, video_trans_status=video_trans_status,
video_trans_start=video_trans_start, video_trans_start=video_trans_start,
video_active=video_active, video_active=video_active,
inst_class=Course() inst_class=self.course_object
) )
video_instance.save()
encode_list = self.heal_instance.determine_fault(video_instance) encode_list = self.heal_instance.determine_fault(video_instance)
if video_instance.edx_id == '1': if video_instance.edx_id == '1':
...@@ -193,9 +198,11 @@ class HealTests(TestCase): ...@@ -193,9 +198,11 @@ class HealTests(TestCase):
video_trans_status=video_object['video_trans_status'], video_trans_status=video_object['video_trans_status'],
video_trans_start=video_object['video_trans_start'], video_trans_start=video_object['video_trans_start'],
video_active=video_object['video_active'], video_active=video_object['video_active'],
inst_class=Course() inst_class=self.course_object
) )
video_instance.save()
encode_list = self.heal_instance.differentiate_encodes( encode_list = self.heal_instance.differentiate_encodes(
uncompleted_encodes, uncompleted_encodes,
expected_encodes, expected_encodes,
...@@ -240,16 +247,17 @@ class HealTests(TestCase): ...@@ -240,16 +247,17 @@ class HealTests(TestCase):
} }
) )
@unpack @unpack
@skip("Failing from day 1 https://github.com/edx/edx-video-pipeline/pull/26")
def test_determine_longterm_corrupt(self, uncompleted_encodes, expected_encodes, video_object): def test_determine_longterm_corrupt(self, uncompleted_encodes, expected_encodes, video_object):
video_instance = Video( video_instance = Video(
edx_id=video_object['edx_id'], edx_id=video_object['edx_id'],
video_trans_status=video_object['video_trans_status'], video_trans_status=video_object['video_trans_status'],
video_trans_start=video_object['video_trans_start'], video_trans_start=video_object['video_trans_start'],
video_active=video_object['video_active'], video_active=video_object['video_active'],
inst_class=Course() inst_class=self.course_object
) )
video_instance.save()
longterm_corrupt = self.heal_instance.determine_longterm_corrupt( longterm_corrupt = self.heal_instance.determine_longterm_corrupt(
uncompleted_encodes, uncompleted_encodes,
expected_encodes, expected_encodes,
......
...@@ -121,7 +121,7 @@ class VedaHeal(object): ...@@ -121,7 +121,7 @@ class VedaHeal(object):
try: try:
if uncompleted_encodes: if uncompleted_encodes:
uncompleted_encodes.remove('review') uncompleted_encodes.remove('review')
except ValueError: except KeyError:
pass pass
# list comparison # list comparison
......
## NOTE: This is not a working req file -- merely a collection for notes. ##
django==1.9 django==1.9
djangorestframework==3.6.4 djangorestframework==3.6.4
django-cors-headers django-cors-headers
......
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