`TestPowerLevels/GET_/rooms/:room_id/state/m.room.power_levels_can_fetch_levels` fails when default values are omitted

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#957 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 2 Tagen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
65/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
go, json
Bereich
backend-api-design, testing

Rechercherichtung

Beginne in tests/csapi/power_levels_test.go in der Nähe des referenzierten Tests und untersuche, wie er die Power-Level-Antwort prüft. Der Fehler zeigt, dass der Serializer Schlüssel mit Standardwerten wie ban auslassen kann. Finde heraus, wie der Test mit diesen ausgelassenen Standardwerten umgehen soll, und führe anschließend den Power-Level-Test aus. Fertig ist die Aufgabe, wenn der Test mit der gemeldeten serialisierten Antwort besteht.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

If the JSON serialiser omits keys with values matching their defaults, as ruma does via serde, the test fails because it cannot find the specified key, even though this has no functional impact:

  ❌ TestPowerLevels/GET_/rooms/:room_id/state/m.room.power_levels_can_fetch_levels (0s)
    power_levels_test.go:105: MatchJSONBytes key 'ban' missing with input = {
          "events": {
            "m.poll.response": 0,
            "m.room.encryption": 100,
            "m.room.history_visibility": 100,
            "m.room.power_levels": 100,
            "m.room.server_acl": 100,
            "m.room.tombstone": 150,
            "org.matrix.msc3381.poll.response": 0
          }
        }

It looks like I added a skip for this when getting Venator running with complement, but forgot to open an issue for it
https://github.com/matrix-org/complement/blob/7b38a63c9db3337292bee5887fc8533be3a57b9f/tests/csapi/power_levels_test.go#L90-L93

Vorherrschende Sprache
Go
Sterne
102
Forks
73
Ø Merge
3 T. 10 Std.
Gemergte PRs (30 T.)
6

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus matrix-org/complement

Alle Issues in matrix-org/complement

Ähnliche Issues

Weitere Issues zu Go

Neue Issues direkt in Ihr Postfach

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