trytond: Add keys in the server context to identify when a module is being installed inside a test [CUSTOM] #93
Conversation
…being installed inside a test [CUSTOM]
jmousset
left a comment
jmousset
left a comment
There was a problem hiding this comment.
il faudra penser à pick sur la 6.8-patches
| return {key: value} | ||
|
|
||
|
|
||
| TEST_CONTEXT = generate_context(value=True) |
There was a problem hiding this comment.
C'est fait exprès d'avoir un Boolean (d'autant plus à true, qui va faire qu'on aura un dict avec {random_key: True} ) ici ?
There was a problem hiding this comment.
oui, je l'ai fait comme ça pour récupérer la clé plus facilement avec:
if Transaction().context.get('random_key')
mais il est théoriquement possible d'ajouter une valeur random aussi, dans ce cas il faudra faire plutôt:
if Transaction().context.get('random_key') == RANDOM_KEY
en tout cas on ne devrait avoir cette clé dans le contexte que si on est dans un test, c'est pour ça que je ne lui ai pas donnée une valeur random aussi.
@jmousset t'en penses quoi? j'ajoute des valeurs random aussi aux clés générées dans le contexte ou un boolean suffit?
There was a problem hiding this comment.
@BeatriceSchuster la version actuelle me semble ok. l'utilisateur a le choix de l'usage.
| return {key: value} | ||
|
|
||
|
|
||
| TEST_CONTEXT = generate_context(value=True) |
There was a problem hiding this comment.
@BeatriceSchuster la version actuelle me semble ok. l'utilisateur a le choix de l'usage.
…being installed inside a test [CUSTOM] (#93) Fix #PJAZZ-1416
…talled inside a test [CUSTOM] (coopengo#93) Fix #PJAZZ-1416
…talled inside a test [CUSTOM] (coopengo#93) Fix #PJAZZ-1416
…talled inside a test [CUSTOM] (coopengo#93) Fix #PJAZZ-1416
…talled inside a test [CUSTOM] (coopengo#93) Fix #PJAZZ-1416
…talled inside a test [CUSTOM] (coopengo#93) Fix #PJAZZ-1416
3 participants
Fix #PJAZZ-1416
needed for https://github.com/coopengo/coog/pull/9320