StoneCypher / jssm / 883a90ed80991cc3c0e7c19064898c5f8705bd49

100%

master: 100%

Pull #962

github

fix(cli): parseFslArgs no longer swallows a following flag as a missing value (5.162.31)

A value-taking flag with no attached value consumed the next argv token
unconditionally, even when that token was itself a flag. `fsl-render
machine.fsl --output --stdout` set output to the literal '--stdout' and never
set the stdout boolean, so the render was written to a file named '--stdout'
in the cwd instead of to stdout.

parseFslArgs now treats a next token that starts with '--' (or the end of
argv) as a missing value and reports it, for both the long and short forms.
Single-'-' tokens are still accepted, so negative-number values (-3) keep
working, and --flag=value is unaffected.

Adds cli-utils.spec.ts cases: --output --stdout and -o --stdout both throw
"requires a value", and --width -3 still parses to the number -3.

Closes StoneCypher/fsl#1956

2645 of 2645 branches covered (100.0%)

Branch coverage included in aggregate %.

6 of 6 new or added lines in 2 files covered. (100.0%)

4281 of 4281 relevant lines covered (100.0%)

48396.23 hits per line