search

lykmapipo / kue-scheduler

A job scheduler utility for kue, backed by redis and built for node.js
246 stars 47 forks source link

use unique job's UUID when restoring a recurring schedule #91

Closed schmod closed 7 years ago

schmod commented 7 years ago

Fixes #90.

schmod commented 7 years ago

@lykmapipo Not sure how you'd prefer to test this, given that it doesn't fit neatly into any of the existing test suites.

I'd be happy to write a test (which I basically already did with the code snippet in #90), but don't want to inadvertently contradict any designs/architecture that you might have had in mind with the existing test suite.

lykmapipo commented 7 years ago

@schmod Let review the test(specs) & I will propose to you where to put it and how to test it.

gmcnaught commented 7 years ago

Oh - I've experienced this same issue @schmod - I can assist in standing up a test in the format proposed in #91