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

[thomaslhostis]

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.

[Lucas-C]

À renomer en allowInexistingInstance=true ?

[thomaslhostis]

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

[Lucas-C]

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]

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]

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.