DGNum/gestioCOF
13
2
Fork 1
Code Issues 77 Pull requests 14 Packages Projects Releases 12 Wiki Activity

petitcours.tests -- Add tests for some views #629

Merged
delobell merged 0 commits from aureplop/cof-tests_petitcours into master 2018-11-25 00:41:52 +01:00
Conversation 40 Commits - Files changed -
delobell commented 2018-10-27 13:51:00 +02:00 (Migrated from git.eleves.ens.fr)
Copy link
  • demande_list
  • demande_detail
  • inscription
  • traitement : tester les perms nécessaires et 1 cas simple (1 dispo + 1 demande → 1 attribution)
  • retraitement : tester les perms nécessaires seulement
  • demande
  • demande_raw

Issue liée : #204
Coverage gestioncof/petits_cours_views.py : 12% → 63.23%

- [x] demande_list - [x] demande_detail - [x] inscription - [x] traitement : tester les perms nécessaires et 1 cas simple (1 dispo + 1 demande → 1 attribution) - [x] retraitement : tester les perms nécessaires seulement - [x] demande - [x] demande_raw Issue liée : #204 Coverage ``gestioncof/petits_cours_views.py`` : 12% → 63.23%
delobell commented 2018-10-27 13:51:30 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in issue #204

mentioned in issue #204
delobell commented 2018-10-27 14:00:54 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed the description

changed the description
delobell commented 2018-10-27 14:01:14 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed the description

changed the description
delobell commented 2018-10-27 14:25:38 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

  • 005dd8f3 - wip -- petitcours.tests -- Add tests for (re)traitement view

Compare with previous version

added 1 commit <ul><li>005dd8f3 - wip -- petitcours.tests -- Add tests for (re)traitement view</li></ul> [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/325/diffs?diff_id=1017&start_sha=808d665158b45b192291b28cf5f53bee83ca4abe)
delobell commented 2018-10-27 14:25:40 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

marked as a Work In Progress from 005dd8f3a951bdd419751783ebb0546a89c7fa81

marked as a **Work In Progress** from 005dd8f3a951bdd419751783ebb0546a89c7fa81
delobell commented 2018-10-27 18:48:55 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

marked the checklist item traitement : tester les perms nécessaires et 1 cas simple (1 dispo + 1 demande → 1 attribution) as completed

marked the checklist item **traitement : tester les perms nécessaires et 1 cas simple (1 dispo + 1 demande → 1 attribution)** as completed
delobell commented 2018-10-27 18:48:56 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

marked the checklist item retraitement : tester les perms nécessaires seulement as completed

marked the checklist item **retraitement : tester les perms nécessaires seulement** as completed
delobell commented 2018-10-27 18:49:00 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

  • 22eca93c - petitcours.tests -- Add tests for inscription and (re)traitement views

Compare with previous version

added 1 commit <ul><li>22eca93c - petitcours.tests -- Add tests for inscription and (re)traitement views</li></ul> [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/325/diffs?diff_id=1018&start_sha=005dd8f3a951bdd419751783ebb0546a89c7fa81)
delobell commented 2018-10-27 22:04:49 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

marked the checklist item demande as completed

marked the checklist item **demande** as completed
delobell commented 2018-10-27 22:04:50 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

marked the checklist item demande_raw as completed

marked the checklist item **demande_raw** as completed
delobell commented 2018-10-27 22:04:51 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 4 commits

  • 25dfe2f4 - petitcours.tests -- Add PCAbility, PCDemande and PCSubject factories
  • 1a5bbf32 - petitcours.tests -- Add tests for demandes list and details views
  • 3d27dc9a - petitcours.tests -- Add tests for inscription and (re)traitement views
  • 539e1edc - petitcours.tests -- Add tests for demande (raw) views

Compare with previous version

