matrix-org / matrix-org/complement

`TestRoomDeleteAlias` tests optional behaviour but fails if it is not performed

Offen Anfängerfreundlich
#900 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

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

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Ö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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.