hesperides icon indicating copy to clipboard operation
hesperides copied to clipboard

Revoir la notion de `simulate=true` dans `getFile(s)`

Open thomaslhostis opened this issue 6 years ago • 5 comments

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.

thomaslhostis avatar Oct 07 '19 12:10 thomaslhostis

À renomer en allowInexistingInstance=true ?

Lucas-C avatar Oct 07 '19 12:10 Lucas-C

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

thomaslhostis avatar Oct 07 '19 12:10 thomaslhostis

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

Lucas-C avatar Oct 07 '19 15:10 Lucas-C

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.

Lucas-C avatar Oct 08 '19 08:10 Lucas-C

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.

Lucas-C avatar Oct 15 '19 15:10 Lucas-C