hesperides
hesperides copied to clipboard
Revoir la notion de `simulate=true` dans `getFile(s)`
Dans la refonte nous avons reproduit le comportement de simulate=true en le traduisant par getModuleValuesIfInstanceDoesntExist mais l'idée de départ est légèrement différente.
Le comportement attendu est le suivant : si simulate=true, ne pas vérifier si l'instance existe.
Peut-être que ce n'est qu'une question de formulation dans l'implémentation mais il me semble qu'il manque des tests fonctionnels.
À renomer en allowInexistingInstance=true ?
Vu ensemble => 2 solutions possibles :
- Modifier le request param mais cela impacte les utilisateurs
- Conserver le request param mais renommer la variable et documenter (au niveau du Swagger)
Je vote pour la deuxième solution
Ok pour la 2e, le ratio bénéfice / "temps à y consacrer" est trop faible pour la 1ère, même si c'est la plus propre
Au passage, documenter dans Swagger le fait que instance_name peut avoir n'importe quelle valeur (qui sera ignorée) si la plateforme n'a PAS d'instance.
RAF:
- corriger le fonctionnement du code maintenant qu'on a compris le sens véritable de ce paramètre
- renommer la variable
- documenter le rpole de ce paramètre dans Swagger
- documenter dans Swagger le fait que instance_name peut avoir n'importe quelle valeur (qui sera ignorée) si la plateforme n'a PAS d'instance.