mozilla / web-ext / 4835

100%

push

travis-ci

fix: support multiple values after --ignore-files again (#1652)

The yargs upgrade from 6.6.0 to 13.2.1 in edebef46eb broke the
documented ability to pass multiple files to ignore after the
`--ignore-files` (aka `-i`) parameter.

This is because yargs 11 started to treat `requiresArg` as an implicit
`nargs: 1`, i.e. to accept exactly one value only.

This patch fixes the issue by dropping `requiresArg` and validating the
number of parameters in the `execute` function instead.

491 of 557 branches covered (88.15%)

2 of 2 new or added lines in 1 file covered. (100.0%)

1305 of 1305 relevant lines covered (100.0%)

74.57 hits per line