kfet.tests -- Deterministic responses for cancel_operation #641

Merged

delobell merged 0 commits from aureplop/kfet-tests_deterministic into master

2019-01-07 14:12:43 +01:00

delobell commented

2019-01-06 10:34:52 +01:00

(Migrated from git.eleves.ens.fr)

Des fois les tests plantent parce que des listes ne sont pas ordonnées répétibles.

FAIL: test_partial_0 (kfet.tests.test_views.KPsulCancelOperationsViewTests)
----------------------------------------------------------------------
AssertionError: {'errors': {}, 'warnings': {'already_canceled': [19]}, 'canceled': [18, 16]} != {'errors': {}, 'warnings': {'already_canceled': [19]}, 'canceled': [16, 18]}
- {'canceled': [18, 16], 'errors': {}, 'warnings': {'already_canceled': [19]}}
?                 ----
+ {'canceled': [16, 18], 'errors': {}, 'warnings': {'already_canceled': [19]}}
?                ++++

Des fois les tests plantent parce que des listes ne sont pas ordonnées répétibles. ```diff FAIL: test_partial_0 (kfet.tests.test_views.KPsulCancelOperationsViewTests) ---------------------------------------------------------------------- AssertionError: {'errors': {}, 'warnings': {'already_canceled': [19]}, 'canceled': [18, 16]} != {'errors': {}, 'warnings': {'already_canceled': [19]}, 'canceled': [16, 18]} - {'canceled': [18, 16], 'errors': {}, 'warnings': {'already_canceled': [19]}} ? ---- + {'canceled': [16, 18], 'errors': {}, 'warnings': {'already_canceled': [19]}} ? ++++ ```

lstephan commented

2019-01-06 13:07:03 +01:00

(Migrated from git.eleves.ens.fr)

Je sais bien que ce que tu rajoutes ne change vraiment quasiment rien au temps d'exécution de cancel_operations, mais par principe il ne vaudrait pas mieux changer le test si la fonction marche bien et que le test merde ? Genre convertir en set, ou un truc dans le genre ?

Je sais bien que ce que tu rajoutes ne change vraiment quasiment rien au temps d'exécution de `cancel_operations`, mais par principe il ne vaudrait pas mieux changer le test si la fonction marche bien et que le test merde ? Genre convertir en `set`, ou un truc dans le genre ?

delobell commented

2019-01-06 13:31:44 +01:00

(Migrated from git.eleves.ens.fr)

Les deux étaient possibles. J'ai trouvé plus simple le coup choisi, suis pas allé chercher plus loin.

Avant de merge, je vais ajouter un petit truc pour les tests qui pètent 1/50 côté kfet.open.

Les deux étaient possibles. J'ai trouvé plus simple le coup choisi, suis pas allé chercher plus loin. Avant de merge, je vais ajouter un petit truc pour les tests qui pètent 1/50 côté `kfet.open`.

delobell commented

2019-01-06 13:31:56 +01:00

(Migrated from git.eleves.ens.fr)

added ~30 and removed ~59 labels

delobell commented

2019-01-06 13:48:57 +01:00

(Migrated from git.eleves.ens.fr)

added 1 commit

805d9449 - kfet.tests -- More isolated tests for kfet.open

Compare with previous version

added 1 commit <ul><li>805d9449 - kfet.tests -- More isolated tests for kfet.open</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/337/diffs?diff_id=1065&start_sha=3494087193e716b2d0eff011c28073084fd9a230)

delobell commented

2019-01-06 13:49:17 +01:00

(Migrated from git.eleves.ens.fr)

added 9 commits

805d9449...7c1d1df1 - 7 commits from branch master
7ca01440 - kfet.tests -- Deterministic responses for cancel_operation
5d14fef0 - kfet.tests -- More isolated tests for kfet.open

Compare with previous version

added 9 commits <ul><li>805d9449...7c1d1df1 - 7 commits from branch <code>master</code></li><li>7ca01440 - kfet.tests -- Deterministic responses for cancel_operation</li><li>5d14fef0 - kfet.tests -- More isolated tests for kfet.open</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/337/diffs?diff_id=1066&start_sha=805d9449288a0685b39aec4b02808175cd65adb3)

delobell commented

2019-01-06 13:55:28 +01:00

(Migrated from git.eleves.ens.fr)

added ~59 and removed ~30 labels

lstephan commented

2019-01-07 14:12:43 +01:00

(Migrated from git.eleves.ens.fr)

merged

lstephan commented

2019-01-07 14:13:03 +01:00

(Migrated from git.eleves.ens.fr)

mentioned in commit 31223aaed9