Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Commit f76b996

Browse files
committed
Clarify comment.
1 parent 1ee0dbb commit f76b996

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

tests/rest/client/test_relations.py

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -981,8 +981,14 @@ def test_background_update(self) -> None:
981981
[annotation_event_id_good, thread_event_id],
982982
)
983983

984-
def test_bundled_aggregations_with_filter(self):
985-
"""If unsigned is not a returned field, the bundled aggregations still get added."""
984+
def test_bundled_aggregations_with_filter(self) -> None:
985+
"""
986+
If "unsigned" is an omitted field (due to filtering), adding the bundled
987+
aggregations should not break.
988+
989+
Note that the spec allows for a server to return additional fields beyond
990+
what is specified.
991+
"""
986992
self._send_relation(RelationTypes.ANNOTATION, "m.reaction", "a")
987993

988994
# Note that the sync filter does not include "unsigned" as a field.

0 commit comments

Comments
 (0)