added 4 commits <ul><li>25dfe2f4 - petitcours.tests -- Add PCAbility, PCDemande and PCSubject factories</li><li>1a5bbf32 - petitcours.tests -- Add tests for demandes list and details views</li><li>3d27dc9a - petitcours.tests -- Add tests for inscription and (re)traitement views</li><li>539e1edc - petitcours.tests -- Add tests for demande (raw) views</li></ul> [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/325/diffs?diff_id=1020&start_sha=22eca93cedcb706ae8aa917ca7af4cb47dbca1c3)
delobell commented 2018-10-27 22:08:57 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

unmarked as a Work In Progress

unmarked as a **Work In Progress**
delobell commented 2018-10-27 22:08:57 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed the description

changed the description
delobell commented 2018-10-27 22:08:57 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added ~59 and removed ~30 labels

added ~59 and removed ~30 labels
delobell commented 2018-10-27 22:19:23 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

  • 5a1fc7ae - petitcours.tests -- Add tests for demande (raw) views

Compare with previous version

added 1 commit <ul><li>5a1fc7ae - petitcours.tests -- Add tests for demande (raw) views</li></ul> [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/325/diffs?diff_id=1021&start_sha=539e1edc5f40099585589bff9fdd98a59055aabc)
delobell commented 2018-10-27 22:30:06 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

  • 0fe63d3e - petitcours.tests -- Add tests for demande (raw) views

Compare with previous version

added 1 commit <ul><li>0fe63d3e - petitcours.tests -- Add tests for demande (raw) views</li></ul> [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/325/diffs?diff_id=1022&start_sha=5a1fc7ae5c004fae7c1f1210ea1d31201002a815)
lstephan commented 2018-11-12 21:44:48 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Peut-être bouger les tests dans un fichier à part ? Les vues des petits cours sont à part, et vu qu'il va falloir les réécrire soon (beaucoup), ça sera plus pratique...

Peut-être bouger les tests dans un fichier à part ? Les vues des petits cours sont à part, et vu qu'il va falloir les réécrire soon (beaucoup), ça sera plus pratique...
lstephan commented 2018-11-12 21:53:58 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Il vaudrait peut-être mieux coder ça pour que ça référence DemandeListView.paginate_by directement, pour éviter que si on modifie le paramètre ça fasse (faussement) rater les tests...

Il vaudrait peut-être mieux coder ça pour que ça référence `DemandeListView.paginate_by` directement, pour éviter que si on modifie le paramètre ça fasse (faussement) rater les tests...
lstephan commented 2018-11-12 21:55:13 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Idem

Idem
lstephan commented 2018-11-12 22:00:06 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

@property ne marche pas là dessus ?

`@property` ne marche pas là dessus ?
lstephan commented 2018-11-12 22:03:54 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Il n'y a pas moyen d'automatiser ce bordel ? Un truc de ce genre : https://schinckel.net/2016/04/30/%28directly%29-testing-django-formsets/

Il n'y a pas moyen d'automatiser ce bordel ? Un truc de ce genre : https://schinckel.net/2016/04/30/%28directly%29-testing-django-formsets/
lstephan commented 2018-11-12 22:58:41 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

On peut la faire hériter de PetitCoursDemandeViewTestCase, ça économise du code.

On peut la faire hériter de `PetitCoursDemandeViewTestCase`, ça économise du code.
bclement commented 2018-11-24 22:58:59 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Les tests c'est bien le seul endroit où il vaut mieux préférer le copier-coller pour la clareté plutôt que d'éliminer de la réutilisation de code :)

