6.0.1 problem when use optional

Hi. I’m not sure but I think it doesn’t work correct when use optional and then use method like toInt always convert undefined to NaN. I’ve tried this with checkSchema too. query('page').optional().isInt().toInt()

[gustavohenke]

You guys should take a look at v6.1.1. It should fix the problem!

[blargism]

The following two tests do not pass. They were run in src/matched-data.spec.ts.

describe('when option includeOptionals is false', () => {
  it('returns object without optional data', done => {
    const req = {
      headers: { foo: 'bla', bar: '123' },
    };

    const middleware = check(['foo', 'bar', 'baz'])
      .optional()
      .trim()
      .isString();

    middleware(req, {}, () => {
      const data = matchedData(req, { includeOptionals: false });
      expect(data).toHaveProperty('bar', '123');
      expect(data.baz).toBeUndefined();

      done();
    });
  });
});

describe('when option includeOptionals is not set', () => {
  it('returns object without optional data', done => {
    const req = {
      headers: { foo: 'bla', bar: '123' },
    };

    const middleware = check(['foo', 'bar', 'baz'])
      .optional()
      .trim()
      .isString();

    middleware(req, {}, () => {
      const data = matchedData(req);
      expect(data).toHaveProperty('bar', '123');
      expect(data.baz).toBeUndefined();

      done();
    });
  });
});