tests: add missing required event fields in tests that build dicts directly

Some test fixtures construct event/PDU dicts inline rather than via
make_test_event — typically because they then run them through
add_hashes_and_signatures or otherwise depend on the exact dict shape.
Add the format-required fields (depth, hashes, auth_events,
prev_events, ...) inline in those cases so they keep working under
the strict (Rust-bound) constructor.

Author: erikjohnston

tests/federation/test_federation_base.py

@@ -36,6 +36,10 @@ def test_events_signed_by_banned_key_are_refused(self) -> None:
             "type": "m.room.message",
             "room_id": "!r:domain",
             "sender": f"@u:{self.hs.config.server.server_name}",
+            "auth_events": [],
+            "prev_events": [],
+            "depth": 1,
+            "hashes": {"sha256": ""},
             "signatures": {},
             "unsigned": {"age_ts": 1000000},
         }

tests/federation/test_federation_client.py

@@ -73,6 +73,7 @@ def test_get_room_state(self) -> None:
                 "content": {"creator": self.creator},
                 "prev_events": [],
                 "auth_events": [],
+                "depth": 1,
                 "origin_server_ts": 500,
             }
         )
@@ -85,6 +86,7 @@ def test_get_room_state(self) -> None:
                 "content": {"membership": "join"},
                 "prev_events": [],
                 "auth_events": [],
+                "depth": 2,
                 "origin_server_ts": 600,
             }
         )
@@ -97,6 +99,7 @@ def test_get_room_state(self) -> None:
                 "content": {},
                 "prev_events": [],
                 "auth_events": [],
+                "depth": 3,
                 "origin_server_ts": 700,
             }
         )
@@ -208,8 +211,8 @@ def _get_pdu_once(self) -> EventBase:
                 "content": {},
                 "prev_events": [],
                 "auth_events": [],
+                "depth": 3,
                 "origin_server_ts": 700,
-                "depth": 10,
             }
         )
 

tests/federation/transport/test_client.py

@@ -36,7 +36,16 @@ class SendJoinParserTestCase(TestCase):
     def test_two_writes(self) -> None:
         """Test that the parser can sensibly deserialise an input given in two slices."""
         parser = SendJoinParser(RoomVersions.V1, True)
+        common_event_fields = {
+            "sender": "@user:example.org",
+            "depth": 1,
+            "origin_server_ts": 1,
+            "hashes": {"sha256": ""},
+            "auth_events": [],
+            "prev_events": [],
+        }
         parent_event = {
+            **common_event_fields,
             "content": {
                 "see_room_version_spec": "The event format changes depending on the room version."
             },
@@ -45,6 +54,7 @@ def test_two_writes(self) -> None:
             "type": "m.room.minimal_pdu",
         }
         state = {
+            **common_event_fields,
             "content": {
                 "see_room_version_spec": "The event format changes depending on the room version."
             },

tests/rest/client/test_third_party_rules.py

@@ -532,6 +532,7 @@ def _send_event_over_federation(self) -> None:
                     "origin_server_ts": self.clock.time_msec(),
                     "prev_events": [],
                     "auth_events": [],
+                    "hashes": {"sha256": ""},
                     "signatures": {},
                     "unsigned": {},
                 }

tests/state/test_v21.py

@@ -495,6 +495,7 @@ def create_event(
             "prev_events": prev_events,
             "auth_events": auth_events,
             "origin_server_ts": monotonic_timestamp(),
+            "hashes": {"sha256": ""},
         }
         if event_type != EventTypes.Create:
             if room_id is None: