Andal.LND/edx-platform
4
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

added more tests on suggestion

Browse Source
This commit is contained in:
Amir Qayyum Khan
2018-05-23 18:28:11 +05:00
parent 594a7c4ac6
commit 70440672c9
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
lms/djangoapps/instructor/tests/test_tools.py
View File

@@ -389,6 +389,9 @@ class TestStudentFromIdentifier(TestCase):
with self.assertRaises(MultipleObjectsReturned):
tools.get_student_from_identifier(self.duplicate_user_name.username)
# can get student with alternative identifier, in this case email.
assert self.duplicate_user_name == tools.get_student_from_identifier(self.duplicate_user_name.email)
def test_student_email_has_conflict_with_others_username(self):
"""
An edge case where there is a user A with email example: foo@touchstone.com and
@@ -397,6 +400,9 @@ class TestStudentFromIdentifier(TestCase):
with self.assertRaises(MultipleObjectsReturned):
tools.get_student_from_identifier(self.duplicate_email.email)
# can get student with alternative identifier, in this case username.
assert self.duplicate_email == tools.get_student_from_identifier(self.duplicate_email.username)
def test_invalid_student_id(self):
"""Test with invalid identifier"""
with self.assertRaises(User.DoesNotExist):