paulocheque / django-dynamic-fixture / 159

0%

Pull #61

travis-ci

Don't swallow exceptions when saving polymorphic models.

If an exception is raised in the `save()` method of a polymorphic model,
DDF tries to print all the field values for the model which will in turn
raise a `DoesNotExist` exception on the `*_ptr` field, that masks the
real exception.

This commit catches and displays the `repr()` of exceptions raised in
when displaying field values in the `print_field_values_of_a_model()`
method, which allows the real exception to exposed via `BadDataError`.

It might be more appropriate to just not display field values for
`*_ptr` fields, though, as they are not set directly by the user.

33 of 33 new or added lines in 4 files covered. (100.0%)

2124 of 2770 relevant lines covered (76.68%)

19.03 hits per line