test_legacy_enrollment.py 14.6 KB
Newer Older
1
"""
dcadams committed
2
Unit tests for enrollment methods in views.py
3

4
"""
5

6
import ddt
7 8
from mock import patch

9
from django.test.utils import override_settings
10
from django.contrib.auth.models import User
11
from django.core.urlresolvers import reverse
12
from courseware.tests.helpers import LoginEnrollmentTestCase
13
from courseware.tests.modulestore_config import TEST_DATA_MIXED_MODULESTORE
14 15 16
from xmodule.modulestore.tests.factories import CourseFactory
from student.tests.factories import UserFactory, CourseEnrollmentFactory, AdminFactory
from xmodule.modulestore.tests.django_utils import ModuleStoreTestCase
17
from student.models import CourseEnrollment, CourseEnrollmentAllowed
18
from instructor.views.legacy import get_and_clean_student_list, send_mail_to_student
19
from django.core import mail
20

21
USER_COUNT = 4
22

23

24
@ddt.ddt
25
@override_settings(MODULESTORE=TEST_DATA_MIXED_MODULESTORE)
26 27 28 29
class TestInstructorEnrollsStudent(ModuleStoreTestCase, LoginEnrollmentTestCase):
    """
    Check Enrollment/Unenrollment with/without auto-enrollment on activation and with/without email notification
    """
30 31 32

    def setUp(self):

33 34
        instructor = AdminFactory.create()
        self.client.login(username=instructor.username, password='test')
35

36
        self.course = CourseFactory.create()
37

38 39 40 41
        self.users = [
            UserFactory.create(username="student%d" % i, email="student%d@test.com" % i)
            for i in xrange(USER_COUNT)
        ]
42

43 44
        for user in self.users:
            CourseEnrollmentFactory.create(user=user, course_id=self.course.id)
dcadams committed
45

46 47
        # Empty the test outbox
        mail.outbox = []
dcadams committed
48

49 50 51 52
    def test_unenrollment_email_off(self):
        """
        Do un-enrollment email off test
        """
53

54
        course = self.course
dcadams committed
55

56
        # Run the Un-enroll students command
Calen Pennington committed
57
        url = reverse('instructor_dashboard_legacy', kwargs={'course_id': course.id.to_deprecated_string()})
58 59 60 61 62 63 64
        response = self.client.post(
            url,
            {
                'action': 'Unenroll multiple students',
                'multiple_students': 'student0@test.com student1@test.com'
            }
        )
dcadams committed
65

66
        # Check the page output
67
        self.assertContains(response, '<td>student0@test.com</td>')
68 69
        self.assertContains(response, '<td>student1@test.com</td>')
        self.assertContains(response, '<td>un-enrolled</td>')
dcadams committed
70

71
        # Check the enrollment table
72
        user = User.objects.get(email='student0@test.com')
73
        self.assertFalse(CourseEnrollment.is_enrolled(user, course.id))
dcadams committed
74

75
        user = User.objects.get(email='student1@test.com')
76
        self.assertFalse(CourseEnrollment.is_enrolled(user, course.id))
77

78
        # Check the outbox
79 80 81 82 83 84 85 86
        self.assertEqual(len(mail.outbox), 0)

    def test_enrollment_new_student_autoenroll_on_email_off(self):
        """
        Do auto-enroll on, email off test
        """

        course = self.course
dcadams committed
87

88
        # Run the Enroll students command
Calen Pennington committed
89
        url = reverse('instructor_dashboard_legacy', kwargs={'course_id': course.id.to_deprecated_string()})
90
        response = self.client.post(url, {'action': 'Enroll multiple students', 'multiple_students': 'student1_1@test.com, student1_2@test.com', 'auto_enroll': 'on'})
dcadams committed
91

92
        # Check the page output
93 94
        self.assertContains(response, '<td>student1_1@test.com</td>')
        self.assertContains(response, '<td>student1_2@test.com</td>')
95
        self.assertContains(response, '<td>user does not exist, enrollment allowed, pending with auto enrollment on</td>')
dcadams committed
96

97
        # Check the outbox
98 99
        self.assertEqual(len(mail.outbox), 0)

