Skip to content

Commit

Permalink
fix: add logger warning for dupe
Browse files Browse the repository at this point in the history
  • Loading branch information
varshamenon4 committed Aug 27, 2024
1 parent 00413ee commit b16da9b
Showing 1 changed file with 6 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -68,9 +68,15 @@ def add_course_staff_from_csv(self, csv_file, batch_size, batch_delay):
for i in range(0, len(reader), batch_size):
users_list = []
for row in reader[i:i + batch_size]:
username = row.get('username')
email = row.get('email')
try:
users_list.append(User.objects.get_or_create(username=row.get('username'), email=row.get('email')))
except IntegrityError:
logger.warning(
f'User with username={username} and email={email} was not created due to an existing duplicate '
f'user with username.'
)
continue

Check failure on line 80 in edx_exams/apps/core/management/commands/bulk_add_course_staff.py

View workflow job for this annotation

GitHub Actions / tests (ubuntu-20.04, 3.8, django42)

Missing coverage

Missing coverage on lines 75-80
users_dict = dict([(u.username, u) for (u, c) in users_list])
users.update(users_dict)
Expand Down

0 comments on commit b16da9b

Please sign in to comment.