feat: LP updated event

Co-Authored-By: Claude <noreply@anthropic.com>

Author: bradenmacdonald

src/openedx_content/applets/publishing/api.py

@@ -157,6 +157,21 @@ def update_learning_package(
     lp.updated = updated
 
     lp.save()
+
+    # Emit LEARNING_PACKAGE_UPDATED once the transaction commits. Note: we only
+    # reach this point if at least one of key/title/description/updated was
+    # passed in (the early-return above handles the no-op case), so the update
+    # really did touch the row.
+    lp_id = lp.id
+    lp_title = lp.title
+
+    def send_event():
+        signals.LEARNING_PACKAGE_UPDATED.send_event(
+            learning_package=signals.LearningPackageEventData(id=lp_id, title=lp_title),
+        )
+
+    on_commit(send_event)
+
     return lp
 
 

src/openedx_content/applets/publishing/signals.py

@@ -18,6 +18,7 @@
     "PublishLogEventData",
     # All events:
     "LEARNING_PACKAGE_CREATED",
+    "LEARNING_PACKAGE_UPDATED",
     "LEARNING_PACKAGE_DELETED",
     "LEARNING_PACKAGE_ENTITIES_CHANGED",
     "LEARNING_PACKAGE_ENTITIES_PUBLISHED",
@@ -106,6 +107,35 @@ class PublishLogEventData:
 """
 
 
+LEARNING_PACKAGE_UPDATED = OpenEdxPublicSignal(
+    event_type="org.openedx.content.publishing.lp_updated.v1",
+    data={
+        "learning_package": LearningPackageEventData,
+    },
+)
+"""
+A ``LearningPackage``'s own metadata (key, title, and/or description) has been
+changed.
+
+This is emitted only when the ``update_learning_package`` API is called, with at
+least one field change that actually modifies the row.
+
+This event covers changes to the ``LearningPackage`` row itself (its ``key``,
+``title``, and ``description``). Changes to the content inside the package
+(entities, versions, drafts, publishes) are covered by
+``LEARNING_PACKAGE_ENTITIES_CHANGED`` and ``LEARNING_PACKAGE_ENTITIES_PUBLISHED``
+instead.
+
+The ``learning_package`` payload reflects the ``id`` and the post-update
+``title`` of the package.
+
+💾 This event is only emitted after the enclosing database transaction has
+been committed. If the transaction is rolled back, no event is emitted.
+
+⏳ This event is emitted synchronously.
+"""
+
+
 LEARNING_PACKAGE_DELETED = OpenEdxPublicSignal(
     event_type="org.openedx.content.publishing.lp_deleted.v1",
     data={

tests/openedx_content/applets/publishing/test_signals.py

@@ -73,6 +73,54 @@ def test_learning_package_created_aborted() -> None:
             api.create_learning_package(key="lp1", title="Test LP 📦")
 
 
+# LEARNING_PACKAGE_UPDATED
+
+
+def test_learning_package_updated() -> None:
+    """
+    Test that LEARNING_PACKAGE_UPDATED is emitted when
+    ``update_learning_package`` actually changes a field, and that the payload
+    reflects the post-update title.
+    """
+    learning_package = api.create_learning_package(key="lp1", title="Original Title")
+
+    with capture_events(signals=[api.signals.LEARNING_PACKAGE_UPDATED], expected_count=1) as captured:
+        api.update_learning_package(learning_package.id, title="New Title 📦")
+
+    event = captured[0]
+    assert event.signal is api.signals.LEARNING_PACKAGE_UPDATED
+    assert event.kwargs["learning_package"].id == learning_package.id
+    assert event.kwargs["learning_package"].title == "New Title 📦"
+
+
+def test_learning_package_updated_noop() -> None:
+    """
+    Test that LEARNING_PACKAGE_UPDATED is NOT emitted when
+    ``update_learning_package`` is called with no field changes (the early
+    return in the API means the row is never saved).
+    """
+    learning_package = api.create_learning_package(key="lp1", title="Test LP 📦")
+
+    with capture_events(signals=[api.signals.LEARNING_PACKAGE_UPDATED], expected_count=0):
+        api.update_learning_package(learning_package.id)
+
+
+def test_learning_package_updated_aborted() -> None:
+    """
+    Test that LEARNING_PACKAGE_UPDATED is NOT emitted when the transaction
+    that would have updated the ``LearningPackage`` is rolled back.
+    """
+    learning_package = api.create_learning_package(key="lp1", title="Original Title")
+
+    with capture_events(signals=[api.signals.LEARNING_PACKAGE_UPDATED], expected_count=0):
+        with abort_transaction():
+            api.update_learning_package(learning_package.id, title="Not going to stick")
+
+    # Confirm the title was not actually changed:
+    learning_package.refresh_from_db()
+    assert learning_package.title == "Original Title"
+
+
 # LEARNING_PACKAGE_DELETED