100
        # Check the enrollmentallowed db entries
101
        cea = CourseEnrollmentAllowed.objects.filter(email='student1_1@test.com', course_id=course.id)
102
        self.assertEqual(1, cea[0].auto_enroll)
103
        cea = CourseEnrollmentAllowed.objects.filter(email='student1_2@test.com', course_id=course.id)
104
        self.assertEqual(1, cea[0].auto_enroll)
dcadams committed
105

106
        # Check there is no enrollment db entry other than for the other students
107
        ce = CourseEnrollment.objects.filter(course_id=course.id, is_active=1)
108
        self.assertEqual(4, len(ce))
dcadams committed
109

110
        # Create and activate student accounts with same email
111
        self.student1 = 'student1_1@test.com'
112 113 114
        self.password = 'bar'
        self.create_account('s1_1', self.student1, self.password)
        self.activate_user(self.student1)
dcadams committed
115

116
        self.student2 = 'student1_2@test.com'
117 118
        self.create_account('s1_2', self.student2, self.password)
        self.activate_user(self.student2)
dcadams committed
119

120
        # Check students are enrolled
121
        user = User.objects.get(email='student1_1@test.com')
122
        self.assertTrue(CourseEnrollment.is_enrolled(user, course.id))
dcadams committed
123

124
        user = User.objects.get(email='student1_2@test.com')
125
        self.assertTrue(CourseEnrollment.is_enrolled(user, course.id))
126

127 128 129 130 131 132 133
    def test_repeat_enroll(self):
        """
        Try to enroll an already enrolled student
        """

        course = self.course

Calen Pennington committed
134
        url = reverse('instructor_dashboard_legacy', kwargs={'course_id': course.id.to_deprecated_string()})
135 136 137 138 139 140 141 142 143 144
        response = self.client.post(url, {'action': 'Enroll multiple students', 'multiple_students': 'student0@test.com', 'auto_enroll': 'on'})
        self.assertContains(response, '<td>student0@test.com</td>')
        self.assertContains(response, '<td>already enrolled</td>')

    def test_enrollmemt_new_student_autoenroll_off_email_off(self):
        """
        Do auto-enroll off, email off test
        """

        course = self.course
dcadams committed
145

146
        # Run the Enroll students command
Calen Pennington committed
147
        url = reverse('instructor_dashboard_legacy', kwargs={'course_id': course.id.to_deprecated_string()})
148
        response = self.client.post(url, {'action': 'Enroll multiple students', 'multiple_students': 'student2_1@test.com, student2_2@test.com'})
dcadams committed
149

150
        # Check the page output
151 152
        self.assertContains(response, '<td>student2_1@test.com</td>')
        self.assertContains(response, '<td>student2_2@test.com</td>')
153
        self.assertContains(response, '<td>user does not exist, enrollment allowed, pending with auto enrollment off</td>')
dcadams committed
154

155
        # Check the outbox
156 157
        self.assertEqual(len(mail.outbox), 0)

158
        # Check the enrollmentallowed db entries
159
        cea = CourseEnrollmentAllowed.objects.filter(email='student2_1@test.com', course_id=course.id)
160
        self.assertEqual(0, cea[0].auto_enroll)
161
        cea = CourseEnrollmentAllowed.objects.filter(email='student2_2@test.com', course_id=course.id)
162
        self.assertEqual(0, cea[0].auto_enroll)
dcadams committed
163

164
        # Check there is no enrollment db entry other than for the setup instructor and students
165
        ce = CourseEnrollment.objects.filter(course_id=course.id, is_active=1)
166
        self.assertEqual(4, len(ce))
dcadams committed
167

168
        # Create and activate student accounts with same email
169
        self.student = 'student2_1@test.com'
170 171 172
        self.password = 'bar'
        self.create_account('s2_1', self.student, self.password)
        self.activate_user(self.student)
dcadams committed
173

174
        self.student = 'student2_2@test.com'
175
        self.create_account('s2_2', self.student, self.password)
dcadams committed
176 177
        self.activate_user(self.student)

178
        # Check students are not enrolled
179
        user = User.objects.get(email='student2_1@test.com')
