When seconds option is true expressions with 5 symbols shouldn't be valid

[ocraciun]

Tried this test and it fails:

it('should not accept 5 symbols if seconds option is true', () => {
  const valid = isValidCron('* * * * *', { seconds: true })
  expect(valid).toBeFalsy()
})

isValidCron('* * * * *', { seconds: true }) should return false when seconds option is true.

[GuillaumeRochat]

From memory, I assumed that seconds could be missing from the expression and we'd still have a valid expression, even with seconds: true.

I could change that behavior, but that is a quite different behavior. Would it work if we added a strictSeconds options instead that explicitly requires a second in the expression?

[XueMeijing]

the difference is that, when i set seconds: true, it shows the length error

[GuillaumeRochat]

@XueMeijing That is not with this cron-validator package but with cron-validate. : https://github.com/Airfooox/cron-validate

Example:
https://runkit.com/embed/bxfrpxgls3o0