Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,8 @@ def append_users_lists(self, file_handler, user_model):
userdata = record.split(',')
username = userdata[0].strip()
user_email = userdata[1].strip()
if username.lower() == 'username' and user_email.lower() in ('email', 'user_email'):
continue
Comment on lines 70 to +74
try:
users.append(User.objects.get(username=username, email=user_email))
except user_model.DoesNotExist:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,30 @@ def test_successful_retire_with_userfile(setup_retirement_states): # lint-amnes
remove_user_file()


@skip_unless_lms
def test_successful_retire_with_userfile_header(
setup_retirement_states
): # lint-amnesty, pylint: disable=redefined-outer-name, unused-argument
user = UserFactory.create(username='header-user', email="header-user@example.com")
username = user.username
user_email = user.email
with open(user_file, 'w', newline='') as file:
write = csv.writer(file)
write.writerow(['username', 'email'])
write.writerow([username, user_email])
Comment on lines +98 to +108

try:
call_command('retire_user', user_file=user_file)
user = User.objects.get(username=username)
retired_user_status = UserRetirementStatus.objects.all()[0]
assert retired_user_status.original_username == username
assert retired_user_status.original_email == user_email
# Make sure that we have changed the email address linked to the original user
assert user.email != user_email
finally:
remove_user_file()


@skip_unless_lms
def test_retire_user_with_usename_email_mismatch(setup_retirement_states): # lint-amnesty, pylint: disable=redefined-outer-name, unused-argument
create_user_file(True)
Expand Down
Loading