fix: course name missing from ORA notification context (#2220)

openassessment/__init__.py

@@ -2,4 +2,4 @@
 Initialization Information for Open Assessment Module
 """
 
-__version__ = '6.11.1'
+__version__ = '6.11.2'

openassessment/xblock/test/test_notifications.py

@@ -2,7 +2,7 @@
 Unit test for notification util
 """
 import unittest
-from unittest.mock import patch
+from unittest.mock import patch, MagicMock
 
 from openassessment.xblock.utils.notifications import send_staff_notification
 
@@ -11,8 +11,9 @@ class TestSendStaffNotification(unittest.TestCase):
  """
  Test for send_staff_notification function
  """
+ @patch('openassessment.xblock.utils.notifications.modulestore')
  @patch('openassessment.xblock.utils.notifications.COURSE_NOTIFICATION_REQUESTED.send_event')
- def test_send_staff_notification(self, mock_send_event):
+ def test_send_staff_notification(self, mock_send_event, mocked_modulestore):
  """
  Test send_staff_notification function
  """
@@ -21,6 +22,8 @@ def test_send_staff_notification(self, mock_send_event):
  problem_id = 'problem_id'
  ora_name = 'ora_name'
 
+ mocked_modulestore.return_value = MagicMock()
+
  # Call the function
  send_staff_notification(course_id, problem_id, ora_name)
 
@@ -37,9 +40,10 @@ def test_send_staff_notification(self, mock_send_event):
  self.assertEqual(notification_data.app_name, "grading")
  self.assertEqual(notification_data.audience_filters['course_roles'], ['staff', 'instructor'])
 
+ @patch('openassessment.xblock.utils.notifications.modulestore')
  @patch('openassessment.xblock.utils.notifications.logger.error')
  @patch('openassessment.xblock.utils.notifications.COURSE_NOTIFICATION_REQUESTED.send_event')
- def test_send_staff_notification_error_logging(self, mock_send_event, mock_logger_error):
+ def test_send_staff_notification_error_logging(self, mock_send_event, mock_logger_error, mocked_modulestore):
  """
  Test send_staff_notification function when an exception is raised
  """
@@ -48,6 +52,8 @@ def test_send_staff_notification_error_logging(self, mock_send_event, mock_logge
  problem_id = 'problem_id'
  ora_name = 'ora_name'
 
+ mocked_modulestore.return_value = MagicMock()
+
  # Mock exception
  mock_exception = Exception('Test exception')
 

openassessment/xblock/utils/notifications.py

@@ -6,6 +6,7 @@
 from django.conf import settings
 from openedx_events.learning.signals import COURSE_NOTIFICATION_REQUESTED
 from openedx_events.learning.data import CourseNotificationData
+from openassessment.runtime_imports.functions import modulestore
 
 logger = logging.getLogger(__name__)
 
@@ -18,10 +19,12 @@ def send_staff_notification(course_id, problem_id, ora_name):
  audience_filters = {
  'course_roles': ['staff', 'instructor']
  }
+ course = modulestore().get_course(course_id)
  notification_data = CourseNotificationData(
  course_key=course_id,
  content_context={
- 'ora_name': ora_name
+ 'ora_name': ora_name,
+ 'course_name': course.display_name,
  },
  notification_type='ora_staff_notification',
  content_url=f"{getattr(settings, 'ORA_GRADING_MICROFRONTEND_URL', '')}/{problem_id}",

package.json

@@ -1,6 +1,6 @@
 {
  "name": "edx-ora2",
- "version": "6.11.1",
+ "version": "6.11.2",
  "repository": "https://github.com/openedx/edx-ora2.git",
  "dependencies": {
  "@edx/frontend-build": "8.0.6",