Commit cff8e16d by Greg Price

fixup! Make notifier API resilient to bad enrollments

Add explicit course key to enrollment for non-existent course in test case
parent 7e984342
...@@ -10,6 +10,7 @@ from django_comment_common.models import Role, Permission ...@@ -10,6 +10,7 @@ from django_comment_common.models import Role, Permission
from lang_pref import LANGUAGE_KEY from lang_pref import LANGUAGE_KEY
from notification_prefs import NOTIFICATION_PREF_KEY from notification_prefs import NOTIFICATION_PREF_KEY
from notifier_api.views import NotifierUsersViewSet from notifier_api.views import NotifierUsersViewSet
from opaque_keys.edx.locator import CourseLocator
from student.models import CourseEnrollment from student.models import CourseEnrollment
from student.tests.factories import UserFactory, CourseEnrollmentFactory from student.tests.factories import UserFactory, CourseEnrollmentFactory
from user_api.models import UserPreference from user_api.models import UserPreference
...@@ -121,7 +122,10 @@ class NotifierUsersViewSetTest(UrlResetMixin, ModuleStoreTestCase): ...@@ -121,7 +122,10 @@ class NotifierUsersViewSetTest(UrlResetMixin, ModuleStoreTestCase):
self.assertEqual(result["course_info"], {}) self.assertEqual(result["course_info"], {})
def test_course_info_non_existent_course_enrollment(self): def test_course_info_non_existent_course_enrollment(self):
CourseEnrollmentFactory(user=self.user) CourseEnrollmentFactory(
user=self.user,
course_id=CourseLocator(org="dummy", course="dummy", run="non_existent")
)
result = self._get_detail() result = self._get_detail()
self.assertEqual(result["course_info"], {}) self.assertEqual(result["course_info"], {})
......
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