Options are not passed correct when scheduling jobs

[janmeier]

The documentation mentions that

options supports all properties in send()

So I assume that means I should be able to pass e.g. { retryLimit: 42 }, and have my scheduled jobs retry 42 times.

I'm adding a failing test to showcase the expected behaviour :)

The options are being correct stored in the DB, but it seems they are not being set correctly when the job is queued.

I had a brief look at the code, and I think we need to spread the options here

pg-boss/src/timekeeper.js

Line 147 in 463f2fc

({ name: QUEUES.SEND_IT, data: { name, data, options }, singletonKey: name, singletonSeconds: 60 }))

LMK if that makes sense, then I will update the PR.

[janmeier]

So I tested the changes with our production deployment at Lenus, and spreading the options object fixes the retrying issues we've seen.
So I've pushed a commit with that :)

[timgit]

thank you

[coveralls]

coverage: 100.0%. remained the same
when pulling 71be52d on janmeier:add-failing-schedule-metadata-test
into 463f2fc on timgit:master.

[jippi]

@timgit Any chance to cut a release with this fix? seem to be the only thing in HEAD compared to the last tag, so a patch release would be appreciated for ease of use for folks wanting to pick it up ❤️

[timgit]

Done!