180 181
        self.assertFalse(CourseEnrollment.is_enrolled(user, course.id))

182
        user = User.objects.get(email='student2_2@test.com')
183
        self.assertFalse(CourseEnrollment.is_enrolled(user, course.id))
184 185

    def test_get_and_clean_student_list(self):
186
        """
187
        Clean user input test
188
        """
189

190
        string = "abc@test.com, def@test.com ghi@test.com \n \n jkl@test.com   \n mno@test.com   "
191
        cleaned_string, cleaned_string_lc = get_and_clean_student_list(string)
192 193
        self.assertEqual(cleaned_string, ['abc@test.com', 'def@test.com', 'ghi@test.com', 'jkl@test.com', 'mno@test.com'])

194 195
    @ddt.data('http', 'https')
    def test_enrollment_email_on(self, protocol):
196 197 198 199 200 201
        """
        Do email on enroll test
        """

        course = self.course

202
        # Create activated, but not enrolled, user
203
        UserFactory.create(username="student3_0", email="student3_0@test.com", first_name='Autoenrolled')
204

Calen Pennington committed
205
        url = reverse('instructor_dashboard_legacy', kwargs={'course_id': course.id.to_deprecated_string()})
206 207 208
        params = {'action': 'Enroll multiple students', 'multiple_students': 'student3_0@test.com, student3_1@test.com, student3_2@test.com', 'auto_enroll': 'on', 'email_students': 'on'}
        environ = {'wsgi.url_scheme': protocol}
        response = self.client.post(url, params, **environ)
209

210
        # Check the page output
211 212 213 214 215 216
        self.assertContains(response, '<td>student3_0@test.com</td>')
        self.assertContains(response, '<td>student3_1@test.com</td>')
        self.assertContains(response, '<td>student3_2@test.com</td>')
        self.assertContains(response, '<td>added, email sent</td>')
        self.assertContains(response, '<td>user does not exist, enrollment allowed, pending with auto enrollment on, email sent</td>')

217
        # Check the outbox
218
        self.assertEqual(len(mail.outbox), 3)
Adam Palay committed
219 220 221 222 223 224
        self.assertEqual(
            mail.outbox[0].subject,
            'You have been enrolled in Robot Super Course'
        )
        self.assertEqual(
            mail.outbox[0].body,
225
            "Dear Autoenrolled Test\n\nYou have been enrolled in Robot Super Course "
Adam Palay committed
226 227 228
            "at edx.org by a member of the course staff. "
            "The course should now appear on your edx.org dashboard.\n\n"
            "To start accessing course materials, please visit "
229 230
            "{}://edx.org/courses/MITx/999/Robot_Super_Course/\n\n"
            "----\nThis email was automatically sent from edx.org to Autoenrolled Test".format(protocol)
Adam Palay committed
231 232 233 234 235 236 237 238 239 240 241 242
        )

        self.assertEqual(
            mail.outbox[1].subject,
            'You have been invited to register for Robot Super Course'
        )
        self.assertEqual(
            mail.outbox[1].body,
            "Dear student,\n\nYou have been invited to join "
            "Robot Super Course at edx.org by a member of the "
            "course staff.\n\n"
            "To finish your registration, please visit "
243
            "{}://edx.org/register and fill out the registration form "
Adam Palay committed
244 245 246 247
            "making sure to use student3_1@test.com in the E-mail field.\n"
            "Once you have registered and activated your account, you will "
            "see Robot Super Course listed on your dashboard.\n\n"
            "----\nThis email was automatically sent from edx.org to "
248
            "student3_1@test.com".format(protocol)
Adam Palay committed
249
        )
250 251 252 253 254 255 256 257

    def test_unenrollment_email_on(self):
        """
        Do email on unenroll test
        """

        course = self.course

258
        # Create invited, but not registered, user
259 260 261
        cea = CourseEnrollmentAllowed(email='student4_0@test.com', course_id=course.id)
        cea.save()

Calen Pennington committed
262
        url = reverse('instructor_dashboard_legacy', kwargs={'course_id': course.id.to_deprecated_string()})
