Produits Entr'ouvert » Hobo

Je coince sur le test.

diff --git a/hobo/multitenant/settings_loaders.py b/hobo/multitenant/settings_loaders.py
index a355f21..1421c7e 100644
--- a/hobo/multitenant/settings_loaders.py
+++ b/hobo/multitenant/settings_loaders.py
@@ -387,7 +387,12 @@ class SettingsJSON(FileBaseSettingsLoader, SettingsDictUpdateMixin):

     def update_settings_from_path(self, tenant_settings, path):
         with open(path) as f:
-            self.handle_settings(tenant_settings, json.load(f))
+            try:
+                settings_json = json.load(f)
+            except ValueError:
+                pass
+            else:
+                self.handle_settings(tenant_settings, settings_json)

 class DictAdapter(dict):
diff --git a/tests_multitenant/test_settings.py b/tests_multitenant/test_settings.py
index 98ee238..8331d25 100644
--- a/tests_multitenant/test_settings.py
+++ b/tests_multitenant/test_settings.py
@@ -303,3 +303,20 @@ def test_tenant_json_settings_reload(tenants, settings, freezer):
         for tenant in tenants:
             with tenant_context(tenant):
                 assert django.conf.settings.EXTEND_ME == [1, 3]
+
+        # move 1 minute in the future
+        freezer.move_to(datetime.timedelta(seconds=60))
+
+        # update EXTEND_ME tenant value
+        for tenant in tenants:
+            with open(os.path.join(tenant.get_directory(), 'settings.json'), 'w') as fd:
+                json.dump({
+                    'EXTEND_ME': [99]
+                }, fd)
+
+        # move 1 minute in the future
+        freezer.move_to(datetime.timedelta(seconds=60))
+
+        for tenant in tenants:
+            with tenant_context(tenant):
+                assert django.conf.settings.EXTEND_ME == [99]

Le test ici n'est pas encore écrit jusqu'à tester un json mal formé, je chercher d'abord à changer la valeur de EXTEND_ME à 99 mais ça ne passe pas, je n'ai pas encore compris pourquoi.

            for tenant in tenants:
                with tenant_context(tenant):
>                   assert django.conf.settings.EXTEND_ME == [99]
E                   assert [1, 3] == [99]
E                     At index 0 diff: 1 != 99
E                     Left contains more items, first extra item: 3
E                     Use -v to get the full diff

Pas encore compris ce qui se passe, je me demande si c'est pas freezer.move_to(datetime.timedelta(seconds=60)) qui, en fait, ne fait pas son job ?... Un peu perdu je suis.

Je pense qu'ils ne faut pas "juste ne pas planter", il faut garder l'ancienne version des setttings le temps que ça remarche et balancer par ailleurs un gros mail d'erreur (mais alors là faut doser, parce qu'on peut facilement s'en prendre 1000)... parce qu'à chaque rechargement les settings sont nettoyés, et si on ne charge pas settings.json on a juste un truc vide (modulo tout ce que les autres settings-loader peuvent charger).

En gros faudrait chopper l'exception plus haut pendant le parcours des settings-loader, si un foire, on revient aux settings précédents on fait un logger.exception(.), et on note dans les settings qu'on l'a fait

settings.LOADER_ERROR = True

On se mangera juste autant de mails que de workers mais c'est mieux que 1000.

Pour ton souci de test, je dirai de tracer dans load_tenant_settings() je ne vois pas mieux.