Produits Entr'ouvert » Hobo

la trace complète :

_________________________________________________________________________________ test_cook _________________________________________________________________________________

db = None, fake_notify = {}, monkeypatch = <_pytest.monkeypatch.MonkeyPatch object at 0x7fcc78c59390>

    def test_cook(db, fake_notify, monkeypatch):
        monkeypatch.setattr(ServiceBase, 'is_resolvable', lambda x: True)
        monkeypatch.setattr(ServiceBase, 'has_valid_certificate', lambda x: True)
>       call_command('cook', 'tests_schemas/recipe.json')

tests_schemas/test_cook.py:12: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
../../envs/publik-env/local/lib/python2.7/site-packages/django/core/management/__init__.py:131: in call_command
    return command.execute(*args, **defaults)
../../envs/publik-env/local/lib/python2.7/site-packages/django/core/management/base.py:330: in execute
    output = self.handle(*args, **options)
hobo/environment/management/commands/cook.py:62: in handle
    self.run_cook(recipe)
hobo/environment/management/commands/cook.py:86: in run_cook
    getattr(self, action.replace('-', '_'))(**action_args)
hobo/environment/management/commands/cook.py:205: in create_authentic
    return self.create_site(Authentic, url, title, slug, template_name, variables)
hobo/environment/management/commands/cook.py:177: in create_site
    obj.full_clean()
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

self = <Authentic: Authentic object>, exclude = ['last_operational_success_timestamp', 'last_operational_check_timestamp'], validate_unique = True

    def full_clean(self, exclude=None, validate_unique=True):
        """ 
        Calls clean_fields, clean, and validate_unique, on the model,
        and raises a ``ValidationError`` for any errors that occurred.
        """ 
        import pdb; pdb.set_trace()
>       errors = {}
E       ValidationError: {'last_operational_success_timestamp': [u'This field cannot be blank.'], 'last_operational_check_timestamp': [u'This field cannot be blank.']}

../../envs/publik-env/local/lib/python2.7/site-packages/django/db/models/base.py:1222: ValidationError

On a la convention d'éviter les regresion fix ou 'enhancement' dans les messages de commit (parce c'est toujours fix ou enhancement donc on ne le répète pas partout).

(il manque une migration)

Mais c'est pas terrible je trouve, parce qu'on teste dans le code des choses comme "if self.last_operational_check_timestamp is None:" et je me demande si le blank=True va pas permettre des choses imprévues.

En fait, se servir d'une validation de formulaire c'est sans doute pas la bonne méthode à utiliser ici...?

https://docs.djangoproject.com/en/1.11/ref/models/instances/#django.db.models.Model.full_clean :

* Model.full_clean(exclude=None, validate_unique=True)

The optional exclude argument can be used to provide a list of field names that can be excluded from validation and cleaning.

Ca serait sans doute mieux d'explicitement exclure les champs qui n'ont pas à être validés ?

Thomas Noël a écrit :

Est-ce qu'un blank=True modifie quelque chose en base?

Je sais pas, peut-être sur certains SGBD... dans le doute, faut ajouter une migration, je pense.

Ça ne modifie rien ça ne concerne que la validation via ModelForm à ma connaissance, contrairement à null=True, on peut simplement l'ajouter à la dernière migration qui a touché ces champs.

Il faut faire comme dit Thomas; dans la mesure où on utilise une API pas faite pour ça (ModelForm s'occupe je pense de passer dans exclude les champs non présent dans un formulaire) c'est à nous de l'appeler correctement.