paulocheque / django-dynamic-fixture

A complete library to create dynamic model instances for testing purposes.
http://django-dynamic-fixture.readthedocs.io/
Other
390 stars 67 forks source link

Allow named primary keys (and failing testcase) #94

Closed MrWeeble closed 5 years ago

MrWeeble commented 6 years ago

I think that this fixes #32 though I have to confess I haven't looked too deeply into the internals so I don't know if this will have any unintended consequences, but the tests seem to still work.

paulocheque commented 5 years ago

Did you try this and are you confident it is ok now? Then we can merge into master.

MrWeeble commented 5 years ago

It worked for me when previously it wasn't, and it doesn't seem to have introduced any other errors