Skip to content

Commit 5c1b48c

Browse files
mariajgrimaldiBetoFandino
authored andcommitted
feat: add first batch of Open edX Filters
* Add PreEnrollmentFilter * Add PreRegisterFilter * Add PreLoginFilter For more info: openedx/edx-platform#29449 Some events that were already on the platform were also added: * Add COURSE_ENROLLMENT_CHANGED: sent after the enrollment update * Add COURSE_ENROLLMENT_CREATED event after the user's enrollment creation * Add COURSE_UNENROLLMENT_COMPLETED: sent after the user's unenrollment For more info: openedx/edx-platform#28266 openedx/edx-platform#28640
1 parent 3b42779 commit 5c1b48c

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

common/djangoapps/student/models.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,7 @@
6666
COURSE_ENROLLMENT_CREATED,
6767
COURSE_UNENROLLMENT_COMPLETED,
6868
)
69+
6970
from openedx_filters.learning.filters import CourseEnrollmentStarted, CourseUnenrollmentStarted
7071
import openedx.core.djangoapps.django_comment_common.comment_client as cc
7172
from common.djangoapps.course_modes.models import CourseMode, get_cosmetic_verified_display_price

common/djangoapps/student/tests/test_filters.py

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,19 @@
22
Test that various filters are fired for models/views in the student app.
33
"""
44
from django.http import HttpResponse
5-
from django.test import override_settings
65
from django.urls import reverse
7-
from openedx_filters import PipelineStep
86
from openedx_filters.learning.filters import DashboardRenderStarted, CourseEnrollmentStarted, CourseUnenrollmentStarted
97
from rest_framework import status
8+
9+
10+
from common.djangoapps.student.models import UnenrollmentNotAllowed
11+
12+
from django.test import override_settings
1013
from xmodule.modulestore.tests.django_utils import ModuleStoreTestCase
1114
from xmodule.modulestore.tests.factories import CourseFactory
12-
13-
from common.djangoapps.student.models import CourseEnrollment, EnrollmentNotAllowed, UnenrollmentNotAllowed
15+
from openedx_filters.learning.filters import CourseEnrollmentStarted
16+
from openedx_filters import PipelineStep
17+
from common.djangoapps.student.models import CourseEnrollment, EnrollmentNotAllowed
1418
from common.djangoapps.student.tests.factories import UserFactory, UserProfileFactory
1519
from openedx.core.djangolib.testing.utils import skip_unless_lms
1620

@@ -109,7 +113,6 @@ def run_filter(self, context, template_name): # pylint: disable=arguments-diffe
109113
response=response,
110114
)
111115

112-
113116
@skip_unless_lms
114117
class EnrollmentFiltersTest(ModuleStoreTestCase):
115118
"""

0 commit comments

Comments
 (0)