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

4
"""
5

6
import ddt
7
from mock import patch
8
from nose.plugins.attrib import attr
9

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

20
USER_COUNT = 4
21

22

23
@attr('shard_1')
24
@ddt.ddt
25 26 27 28
class TestInstructorEnrollsStudent(ModuleStoreTestCase, LoginEnrollmentTestCase):
    """
    Check Enrollment/Unenrollment with/without auto-enrollment on activation and with/without email notification
    """
29 30

    def setUp(self):
31
        super(TestInstructorEnrollsStudent, self).setUp()
32

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   "
stv committed
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
        self.assertEqual(
            mail.outbox[0].subject,
Don Mitchell committed
221
            'You have been enrolled in {}'.format(course.display_name)
Adam Palay committed
222 223 224
        )
        self.assertEqual(
            mail.outbox[0].body,
Don Mitchell committed
225
            "Dear Autoenrolled Test\n\nYou have been enrolled in {} "
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 "
Don Mitchell committed
229 230 231 232
            "{}://edx.org/courses/{}/\n\n"
            "----\nThis email was automatically sent from edx.org to Autoenrolled Test".format(
                course.display_name, protocol, unicode(course.id)
            )
Adam Palay committed
233 234 235 236
        )

        self.assertEqual(
            mail.outbox[1].subject,
Don Mitchell committed
237
            'You have been invited to register for {}'.format(course.display_name)
Adam Palay committed
238 239 240 241
        )
        self.assertEqual(
            mail.outbox[1].body,
            "Dear student,\n\nYou have been invited to join "
Don Mitchell committed
242
            "{display_name} at edx.org by a member of the "
Adam Palay committed
243 244
            "course staff.\n\n"
            "To finish your registration, please visit "
245
            "{}://edx.org/register and fill out the registration form "
Adam Palay committed
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 "
Don Mitchell committed
248
            "see {display_name} listed on your dashboard.\n\n"
Adam Palay committed
249
            "----\nThis email was automatically sent from edx.org to "
Don Mitchell committed
250
            "student3_1@test.com".format(protocol, display_name=course.display_name)
Adam Palay committed
251
        )
252 253 254 255 256 257 258 259

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

        course = self.course

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

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

267
        # Check the page output
268 269 270 271
        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>')

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

291 292 293 294 295 296 297 298 299
    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)
300

301
    @ddt.data('http', 'https')
302
    @patch('instructor.views.legacy.uses_shib')
303
    def test_enrollment_email_on_shib_on(self, protocol, mock_uses_shib):
304 305 306 307 308 309 310 311
        # 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
312
        url = reverse('instructor_dashboard_legacy', kwargs={'course_id': course.id.to_deprecated_string()})
313 314 315
        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)
316 317 318 319 320 321 322 323 324 325 326

        # 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,
Don Mitchell committed
327
            'You have been enrolled in {}'.format(course.display_name)
328 329 330
        )
        self.assertEqual(
            mail.outbox[0].body,
Don Mitchell committed
331
            "Dear ShibTest Enrolled\n\nYou have been enrolled in {} "
332 333 334
            "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 "
Don Mitchell committed
335 336 337 338
            "{}://edx.org/courses/{}/\n\n"
            "----\nThis email was automatically sent from edx.org to ShibTest Enrolled".format(
                course.display_name, protocol, unicode(course.id)
            )
339 340 341 342
        )

        self.assertEqual(
            mail.outbox[1].subject,
Don Mitchell committed
343
            'You have been invited to register for {}'.format(course.display_name)
344 345 346 347
        )
        self.assertEqual(
            mail.outbox[1].body,
            "Dear student,\n\nYou have been invited to join "
Don Mitchell committed
348
            "{} at edx.org by a member of the "
349
            "course staff.\n\n"
Don Mitchell committed
350
            "To access the course visit {}://edx.org/courses/{}/ and login.\n\n"
351
            "----\nThis email was automatically sent from edx.org to "
Don Mitchell committed
352 353 354
            "student5_1@test.com".format(
                course.display_name, protocol, course.id
            )
355
        )