Commit eba951fe by Jonathan Piacenti

Fix self-paced badge naming.

parent c820ae14
...@@ -117,7 +117,7 @@ class BadgeHandler(object): ...@@ -117,7 +117,7 @@ class BadgeHandler(object):
) )
else: else:
return _(u'Completed the course "{course_name}" ({course_mode})').format( return _(u'Completed the course "{course_name}" ({course_mode})').format(
start_date=course.display_name, course_name=course.display_name,
course_mode=mode, course_mode=mode,
) )
......
...@@ -115,6 +115,13 @@ class BadgeHandlerTestCase(ModuleStoreTestCase, EventTrackingTestCase): ...@@ -115,6 +115,13 @@ class BadgeHandlerTestCase(ModuleStoreTestCase, EventTrackingTestCase):
} }
) )
def test_self_paced_description(self):
"""
Verify that a badge created for a course with no end date gets a different description.
"""
self.course.end = None
self.assertEqual(BadgeHandler.badge_description(self.course, 'honor'), 'Completed the course "Badged" (honor)')
def test_ensure_badge_created_cache(self): def test_ensure_badge_created_cache(self):
""" """
Make sure ensure_badge_created doesn't call create_badge if we know the badge is already there. Make sure ensure_badge_created doesn't call create_badge if we know the badge is already there.
......
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