Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ExecutionTime.lastExecution doesn't work #30

Closed
sparklton opened this issue Jul 23, 2015 · 2 comments
Closed

ExecutionTime.lastExecution doesn't work #30

sparklton opened this issue Jul 23, 2015 · 2 comments

Comments

@sparklton
Copy link

The following test case fails:

DateTime now = DateTime.now();
CronParser parser = new CronParser(CronDefinitionBuilder.instanceDefinitionFor(CronType.QUARTZ));
ExecutionTime executionTime = ExecutionTime.forCron(parser.parse("0 0 3 ? * 6"));
DateTime last = executionTime.lastExecution(now);
DateTime next = executionTime.nextExecution(now);
Assert.assertNotEquals(last, next);

I expected the last to be "3 AM of the nearest Saturday looking past from now" and next to be "3 AM of the nearest Saturday looking forward". Am I wrong with my expectations?

@jmrozanec jmrozanec added the bug label Jul 27, 2015
@jmrozanec
Copy link
Owner

@sparklton you are right. This issue is very similar to the #26 we are working on. Will keep you updated about progress. Help and ideas are welcome! Thank you for reporting the issue.

jmrozanec added a commit that referenced this issue Aug 15, 2015
…ion mark handling and bad descriptions.
@jmrozanec
Copy link
Owner

@sparklton Issue fixed! Thank you for reporting this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants