fixup! feat: Add the command to schedule the celery task for populating the index.

Author: farhaanbukhsh

openedx/core/djangoapps/content/search/management/commands/reindex_studio.py

@@ -9,15 +9,19 @@
 indexes LMS (published) courses in ElasticSearch.
 """
 
+import logging
 from django.core.management import BaseCommand, CommandError
 
 from ... import api
 from ...tasks import rebuild_index_incremental
 
 
+log = logging.getLogger(__name__)
+
+
 class Command(BaseCommand):
     """
-    Queue incremental population of the Meilisearch search index for Studio.
+    Add all course and library content to the Studio search index.
 
     This enqueues a Celery task that incrementally indexes all courses and
     libraries. Progress is tracked via IncrementalIndexCompleted, so the task
@@ -27,7 +31,7 @@ class Command(BaseCommand):
     (runs automatically on ./manage.py cms migrate).
     """
 
-    help = "Queue incremental population of the Studio Meilisearch search index via Celery."
+    help = "Add all course and library content to the Studio search index."
 
     def add_arguments(self, parser):
         # Removed flags — provide clear error messages for operators with old automation.
@@ -69,7 +73,7 @@ def handle(self, *args, **options):
             )
 
         if options["incremental"]:
-            raise CommandError(
+            log.warning(
                 "The --incremental flag has been removed. "
                 "Incremental population is now the default behavior of this command."
             )

openedx/core/djangoapps/content/search/tests/test_reindex_cmd.py

@@ -49,10 +49,16 @@ def test_init_flag_removed(self):
         with pytest.raises(CommandError, match="--init flag has been removed"):
             call_command("reindex_studio", "--init")
 
-    def test_incremental_flag_removed(self):
-        """Passing --incremental raises a clear error."""
-        with pytest.raises(CommandError, match="--incremental flag has been removed"):
-            call_command("reindex_studio", "--incremental")
+    @patch("openedx.core.djangoapps.content.search.tasks.rebuild_index_incremental.delay")
+    @patch("openedx.core.djangoapps.content.search.management.commands.reindex_studio.log")
+    def test_incremental_flag_accepted_with_warning(self, mock_log, mock_delay):
+        """Passing --incremental logs a warning but still enqueues the task."""
+        mock_delay.return_value = Mock(id="fake-task-id")
+
+        call_command("reindex_studio", "--incremental")
+
+        mock_log.warning.assert_called_once()
+        mock_delay.assert_called_once_with()
 
 
 @skip_unless_cms