fix: call enabled_aggregator_events with a settings arg

and rearrange INSTALLED_APPS to fix tests.
open-craft · Jun 13, 2024 · e82c49d · e82c49d
1 parent bb2f8e6
commit e82c49d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/completion_aggregator/settings/common.py b/completion_aggregator/settings/common.py
@@ -65,6 +65,7 @@ def plugin_settings(settings):
     ]
     settings.EVENT_TRACKING_BACKENDS_ALLOWED_XAPI_EVENTS += enabled_aggregator_events
     settings.EVENT_TRACKING_BACKENDS.update(event_tracking_backends_config(
+        settings,
         settings.EVENT_TRACKING_BACKENDS_ALLOWED_XAPI_EVENTS,
         settings.EVENT_TRACKING_BACKENDS_ALLOWED_CALIPER_EVENTS,
     ))
diff --git a/test_settings.py b/test_settings.py
@@ -62,13 +62,13 @@ def root(*args):
     'django.contrib.auth',
     'django.contrib.sessions',
     'django.contrib.admin',
-    'completion_aggregator',
     'completion',
     'oauth2_provider',
     'waffle',
     'test_utils.test_app',
     'eventtracking.django.apps.EventTrackingConfig',
     'event_routing_backends',
+    'completion_aggregator',
 )
 
 LOCALE_PATHS = [root('completion_aggregator', 'conf', 'locale')]
@@ -99,6 +99,8 @@ def root(*args):
 
 EVENT_TRACKING_ENABLED = True
 EVENT_TRACKING_BACKENDS = {}
+EVENT_TRACKING_BACKENDS_ALLOWED_XAPI_EVENTS = []
+EVENT_TRACKING_BACKENDS_ALLOWED_CALIPER_EVENTS = []
 LMS_ROOT_URL = "http://lms.url"
 
 # pylint: disable=unused-import,wrong-import-position