Skip to content

Commit 3902b85

Browse files
fix: unsorted event payload causing flaky test
1 parent ff15eb2 commit 3902b85

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

  • src/openedx_content/applets/collections

src/openedx_content/applets/collections/api.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ def add_to_collection(
206206
collection.entities.add(*ids_to_add, through_defaults={"created_by_id": created_by})
207207
collection.modified = datetime.now(tz=timezone.utc)
208208
collection.save()
209-
_queue_change_event(collection, entities_added=list(set(ids_to_add) - existing_ids), user_id=created_by)
209+
_queue_change_event(collection, entities_added=sorted(list(set(ids_to_add) - existing_ids)), user_id=created_by)
210210

211211
return collection
212212

@@ -228,7 +228,7 @@ def remove_from_collection(
228228
collection = get_collection(learning_package_id, collection_code)
229229

230230
ids_to_remove = list(entities_qset.values_list("id", flat=True))
231-
entities_removed = list(collection.entities.filter(id__in=ids_to_remove).values_list("id", flat=True))
231+
entities_removed = sorted(list(collection.entities.filter(id__in=ids_to_remove).values_list("id", flat=True)))
232232
collection.entities.remove(*ids_to_remove)
233233
collection.modified = datetime.now(tz=timezone.utc)
234234
collection.save()

0 commit comments

Comments
 (0)