thubrecht/poulpe
1
0
Fork 0
Code Issues 5 Pull requests 1 Projects Releases Packages Wiki Activity Actions

Add a test for the user creation view #17

Merged
mpepin merged 3 commits from Kerl/test_user_creation into master 2017-05-18 18:45:39 +02:00
Conversation 12 Commits 3 Files changed 1 +33 -2
mpepin commented 2017-05-17 18:21:54 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Si ce test avait existé, je n'aurais pas pu oublier la CREATE_USER_KEY dans la précédente MR ;)

Si ce test avait existé, je n'aurais pas pu oublier la `CREATE_USER_KEY` dans la précédente MR ;)
narmanli commented 2017-05-18 16:14:06 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Pourquoi pas juste un get(username=data["username"]) ?

Pourquoi pas juste un `get(username=data["username"])` ?
mpepin commented 2017-05-18 16:26:33 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Pour pouvoir utiliser values qui me permet d'être flemmard et de ne pas avoir à tester tous les champs à la main.

Pour pouvoir utiliser `values` qui me permet d'être flemmard et de ne pas avoir à tester tous les champs à la main.
narmanli commented 2017-05-18 16:36:31 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ah oui, effectivement. Du coup j'ai cherché un peu et visiblement il est aussi possible possible de mettre le get après le values, il semblerait :) Bref c'est du chipotage.

Ah oui, effectivement. Du coup j'ai cherché un peu et visiblement il est aussi possible possible de mettre le `get` après le `values`, il semblerait :) Bref c'est du chipotage.
mpepin commented 2017-05-18 17:41:55 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Non non c'est pas tellement du chipotage, ça rend le test plus robuste, je fais ça tout de suite

Non non c'est pas tellement du chipotage, ça rend le test plus robuste, je fais ça tout de suite
mpepin commented 2017-05-18 17:54:03 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

  • f105b17e - replace [0] by .get() for a more robust test

Compare with previous version

added 1 commit * f105b17e - replace `[0]` by `.get()` for a more robust test [Compare with previous version](https://git.eleves.ens.fr/cof-geek/GestionEvenementiel/merge_requests/12/diffs?diff_id=496&start_sha=27fc3560454ff5383fec5226e3d07106d2555a35)
mpepin commented 2017-05-18 18:15:25 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit * a53a9249 - typo [Compare with previous version](https://git.eleves.ens.fr/cof-geek/GestionEvenementiel/merge_requests/12/diffs?diff_id=497&start_sha=f105b17e3f8757c3dbd0c2314d677563825117d5)
narmanli commented 2017-05-18 18:19:24 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

C'est rigolo, python manage.py test user et python manage.py test event fonctionnent tous les deux mais python manage.py test échoue :/

C'est rigolo, `python manage.py test user` et `python manage.py test event` fonctionnent tous les deux mais `python manage.py test` échoue :/
mpepin commented 2017-05-18 18:21:16 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Je pense que test userS fail du coup

Je pense que `test userS` fail du coup
narmanli commented 2017-05-18 18:29:31 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Habille. Effectivement ^^

Habille. Effectivement \^\^
mpepin commented 2017-05-18 18:35:44 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Et c'est à cause de la debug toolbar, pour tester que le test marche bien, tu dois retirer ddt de INSTALLED_APPS.

Le problème de ddt est géré sur l'autre branche où on peut utiliser settings.common pour les tests

Et c'est à cause de la debug toolbar, pour tester que le test marche bien, tu dois retirer ddt de `INSTALLED_APPS`. Le problème de ddt est géré sur l'autre branche où on peut utiliser `settings.common` pour les tests
narmanli commented 2017-05-18 18:36:52 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Promis ? :p

Promis ? :p
narmanli commented 2017-05-18 18:45:39 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

merged

merged
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Doing

   
Doing

   
Good first issue

   
To Do
No labels
Doing
Doing
Good first issue
To Do
Milestone
Clear milestone
No items
No milestone
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: thubrecht/poulpe#17
No description provided.
Delete branch "Kerl/test_user_creation"

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?