DirectusApiManager 1.9.2 - Erreur de test

Salut @mbritto et @jmjoary !

Je viens de me remettre sur flutter et de mettre a jour DirectusApiManager (v. 1.9.2).

Lorsque je lance les tests, le "Creating an invalid user should throw" ne passe pas car la ligne 9 retourne un DirectusUser:

expect(() => DirectusUser({"id": "123-abc"}), throwsException);

Le commit 42915b2 du 4 Novembre 2024 precise:

User email field is now optional to better reflect what actually happens in Directus. When reading a user objecxt, directus will return the fields the current user has access, if the user doesn’t have access to the « email » field then the value will be null.

Changements étant:

- /// [_rawReceivedData] must contain at least an `"id"` and an `"email"` properties. Throws [Exception] if they are missing.
+ /// [_rawReceivedData] must contain at least an `"id"`. Throws [Exception] if it is missing.
+ DirectusUser(super.rawReceivedData);

Je ne pense pas que Maxime ait publie sur GitHub sans avoir lance les tests, alors je me demande si ce n’est pas un bug de mon cote, ou si je n’ai pas compris quelque-chose…

Cedric

2 « J'aime »

Salut Cédric,

je n’aime pas balancé mes petits copains, mais je pense que les tests n’ont pas été mis à jour suite à ce changement.

2 « J'aime »

Je me demandais aussi… je ne pouvais pas croire que le maitre du Dart se trompe ! :sweat_smile:

1 « J'aime »

oups :grin:
Je ne vois pas qui ça pourrait être :sweat_smile:

2 « J'aime »