Update search index after updating next_start_date (#2639)

Author: mbertrand

learning_resources/tasks.py

@@ -33,6 +33,7 @@
     html_to_markdown,
     load_course_blocklist,
     resource_unpublished_actions,
+    resource_upserted_actions,
 )
 from learning_resources_search.constants import COURSE_TYPE
 from learning_resources_search.exceptions import RetryError
@@ -79,6 +80,8 @@ def update_next_start_date_and_prices():
     resources = LearningResource.objects.filter(next_start_date__lt=timezone.now())
     for resource in resources:
         load_run_dependent_values(resource)
+        if resource.published:
+            resource_upserted_actions(resource, percolate=False)
     clear_search_cache()
     return len(resources)
 

learning_resources/tasks_test.py

@@ -475,17 +475,26 @@ def test_get_youtube_transcripts(mocker):
     )
 
 
-def test_update_next_start_date(mocker):
+@pytest.mark.parametrize("published", [True, False])
+def test_update_next_start_date(mocker, published):
     learning_resource = LearningResourceFactory.create(
-        next_start_date=(timezone.now() - timedelta(10))
+        next_start_date=(timezone.now() - timedelta(10)),
+        published=published,
     )
     LearningResourceFactory.create(next_start_date=(timezone.now() + timedelta(1)))
 
     mock_load_next_start_date = mocker.patch(
         "learning_resources.tasks.load_run_dependent_values"
     )
+    mock_upsert_index = mocker.patch(
+        "learning_resources.tasks.resource_upserted_actions"
+    )
     update_next_start_date_and_prices()
     mock_load_next_start_date.assert_called_once_with(learning_resource)
+    if published:
+        mock_upsert_index.assert_called_once_with(learning_resource, percolate=False)
+    else:
+        mock_upsert_index.assert_not_called()
 
 
 @pytest.mark.parametrize(