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

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#957 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 2 jours

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
65/100
Type d'issue
Bug
Clarté
Plutôt claire
Activité
Active
Stack technique
go, json
Domaine
backend-api-design, testing

Piste de recherche

Commencez par tests/csapi/power_levels_test.go, près du test mentionné, et examinez comment il vérifie la réponse des niveaux de pouvoir. L’échec montre que le sérialiseur peut omettre des clés ayant une valeur par défaut, comme ban ; déterminez comment le test doit gérer ces valeurs par défaut omises, puis exécutez le test des niveaux de pouvoir. Le travail est terminé lorsque le test réussit avec la réponse sérialisée indiquée.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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

Langage dominant
Go
Étoiles
102
Forks
73
Merge moyen
3 j 10 h
PR mergées (30 j)
6

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de matrix-org/complement

Toutes les issues de matrix-org/complement

Issues similaires

Plus d'issues Go

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.