matrix-org / matrix-org/complement
`TestRoomDeleteAlias` tests optional behaviour but fails if it is not performed
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 99
- Forks
- 72
- Ø Merge
- 4 T. 1 Std.
- Gemergte PRs (30 T.)
- 8
Beschreibung
https://github.com/matrix-org/complement/blob/1a2baac5dbf45ead46409793fe625b3c253eb4f6/tests/csapi/apidoc_room_alias_test.go#L277-L308
The test `Can delete canonical alias` *requires* that servers update the `m.room.canonical_alias` event when an associated alias is deleted via `DELETE /_matrix/client/v3/directory/room/{roomAlias}`, but this behaviour is optional, and not even a SHOULD in the spec:
> Note: Servers **may** choose to update the `alt_aliases` for the `m.room.canonical_alias` state event in the room when an alias is removed. Servers which choose to update the canonical alias event are recommended to, in addition to their other relevant permission checks, delete the alias and return a successful response even if the user does not have permission to update the `m.room.canonical_alias event`.
This means the test will fail on any server that does not choose to implement this behaviour. The test should simply pass or at least be skipped if the canonical alias event update does not come down sync.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in tests/csapi/apidoc_room_alias_test.go beim Test `Can delete canonical alias`, Zeilen 277-308. Überprüfe, wie der Test das `m.room.canonical_alias`-Ereignis nach dem Löschen eines Alias über sync beobachtet. Als erledigt gilt die Änderung, wenn der Test besteht oder übersprungen wird, falls das optionale Ereignis-Update fehlt, und das Update weiterhin prüft, wenn es bereitgestellt wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- go
- Bereich
- testing
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Ruhig
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 78/100