From 4c35bfbda4f10c2a61d607ee209cbf081ea4f66c Mon Sep 17 00:00:00 2001 From: Jason Volk Date: Sat, 26 Jul 2025 06:42:31 +0000 Subject: [PATCH] Fix M_UNKNOWN error which should be M_BAD_ALIAS. Signed-off-by: Jason Volk --- src/api/client/state.rs | 2 +- tests/test_results/complement/test_results.jsonl | 10 +++++----- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/api/client/state.rs b/src/api/client/state.rs index 5cfa51f3..a825db2e 100644 --- a/src/api/client/state.rs +++ b/src/api/client/state.rs @@ -337,7 +337,7 @@ async fn allowed_to_send_state_event( .resolve_alias(&alias, None) .await .map_err(|e| { - err!(Request(Unknown("Failed resolving alias \"{alias}\": {e}"))) + err!(Request(BadAlias("Failed resolving alias \"{alias}\": {e}"))) })?; if alias_room_id != room_id { diff --git a/tests/test_results/complement/test_results.jsonl b/tests/test_results/complement/test_results.jsonl index 9f3b581a..7a7e67e2 100644 --- a/tests/test_results/complement/test_results.jsonl +++ b/tests/test_results/complement/test_results.jsonl @@ -475,17 +475,17 @@ {"Action":"pass","Test":"TestRoomAlias/Parallel/Only_room_members_can_list_aliases_of_a_room"} {"Action":"pass","Test":"TestRoomAlias/Parallel/PUT_/directory/room/:room_alias_creates_alias"} {"Action":"pass","Test":"TestRoomAlias/Parallel/Room_aliases_can_contain_Unicode"} -{"Action":"fail","Test":"TestRoomCanonicalAlias"} -{"Action":"fail","Test":"TestRoomCanonicalAlias/Parallel"} +{"Action":"pass","Test":"TestRoomCanonicalAlias"} +{"Action":"pass","Test":"TestRoomCanonicalAlias/Parallel"} {"Action":"pass","Test":"TestRoomCanonicalAlias/Parallel/m.room.canonical_alias_accepts_present_aliases"} {"Action":"pass","Test":"TestRoomCanonicalAlias/Parallel/m.room.canonical_alias_accepts_present_alt_aliases"} {"Action":"pass","Test":"TestRoomCanonicalAlias/Parallel/m.room.canonical_alias_rejects_alias_pointing_to_different_local_room"} {"Action":"pass","Test":"TestRoomCanonicalAlias/Parallel/m.room.canonical_alias_rejects_alt_alias_pointing_to_different_local_room"} {"Action":"pass","Test":"TestRoomCanonicalAlias/Parallel/m.room.canonical_alias_rejects_invalid_aliases"} {"Action":"pass","Test":"TestRoomCanonicalAlias/Parallel/m.room.canonical_alias_rejects_invalid_aliases#01"} -{"Action":"fail","Test":"TestRoomCanonicalAlias/Parallel/m.room.canonical_alias_rejects_missing_aliases"} -{"Action":"fail","Test":"TestRoomCanonicalAlias/Parallel/m.room.canonical_alias_rejects_missing_aliases#01"} -{"Action":"fail","Test":"TestRoomCanonicalAlias/Parallel/m.room.canonical_alias_setting_rejects_deleted_aliases"} +{"Action":"pass","Test":"TestRoomCanonicalAlias/Parallel/m.room.canonical_alias_rejects_missing_aliases"} +{"Action":"pass","Test":"TestRoomCanonicalAlias/Parallel/m.room.canonical_alias_rejects_missing_aliases#01"} +{"Action":"pass","Test":"TestRoomCanonicalAlias/Parallel/m.room.canonical_alias_setting_rejects_deleted_aliases"} {"Action":"pass","Test":"TestRoomCreate"} {"Action":"pass","Test":"TestRoomCreate/Parallel"} {"Action":"pass","Test":"TestRoomCreate/Parallel/Can_/sync_newly_created_room"}