DGNum/gestioCOF

Fork 1

cof -- Add tests for registration views #591

Merged

delobell merged 0 commits from aureplop/cof-tests_registration into master

2018-10-06 11:25:47 +02:00

delobell commented

2018-01-20 17:07:49 +01:00

(Migrated from git.eleves.ens.fr)

Corrections / améliorations :

ajout d'un nom d'url pour la vue d'autocomplete des inscriptions au COF,
fix de noms d'urls dans un template,
la commande syncmails respecte l'options verbosity (-v).

Corrections / améliorations : - ajout d'un nom d'url pour la vue d'autocomplete des inscriptions au COF, - fix de noms d'urls dans un template, - la commande ``syncmails`` respecte l'options ``verbosity`` (``-v``).

mpepin commented

2018-04-07 13:21:01 +02:00

(Migrated from git.eleves.ens.fr)

added 52 commits

7bf2f73e...3463017d - 51 commits from branch master
75124548 - Merge branch 'master' into aureplop/cof-tests_registration

Compare with previous version

added 52 commits * 7bf2f73e...3463017d - 51 commits from branch `master` * 75124548 - Merge branch 'master' into aureplop/cof-tests_registration [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/287/diffs?diff_id=815&start_sha=7bf2f73e7218142540f0d045428a6fcc4d43aa50)

mpepin commented

2018-04-07 13:22:53 +02:00

(Migrated from git.eleves.ens.fr)

J'ai mis à jour avec master mais il y a un certain nombre de variables non utilisées qui suggèrent qu'il manque du code dans certains tests. Par exemple :

    def test_empty(self):
        r = self.client.get(self.t_urls[0])

        user_form = r.context['user_form']
        profile_form = r.context['profile_form']
        events_form = r.context['event_formset']
        clubs_form = r.context['clubs_form']
        # Fin de la fonction

@delobell c'est délibéré ou il manque effectivement du code ? En attendant je ne merge pas

J'ai mis à jour avec master mais il y a un certain nombre de variables non utilisées qui suggèrent qu'il manque du code dans certains tests. Par exemple : ```python def test_empty(self): r = self.client.get(self.t_urls[0]) user_form = r.context['user_form'] profile_form = r.context['profile_form'] events_form = r.context['event_formset'] clubs_form = r.context['clubs_form'] # Fin de la fonction ``` @delobell c'est délibéré ou il manque effectivement du code ? En attendant je ne merge pas

delobell commented

2018-09-30 12:57:49 +02:00

(Migrated from git.eleves.ens.fr)

added 79 commits

75124548...f297a1a0 - 76 commits from branch master
44e5387f - cof.tests -- Really check initial of built form
064c2390 - cof.tests -- Address flake8 concerns
10f4bd02 - Merge branch 'master' into aureplop/cof-tests_registration

Compare with previous version

added 79 commits * 75124548...f297a1a0 - 76 commits from branch `master` * 44e5387f - cof.tests -- Really check initial of built form * 064c2390 - cof.tests -- Address flake8 concerns * 10f4bd02 - Merge branch 'master' into aureplop/cof-tests_registration [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/287/diffs?diff_id=925&start_sha=75124548259a76f48d494597d710132f49919364)

delobell commented

2018-09-30 12:59:48 +02:00

(Migrated from git.eleves.ens.fr)

@mpepin sûrement que c'était en mode : complétez moi au besoin
J'ai remplacé par des assertIn(..., r.context)

Au passage, j'ai nettoyé les checks d'initial qui m'ont choqué. Je testais une fonction de l'API des formulaires, plutôt que de tester le résultat du formulaire...

J'ai aussi merge master. Mergeons et itérons ? (si les tests passent)

@mpepin sûrement que c'était en mode : complétez moi au besoin J'ai remplacé par des `assertIn(..., r.context)` Au passage, j'ai nettoyé les checks d'`initial` qui m'ont choqué. Je testais une fonction de l'API des formulaires, plutôt que de tester le résultat du formulaire... J'ai aussi merge master. Mergeons et itérons ? (si les tests passent)

delobell commented

2018-09-30 14:45:44 +02:00

(Migrated from git.eleves.ens.fr)

added ~60 ~59 ~28 labels

delobell commented

2018-10-01 14:57:02 +02:00

(Migrated from git.eleves.ens.fr)

changed milestone to %2

delobell commented

2018-10-01 15:07:39 +02:00

(Migrated from git.eleves.ens.fr)

On passe à de 67% à 85% de coverage sur gestioncof/views.py 👍

On passe à de 67% à 85% de coverage sur `gestioncof/views.py` :thumbsup:

mpepin commented

2018-10-04 23:41:10 +02:00

(Migrated from git.eleves.ens.fr)

pourquoi _minimal_data est une méthode et pas un attribut ?

pourquoi `_minimal_data` est une méthode et pas un attribut ?

mpepin commented

2018-10-06 11:25:28 +02:00

(Migrated from git.eleves.ens.fr)

bon osef

mpepin commented

2018-10-06 11:25:29 +02:00

(Migrated from git.eleves.ens.fr)

resolved all discussions

mpepin commented

2018-10-06 11:25:47 +02:00

(Migrated from git.eleves.ens.fr)

merged

mpepin commented

2018-10-06 11:25:47 +02:00

(Migrated from git.eleves.ens.fr)

mentioned in commit b39b6d6bb3

mentioned in commit b39b6d6bb3fc01e61f49f720ec653613f11a8cb9

delobell commented

2018-10-06 13:20:02 +02:00

(Migrated from git.eleves.ens.fr)

whoopsi, j'avais oublié ton message pendant que je me battais avec la CI
oui non y'a pas de rien pour ce cas, dans d'autres tests ça sert pour récupérer les pk d'objets créés dans le setup, mais là y'avait pas lieu...

whoopsi, j'avais oublié ton message pendant que je me battais avec la CI oui non y'a pas de rien pour ce cas, dans d'autres tests ça sert pour récupérer les pk d'objets créés dans le setup, mais là y'avait pas lieu...

No reviewers

No milestone

No project

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#591

No description provided.