263 264
        response = self.client.post(url, {'action': 'Unenroll multiple students', 'multiple_students': 'student4_0@test.com, student2@test.com, student3@test.com', 'email_students': 'on'})

265
        # Check the page output
266 267 268 269
        self.assertContains(response, '<td>student2@test.com</td>')
        self.assertContains(response, '<td>student3@test.com</td>')
        self.assertContains(response, '<td>un-enrolled, email sent</td>')

270
        # Check the outbox
271
        self.assertEqual(len(mail.outbox), 3)
Adam Palay committed
272 273 274 275 276 277 278 279 280 281 282 283 284 285 286 287 288
        self.assertEqual(
            mail.outbox[0].subject,
            'You have been un-enrolled from Robot Super Course'
        )
        self.assertEqual(
            mail.outbox[0].body,
            "Dear Student,\n\nYou have been un-enrolled from course "
            "Robot Super Course by a member of the course staff. "
            "Please disregard the invitation previously sent.\n\n"
            "----\nThis email was automatically sent from edx.org "
            "to student4_0@test.com"
        )
        self.assertEqual(
            mail.outbox[1].subject,
            'You have been un-enrolled from Robot Super Course'
        )

289 290 291 292 293 294 295 296 297
    def test_send_mail_to_student(self):
        """
        Do invalid mail template test
        """

        d = {'message': 'message_type_that_doesn\'t_exist'}

        send_mail_ret = send_mail_to_student('student0@test.com', d)
        self.assertFalse(send_mail_ret)
298

299
    @ddt.data('http', 'https')
300
    @patch('instructor.views.legacy.uses_shib')
301
    def test_enrollment_email_on_shib_on(self, protocol, mock_uses_shib):
302 303 304 305 306 307 308 309
        # Do email on enroll, shibboleth on test

        course = self.course
        mock_uses_shib.return_value = True

        # Create activated, but not enrolled, user
        UserFactory.create(username="student5_0", email="student5_0@test.com", first_name="ShibTest", last_name="Enrolled")

Calen Pennington committed
310
        url = reverse('instructor_dashboard_legacy', kwargs={'course_id': course.id.to_deprecated_string()})
311 312 313
        params = {'action': 'Enroll multiple students', 'multiple_students': 'student5_0@test.com, student5_1@test.com', 'auto_enroll': 'on', 'email_students': 'on'}
        environ = {'wsgi.url_scheme': protocol}
        response = self.client.post(url, params, **environ)
314 315 316 317 318 319 320 321 322 323 324 325 326 327 328 329 330 331 332

        # Check the page output
        self.assertContains(response, '<td>student5_0@test.com</td>')
        self.assertContains(response, '<td>student5_1@test.com</td>')
        self.assertContains(response, '<td>added, email sent</td>')
        self.assertContains(response, '<td>user does not exist, enrollment allowed, pending with auto enrollment on, email sent</td>')

        # Check the outbox
        self.assertEqual(len(mail.outbox), 2)
        self.assertEqual(
            mail.outbox[0].subject,
            'You have been enrolled in Robot Super Course'
        )
        self.assertEqual(
            mail.outbox[0].body,
            "Dear ShibTest Enrolled\n\nYou have been enrolled in Robot Super Course "
            "at edx.org by a member of the course staff. "
            "The course should now appear on your edx.org dashboard.\n\n"
            "To start accessing course materials, please visit "
333 334
            "{}://edx.org/courses/MITx/999/Robot_Super_Course/\n\n"
            "----\nThis email was automatically sent from edx.org to ShibTest Enrolled".format(protocol)
335 336 337 338 339 340 341 342 343 344 345
        )

        self.assertEqual(
            mail.outbox[1].subject,
            'You have been invited to register for Robot Super Course'
        )
        self.assertEqual(
            mail.outbox[1].body,
            "Dear student,\n\nYou have been invited to join "
            "Robot Super Course at edx.org by a member of the "
            "course staff.\n\n"
346
            "To access the course visit {}://edx.org/courses/MITx/999/Robot_Super_Course/ and login.\n\n"
347
            "----\nThis email was automatically sent from edx.org to "
348
            "student5_1@test.com".format(protocol)
349
        )