-
-
Notifications
You must be signed in to change notification settings - Fork 15
Open
Labels
bugSomething isn't workingSomething isn't working
Description
Describe the bug
- Later version: 4.1.0
- Node.js version: v16.13.2
- OS & version: Arch Linux updated yesterday
When later.parse.text has errors, passing its result to later.schedule gives valid schedule
Actual behavior
If I give an invalid text string to later.parse.text, say, at 20h, the result has errors but when passed to later.schedule, it generates something that is correct, but never runs. It breaks Bree, which runs (in src/job-utils.js):
const schedule = later.schedule(later.parse.text(value));
if (schedule.isValid()) return later.parse.text(value);Expected behavior
I expect isValid to return false when the schedule is not valid.
Code to reproduce
> later.parse.text('at 20h').error
3
> later.schedule(later.parse.text('at 20h')).isValid()
true
Checklist
- I have read the documentation.
- I have tried my code with the latest version of Node.js and @breejs/later.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working