Les tests c'est bien le seul endroit où il vaut mieux préférer le copier-coller pour la clareté plutôt que d'éliminer de la réutilisation de code :)
bclement commented 2018-11-24 23:01:02 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Je dirais que c'est OK comme ça et que l'automatisation peut se faire à postériori si nécessaire (la technique décrite n'est pas moins fragile, elle diminue juste un peu le boilerplate).

Je dirais que c'est OK comme ça et que l'automatisation peut se faire à postériori si nécessaire (la technique décrite n'est pas moins fragile, elle diminue juste un peu le boilerplate).
bclement commented 2018-11-24 23:02:44 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Je ne suis pas convaincu de l'utilité de ce test. On n'a pas de code dans PetitCoursDemandeListView, donc on est juste en train de tester le code de Django ici.

Je ne suis pas convaincu de l'utilité de ce test. On n'a pas de code dans `PetitCoursDemandeListView`, donc on est juste en train de tester le code de Django ici.
bclement commented 2018-11-24 23:47:49 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

@lstephan est-ce qu'on peut merger ça si je fais les changements discutés ci-dessus (déplacement des tests dans un fichier à part, suppression du test de pagination, @property) ? Ça permettrait de commencer à travailler sur #208 plus sereinement.

(Je suis aussi en train de profiter de l'occasion pour bouger tous les petits cours dans gestioncof/petitscours, à défaut d'en faire une app à part parcequ'il faudrait une migration)

@lstephan est-ce qu'on peut merger ça si je fais les changements discutés ci-dessus (déplacement des tests dans un fichier à part, suppression du test de pagination, @property) ? Ça permettrait de commencer à travailler sur #208 plus sereinement. (Je suis aussi en train de profiter de l'occasion pour bouger tous les petits cours dans `gestioncof/petitscours`, à défaut d'en faire une app à part parcequ'il faudrait une migration)
lstephan commented 2018-11-25 00:03:22 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Ça me va, oui !

Ça me va, oui !
bclement commented 2018-11-25 00:23:53 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

changed this line in version 7 of the diff

changed this line in [version 7 of the diff](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/325/diffs?diff_id=1038&start_sha=0fe63d3eae226f643d0935a9ebb7b2105feb21b7#08c2714d982902ca4785d09e96b54f7573e2ce1a_1273_1243)
bclement commented 2018-11-25 00:23:54 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

changed this line in version 7 of the diff

changed this line in [version 7 of the diff](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/325/diffs?diff_id=1038&start_sha=0fe63d3eae226f643d0935a9ebb7b2105feb21b7#08c2714d982902ca4785d09e96b54f7573e2ce1a_1279_1243)
bclement commented 2018-11-25 00:23:54 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

changed this line in version 7 of the diff

changed this line in [version 7 of the diff](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/325/diffs?diff_id=1038&start_sha=0fe63d3eae226f643d0935a9ebb7b2105feb21b7#08c2714d982902ca4785d09e96b54f7573e2ce1a_1345_1243)
bclement commented 2018-11-25 00:23:54 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

changed this line in version 7 of the diff

changed this line in [version 7 of the diff](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/325/diffs?diff_id=1038&start_sha=0fe63d3eae226f643d0935a9ebb7b2105feb21b7#08c2714d982902ca4785d09e96b54f7573e2ce1a_1350_1243)
bclement commented 2018-11-25 00:23:55 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

changed this line in version 7 of the diff

changed this line in [version 7 of the diff](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/325/diffs?diff_id=1038&start_sha=0fe63d3eae226f643d0935a9ebb7b2105feb21b7#08c2714d982902ca4785d09e96b54f7573e2ce1a_1552_1243)
bclement commented 2018-11-25 00:23:56 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

changed this line in version 7 of the diff

changed this line in [version 7 of the diff](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/325/diffs?diff_id=1038&start_sha=0fe63d3eae226f643d0935a9ebb7b2105feb21b7#08c2714d982902ca4785d09e96b54f7573e2ce1a_1272_1243)
bclement commented 2018-11-25 00:23:56 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

added 15 commits

  • 0fe63d3e...6524f89e - 12 commits from branch master
  • d48cb3aa - petitcours.tests -- Deplace les tests dans leur propre fichier
  • a2116bf2 - Merge branch 'master' into aureplop/cof-tests_petitcours
  • 4e34583e - black

Compare with previous version

added 15 commits <ul><li>0fe63d3e...6524f89e - 12 commits from branch <code>master</code></li><li>d48cb3aa - petitcours.tests -- Deplace les tests dans leur propre fichier</li><li>a2116bf2 - Merge branch &#39;master&#39; into aureplop/cof-tests_petitcours</li><li>4e34583e - black</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/325/diffs?diff_id=1038&start_sha=0fe63d3eae226f643d0935a9ebb7b2105feb21b7)
bclement commented 2018-11-25 00:24:35 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Done! @lstephan je te laisse merge si ça te va ?

Done! @lstephan je te laisse merge si ça te va ?
bclement commented 2018-11-25 00:35:18 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Les tests qui échhouent sont des tests dans kfet, avec l'erreur "redis.exceptions.DataError: Invalid input of type: 'CacheKey'. Convert to a byte, string or number first.". Ce n'est donc pas dû à la PR, à priori.

Les tests qui échhouent sont des tests dans kfet, avec l'erreur "redis.exceptions.DataError: Invalid input of type: 'CacheKey'. Convert to a byte, string or number first.". Ce n'est donc pas dû à la PR, à priori.
lstephan commented 2018-11-25 00:39:26 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Effectivement, il faut juste bump la version de django-redis-cache dans requirements.txt, apparemment.

Effectivement, il faut juste bump la version de `django-redis-cache` dans `requirements.txt`, apparemment.
lstephan commented 2018-11-25 00:40:38 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

resolved all discussions

resolved all discussions
lstephan commented 2018-11-25 00:41:52 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

merged

merged
lstephan commented 2018-11-25 00:41:53 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in commit 4ae0b3c5f0

mentioned in commit 4ae0b3c5f0f7857f655697a748dc718cf3aaf594
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
devtype -- backend

Au programme : du Python, du Django...

  
devtype -- docs

   
devtype -- frontend

   
devtype -- user interface

Au programme : templating, JS, CSS...

  
difficulty -- easy

   
difficulty -- hard

Difficile et/ou risqué

  
difficulty -- normal

   
Doing

   
domain -- bda

Concerne l'application BdA

  
domain -- bds

   
domain -- cof

Concerne l'application COF

  
domain -- core

Dépendances externes, CI, linting...

  
domain -- kfet

Concerne l'application K-Fêt

  
Good first issue

   
priority -- high

Urgent quoi

  
priority -- low

   
priority -- medium

   
priority -- staff-wanted

L'équipe dirigeante du domaine en question souhaite cette fonctionnalité.

  
status -- development

En cours de développement

  
status -- discussion

Avant de coder, c'est bien de discuter

  
status -- need review

En attente de code review

  
status -- production

L'ajout/la modification est présente en production

  
status -- ready to merge

Plus qu'à merger !

  
status -- todo

Le développement devrait bientôt commencer

  
To Do

   
type -- bug

Signalement ou correction de bug

  
type -- hygiene

Tout ce qui amène à du code plus propre et un repo mieux organisé.

  
type -- improvement

Ajout/Demande d'évolution mineure d'une fonctionnalité existante

  
type -- new feature

Ajout/Demande d'une nouvelle fonctionnalité

No labels
devtype -- backend
devtype -- docs
devtype -- frontend
devtype -- user interface
difficulty -- easy
difficulty -- hard
difficulty -- normal
Doing
domain -- bda
domain -- bds
domain -- cof
domain -- core
domain -- kfet
Good first issue
priority -- high
priority -- low
priority -- medium
priority -- staff-wanted
status -- development
status -- discussion
status -- need review
status -- production
status -- ready to merge
status -- todo
To Do
type -- bug
type -- hygiene
type -- improvement
type -- new feature
Milestone
Clear milestone
No items
No milestone
Coverage - Views
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: DGNum/gestioCOF#629
No description provided.
Delete branch "aureplop/cof-tests_petitcours"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?