Allow defining a job priority [implements #56]

samsondav / rihanna

Rihanna is a high performance postgres-backed job queue for Elixir

MIT License

439 stars 49 forks source link

Allow defining a job priority [implements #56] #57

Closed KushalP closed 5 years ago

KushalP commented 5 years ago

This is a work-in-progress PR.

Tasks

[X] Add priority column
[X] Update rihanna_jobs_enqueued_at_id index to use priority column
[X] Define a priority method on Rihanna.Job
[ ] Sufficient testing of the behaviour

Testing

What's the preferred way to test this behaviour? I'm a little confused with the style taken. If you could provide a single sample test case, that's probably sufficient here.

KushalP commented 5 years ago

Just realised the description is a little vague here. What's the preferred place for me to put any tests related to this new new field? How many tests are sufficient? Is setting a default value okay in this case?