ZeitOnline / celery_longterm_scheduler

Schedules celery tasks to run in the potentially far future
BSD 3-Clause "New" or "Revised" License
27 stars 8 forks source link

Review notes from hoshi #1

Closed zon-hoppe closed 7 years ago

zon-hoppe commented 7 years ago

Ich hab gesehen, dass du dem execute_pending den gleichen timestamp mitgegeben hast, mit dem der task versehen ist. ich hab den mal so umgebaut, dass der test testet ob auch rueckwirkend tasks abgearbeitet werden und zusaetzlich noch einen test hinzugefuegt fuer "zukuenftige tasks bitte nicht abarbeiten". ich denke das rueckwirkend sollte auf jeden fall getestet werden, wenn du dennoch einen test weiteren test fuer gleichen zeitstempel haben moechtest kann ich noch nen dritten hinzufuegen. :)