fix: SyntaxError message for v flag with RegExpValidator#validatePattern

[ota-meshi]

See eslint/eslint#17404 (comment)

[mdjermanovic]

LGTM as far as I can tell. Just left one question.

[mdjermanovic]

In this example:

const validator = new RegExpValidator();

validator.validatePattern("abcd", 0, 2, { unicode: true, unicodeSets: true });

It throws error with an index that doesn't seem to point to the / in the whole source:

{
    "message": "Invalid regular expression: /abcd/uv: Invalid regular expression flags"
    "index": 3
}

Is that okay? (I guess 5 would be wrong as out-of-range for the parsed part of the pattern).

[ota-meshi]

Thank you for checking this PR!

Hmm. I think it's probably a pre-existing bug.

For example, the following case also returns the wrong pattern.

validator.validatePattern("[A]", 0, 2, { unicode: true, unicodeSets: false });

{
    "message": "Invalid regular expression: /[A]/u: Unterminated character class",
    "index": 2,
}

I think it's a bug that doesn't occur via ESLint, so I think it can be addressed in another PR.

[ota-meshi]

I posted an issue #112.

[ota-meshi]

Thank you for the review!
I think the new problem will be fixed another time. So I'm merging this PR.

[github-actions]

🎉 This PR is included in version 4.6.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