Python 3 support #6
Conversation
There was a problem hiding this comment.
If anyone can think of a better way of doing this, please let me know. With Django's built-in UUIDField, self.pk is already a UUID at this point.
There was a problem hiding this comment.
the model has a state field: perhaps checking self.state == STATES.NEW?
There was a problem hiding this comment.
Tried that. The tests fail. They fail because when the worker is running in a real app, the state is prodded externally by the worker process as the jobs are processed. In the tests, this prodding wasn't happening. I could do the prodding in the tests, but it feels a bit fragile..
There was a problem hiding this comment.
Then perhaps this initialisation step should be in a model classmethod or manager method for creating the job?
|
Since my problem with this is not a functional change, 🌠 |
No description provided.