Properly include ddt's urls using `include` #494

Merged

mpepin merged 0 commits from Kerl/1_11_urls into supportBDS

2017-03-26 15:41:14 +02:00

mpepin commented

2017-03-18 21:53:48 +01:00

(Migrated from git.eleves.ens.fr)

Django debug toolbar's urls were not included properly according to Django 1.11's standards. We now use the include function.

Django debug toolbar's urls were not included properly according to Django 1.11's standards. We now use the `include` function.

delobell commented

2017-03-25 10:41:11 +01:00

(Migrated from git.eleves.ens.fr)

C'est normal que settings.DEBUG est sauté ?

C'est normal que `settings.DEBUG` est sauté ?

mpepin commented

2017-03-25 14:20:43 +01:00

(Migrated from git.eleves.ens.fr)

Oui, on ne pouvait pas lancer les tests unitaires à cause de ça. Quand tu lances les tests il se met en DEBUG = False mais il charge quand même le fichier settings_dev.py (obviously), du coup ddt se retrouve dans les installed apps mais ne trouve pas ses urls et râle.

Le workaround qu'on a trouvé a été de remplacer le settings.DEBUG par 'debug_toolbar' in settings.INSTALLED_APPS.

Oui, on ne pouvait pas lancer les tests unitaires à cause de ça. Quand tu lances les tests il se met en `DEBUG = False` mais il charge quand même le fichier `settings_dev.py` (obviously), du coup ddt se retrouve dans les installed apps mais ne trouve pas ses urls et râle. Le workaround qu'on a trouvé a été de remplacer le `settings.DEBUG` par `'debug_toolbar' in settings.INSTALLED_APPS`.

delobell commented

2017-03-25 14:30:27 +01:00

(Migrated from git.eleves.ens.fr)

resolved all discussions

delobell commented

2017-03-26 15:41:14 +02:00

(Migrated from git.eleves.ens.fr)

merged

delobell commented

2017-03-26 15:41:14 +02:00

(Migrated from git.eleves.ens.fr)

mentioned in commit 95b96d470f