MICROBA-1032 Remove command in favor of querying the db for more detailed info (#26787)
This commit is contained in:
Christie Rice
@@ -1,8 +0,0 @@
|
||||
"""Deprecated import support. Auto-generated by import_shims/generate_shims.sh."""
|
||||
# pylint: disable=redefined-builtin,wrong-import-position,wildcard-import,useless-suppression,line-too-long
|
||||
|
||||
from import_shims.warn import warn_deprecated_import
|
||||
|
||||
warn_deprecated_import('certificates.management.commands.gen_cert_report', 'lms.djangoapps.certificates.management.commands.gen_cert_report')
|
||||
|
||||
from lms.djangoapps.certificates.management.commands.gen_cert_report import *
|
||||
@@ -1,8 +0,0 @@
|
||||
"""Deprecated import support. Auto-generated by import_shims/generate_shims.sh."""
|
||||
# pylint: disable=redefined-builtin,wrong-import-position,wildcard-import,useless-suppression,line-too-long
|
||||
|
||||
from import_shims.warn import warn_deprecated_import
|
||||
|
||||
warn_deprecated_import('certificates.management.commands.tests.test_gen_cert_report', 'lms.djangoapps.certificates.management.commands.tests.test_gen_cert_report')
|
||||
|
||||
from lms.djangoapps.certificates.management.commands.tests.test_gen_cert_report import *
|
||||
@@ -1,128 +0,0 @@
|
||||
"""
|
||||
Generate a report of certificate statuses
|
||||
"""
|
||||
|
||||
|
||||
from django.contrib.auth import get_user_model
|
||||
from django.core.management.base import BaseCommand, CommandError
|
||||
from django.db.models import Count
|
||||
from opaque_keys.edx.keys import CourseKey
|
||||
|
||||
from lms.djangoapps.certificates.models import GeneratedCertificate
|
||||
|
||||
User = get_user_model()
|
||||
|
||||
|
||||
class Command(BaseCommand):
|
||||
"""
|
||||
Management command to generate a certificate status
|
||||
report for a given course.
|
||||
"""
|
||||
|
||||
help = """
|
||||
|
||||
Generate a certificate status report for a given course.
|
||||
This command does not do anything other than report the current
|
||||
certificate status.
|
||||
|
||||
generating - A request has been made to generate a certificate,
|
||||
but it has not been generated yet.
|
||||
regenerating - A request has been made to regenerate a certificate,
|
||||
but it has not been generated yet.
|
||||
deleting - A request has been made to delete a certificate.
|
||||
|
||||
deleted - The certificate has been deleted.
|
||||
downloadable - The certificate is available for download.
|
||||
notpassing - The student was graded but is not passing
|
||||
|
||||
"""
|
||||
|
||||
def add_arguments(self, parser):
|
||||
parser.add_argument(
|
||||
'-c', '--course',
|
||||
metavar='COURSE_ID',
|
||||
dest='course',
|
||||
default=None,
|
||||
help='Only generate for COURSE_ID'
|
||||
)
|
||||
|
||||
def handle(self, *args, **options):
|
||||
# Find all courses that have ended
|
||||
|
||||
if options['course']:
|
||||
course_id = CourseKey.from_string(options['course'])
|
||||
else:
|
||||
raise CommandError("You must specify a course")
|
||||
|
||||
cert_data = {}
|
||||
|
||||
# find students who are active
|
||||
# number of enrolled students = downloadable + notpassing
|
||||
print("Looking up certificate states for {}".format(options['course']))
|
||||
enrolled_current = User.objects.filter(
|
||||
courseenrollment__course_id=course_id,
|
||||
courseenrollment__is_active=True
|
||||
)
|
||||
enrolled_total = User.objects.filter(
|
||||
courseenrollment__course_id=course_id
|
||||
)
|
||||
verified_enrolled = GeneratedCertificate.objects.filter(
|
||||
course_id__exact=course_id,
|
||||
mode__exact='verified'
|
||||
)
|
||||
honor_enrolled = GeneratedCertificate.objects.filter(
|
||||
course_id__exact=course_id,
|
||||
mode__exact='honor'
|
||||
)
|
||||
audit_enrolled = GeneratedCertificate.objects.filter(
|
||||
course_id__exact=course_id,
|
||||
mode__exact='audit'
|
||||
)
|
||||
|
||||
cert_data[course_id] = {
|
||||
'enrolled_current': enrolled_current.count(),
|
||||
'enrolled_total': enrolled_total.count(),
|
||||
'verified_enrolled': verified_enrolled.count(),
|
||||
'honor_enrolled': honor_enrolled.count(),
|
||||
'audit_enrolled': audit_enrolled.count()
|
||||
}
|
||||
|
||||
status_tally = GeneratedCertificate.objects.filter(
|
||||
course_id__exact=course_id
|
||||
).values('status').annotate(
|
||||
dcount=Count('status')
|
||||
)
|
||||
|
||||
cert_data[course_id].update(
|
||||
{
|
||||
status['status']: status['dcount'] for status in status_tally
|
||||
}
|
||||
)
|
||||
|
||||
mode_tally = GeneratedCertificate.objects.filter(
|
||||
course_id__exact=course_id,
|
||||
status__exact='downloadable'
|
||||
).values('mode').annotate(
|
||||
dcount=Count('mode')
|
||||
)
|
||||
cert_data[course_id].update(
|
||||
{mode['mode']: mode['dcount'] for mode in mode_tally}
|
||||
)
|
||||
|
||||
# all states we have seen far all courses
|
||||
status_headings = sorted(
|
||||
{status for course in cert_data for status in cert_data[course]} # pylint: disable=consider-using-set-comprehension
|
||||
)
|
||||
|
||||
# print the heading for the report
|
||||
print("{:>26}".format("course ID"), end=' ')
|
||||
print(' '.join([f"{heading:>16}" for heading in status_headings]))
|
||||
|
||||
# print the report
|
||||
print("{:>26}".format(str(course_id)), end=' ')
|
||||
for heading in status_headings:
|
||||
if heading in cert_data[course_id]:
|
||||
print("{:>16}".format(cert_data[course_id][heading]), end=' ')
|
||||
else:
|
||||
print(" " * 16, end=' ')
|
||||
print()
|
||||
@@ -1,18 +0,0 @@
|
||||
"""
|
||||
Extremely basic tests for the gen_cert_report command
|
||||
"""
|
||||
|
||||
|
||||
import pytest
|
||||
from django.core.management import call_command
|
||||
|
||||
|
||||
def test_cert_report_help(capsys):
|
||||
"""
|
||||
Basic test to see if the command will parse and get args
|
||||
"""
|
||||
with pytest.raises(SystemExit):
|
||||
call_command('gen_cert_report', '--help')
|
||||
|
||||
out, err = capsys.readouterr() # pylint: disable=unused-variable
|
||||
assert "COURSE_ID" in out
|
||||
Reference in New Issue
Block a user