fixup! chore: add support for Python 3.11 + 3.12 and Django 4.2 + 5.2

Author: Agrendalath

completion_aggregator/api/v1/views.py

@@ -563,11 +563,16 @@ def get(self, request, course_key):
             course_key=course_key,
             aggregation_name='course',
             user_id__in=[enrollment.user_id for enrollment in enrollments])
-        completion_stats = aggregator_qs.aggregate(
-            possible=Avg('possible'),
-            earned=Sum('earned') / len(enrollments),
-            percent=Sum('earned') / (Avg('possible') * len(enrollments)))
-        completion_stats['course_key'] = course_key
+        num_enrollments = len(enrollments)
+        aggregates = aggregator_qs.aggregate(avg_possible=Avg("possible"), total_earned=Sum("earned"))
+        avg_possible = aggregates["avg_possible"] or 0
+        total_earned = aggregates["total_earned"] or 0
+        completion_stats = {
+            "course_key": course_key,
+            "possible": avg_possible,
+            "earned": total_earned / num_enrollments if num_enrollments else 0,
+            "percent": total_earned / (avg_possible * num_enrollments) if avg_possible and num_enrollments else 0,
+        }
 
         serializer = self.get_serializer_class()(
             instance=completion_stats,

completion_aggregator/management/commands/migrate_progress.py

@@ -72,6 +72,6 @@ def _configure_logging(self, options):
 
         if options.get('verbosity') == 0:
             log_level = logging.WARNING
-        elif options.get('verbosity') >= 1:
+        else:
             log_level = logging.INFO
         log.setLevel(log_level)

completion_aggregator/utils.py

@@ -1,9 +1,10 @@
 """
 Various utility functionality.
 """
+from datetime import timezone
+
 import django
 from django.contrib.auth import get_user_model
-from django.utils import timezone
 from django.utils.translation import gettext as _
 
 WAFFLE_AGGREGATE_STALE_FROM_SCRATCH = 'completion_aggregator.aggregate_stale_from_scratch'

pylintrc

@@ -64,7 +64,7 @@
 # SERIOUSLY.
 #
 # ------------------------------
-# Generated by edx-lint version: 5.3.2
+# Generated by edx-lint version: 5.6.0
 # ------------------------------
 [MASTER]
 ignore = migrations
@@ -260,6 +260,7 @@ enable =
 disable = 
 	import-outside-toplevel,
 	too-many-arguments,
+	too-many-positional-arguments,
 	too-many-instance-attributes,
 	too-many-locals,
 	invalid-name,
@@ -366,6 +367,6 @@ ext-import-graph =
 int-import-graph = 
 
 [EXCEPTIONS]
-overgeneral-exceptions = Exception
+overgeneral-exceptions = builtins.Exception
 
-# 63e94d8b98e89cb5ced90a5c57c522c6889d2f1e
+# 893b2dd614b484d9379a0604462f727e63fe557c

pylintrc_tweaks

@@ -8,6 +8,7 @@ load-plugins = edx_lint.pylint,pylint_django,pylint_celery
 disable =
     import-outside-toplevel,
     too-many-arguments,
+    too-many-positional-arguments,
     too-many-instance-attributes,
     too-many-locals,
     invalid-name,
@@ -20,3 +21,6 @@ disable =
     superfluous-parens,
     unnecessary-pass,
     too-many-branches,
+
+[EXCEPTIONS]
+overgeneral-exceptions = builtins.Exception

test_settings.py

@@ -91,6 +91,18 @@ def root(*args):
     },
 ]
 USE_TZ = True
+DEFAULT_AUTO_FIELD = 'django.db.models.AutoField'
+
+# Suppress noisy import errors from event_routing_backends when running outside LMS.
+LOGGING = {
+    'version': 1,
+    'disable_existing_loggers': False,
+    'loggers': {
+        'event_routing_backends.helpers': {
+            'level': 'CRITICAL',
+        },
+    },
+}
 
 # Enables event tracking in the tests, see https://github.com/openedx/event-tracking
 EVENT_TRACKING_ENABLED = True

tox.ini

@@ -19,8 +19,11 @@ match-dir = (?!migrations)
 
 [pytest]
 DJANGO_SETTINGS_MODULE = test_settings
-addopts = --cov completion_aggregator --cov-report term-missing --cov-report xml -W error
+addopts = --cov completion_aggregator --cov-report term-missing --cov-report xml
 norecursedirs = .* docs requirements
+filterwarnings =
+    error
+    ignore:'cgi' is deprecated:DeprecationWarning:webob
 
 [testenv]
 deps =