fix(jobs): run periodic jobs once at start time

deis / workflow-manager

Deis Workflow Manager: Cluster First Aid

MIT License

14 stars 15 forks source link

fix(jobs): run periodic jobs once at start time #89

Closed jackfrancis closed 8 years ago

jackfrancis commented 8 years ago

a recent refactor of DoPeriodic (inadvertently) removed the initial invocation of a periodic job, and waited instead until the first frequency interval to execute. We want the first execution to be immediate.

Includes updates to tests that verify expected job execution counts after definite time lapse periods.

deis-bot commented 8 years ago

@arschles and @kmala are potential reviewers of this pull request based on my analysis of git blame information. Thanks @jackfrancis!

codecov-io commented 8 years ago

Current coverage is 30.38% (diff: 33.33%)

Merging #89 into master will decrease coverage by 0.81%

@@             master        #89   diff @@
==========================================
  Files             8          8          
  Lines           359        362     +3   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits            112        110     -2   
- Misses          220        224     +4   
- Partials         27         28     +1

Powered by Codecov. Last update 4e49625...2d4cb4c