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

MB-714 | Fix log message when generating certificates

Browse Source 
[MB-714]
- While investigating a certificate generation issue I happened to notice a log statement whose arguments were backwards
This commit is contained in:
Justin Hynes
2020-11-17 14:14:07 -05:00
parent 266e27ff80
commit dcfa7ea72c
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lms/djangoapps/certificates/api.py
View File

@@ -208,7 +208,7 @@ def generate_user_certificates(student, course_key, course=None, insecure=False,
if beta_testers_queryset.filter(username=student.username): if beta_testers_queryset.filter(username=student.username):
message = u'Cancelling course certificate generation for user [{}] against course [{}], user is a Beta Tester.' message = u'Cancelling course certificate generation for user [{}] against course [{}], user is a Beta Tester.'
log.info(message.format(course_key, student.username)) log.info(message.format(student.username, course_key))
return return
xqueue = XQueueCertInterface() xqueue = XQueueCertInterface()

3
lms/djangoapps/instructor/tests/test_api.py
View File

@@ -1967,7 +1967,8 @@ class TestInstructorAPIBulkBetaEnrollment(SharedModuleStoreTestCase, LoginEnroll
with LogCapture() as capture: with LogCapture() as capture:
message = u'Cancelling course certificate generation for user [{}] against course [{}], ' \ message = u'Cancelling course certificate generation for user [{}] against course [{}], ' \
u'user is a Beta Tester.' u'user is a Beta Tester.'
message = message.format(self.course.id, self.beta_tester.username) message = message.format(self.beta_tester.username, self.course.id)
generate_user_certificates(self.beta_tester, self.course.id, self.course) generate_user_certificates(self.beta_tester, self.course.id, self.course)
capture.check_present(('edx.certificate', 'INFO', message)) capture.check_present(('edx.certificate', 'INFO', message))