fix: move course_app extraction after setting the app status

Author: marslanabdulrauf

cms/djangoapps/contentstore/rest_api/v0/tests/test_advanced_settings.py

@@ -8,6 +8,7 @@
 from django.test import override_settings
 from django.urls import reverse
 from milestones.tests.utils import MilestonesTestCaseMixin
+from rest_framework.exceptions import ValidationError
 
 from cms.djangoapps.contentstore.tests.utils import CourseTestCase
 
@@ -154,7 +155,7 @@ def test_patch_advanced_setting_with_exception(self, mock_set_course_app_status)
         Test that exceptions in set_course_app_status are caught and don't break the flow.
         """
         # Mock set_course_app_status to raise an exception
-        mock_set_course_app_status.side_effect = Exception("Course app error")
+        mock_set_course_app_status.side_effect = ValidationError("Course app error")
 
         data = {
             "show_calculator": {

openedx/core/djangoapps/course_apps/rest_api/v1/views.py

@@ -192,8 +192,8 @@ def patch(self, request: Request, course_id: str):
         if enabled is None:
             raise ValidationError({"enabled": "Must provide value for `enabled` field."})
 
-        course_app = CourseAppsPluginManager.get_plugin(app_id)
         is_enabled = set_course_app_status(course_key=course_key, app_id=app_id, enabled=enabled, request=request)
+        course_app = CourseAppsPluginManager.get_plugin(app_id)
         serializer = CourseAppSerializer(
             course_app,
             context={