DGNum/poulpe

Fork 0

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

mpepin commented

2017-05-17 18:21:54 +02:00

(Migrated from git.eleves.ens.fr)

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)

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)

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)

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)

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)

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)

added 1 commit

a53a9249 - typo

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)

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)

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)

Habille. Effectivement ^^

Habille. Effectivement \^\^

mpepin commented

2017-05-18 18:35:44 +02:00

(Migrated from git.eleves.ens.fr)

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)

Promis ? :p

narmanli commented

2017-05-18 18:45:39 +02:00

(Migrated from git.eleves.ens.fr)

merged