Closed hammzj closed 5 months ago
See constructor: https://github.com/cucumber/cucumber-expressions/blob/b4d5d7025ecfe7405b5235bade7acce1d827b30b/javascript/src/ParameterType.ts#L45-L62
This also supports type
to directly define the type, but not sure if needed.
Hi @hammzj, I've fixed with this v20.0.4.
Current behavior
https://github.com/badeball/cypress-cucumber-preprocessor/blob/055d8df6a62009c94057b0d894a30e142cb87b94/lib/public-member-types.ts#L3-L7
transformer
is required, but should be optional. If a transformer is not provided for a parameter type that returns a single capture group, then it will return the original value as a string.However, if a parameter type has two capture groups, it needs a transformer that returns all arguments, or else it only returns only the first found argument in a capture group. That, however, does not require a transformer as the code will still execute.
Desired behavior
Make
transformer
be optional:Test code to reproduce
Versions
Checklist
cypress-cucumber-preprocessor@4.3.1
(package name has changed and it is no longer the most recent version, see #689).